Skip to content

exp: add stereoscopic camera gui - #46

Merged
elliot-ausocean merged 2 commits into
mainfrom
add-stereoscopic-gui
Jan 27, 2026
Merged

elliot-ausocean merged 2 commits into
mainfrom
add-stereoscopic-gui

Conversation

@elliot-ausocean

@elliot-ausocean elliot-ausocean commented Jan 21, 2026 •

Copy link
Copy Markdown
Collaborator

This change adds a basic stereoscopic camera gui used for creating test recordings. It is designed to be easy to use in the field. This code isn't the neatest but it is functional enough to be used as an internal testing tool.

This change adds a basic steroscopic camera gui used for creating test
recordings. It is designed to be easy to use in the field.

@Saxon1 Saxon1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems OK as exp code, but the left and right files seem very similar in a lot of ways. I asked ChatGPT about this (https://chatgpt.com/share/6971c6e9-1e84-8004-9751-65f8a7674176) might be useful for you.

Comment thread exp/stereo-gui/left.py Outdated
@@ -0,0 +1,474 @@
import time

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File header.

@trekhopton trekhopton left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just add a header comment at the top of the file.

This makes the code much easier to read and modify. It also splits the
webpage into a separate html file, which allows for easier changes.
@elliot-ausocean
elliot-ausocean merged commit 4c1a00a into main Jan 27, 2026
1 check passed
@elliot-ausocean
elliot-ausocean deleted the add-stereoscopic-gui branch January 27, 2026 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants