Skip to content

sync-analyser: initial version - #48

Merged
elliot-ausocean merged 3 commits into
mainfrom
03-26-sync-analyser_initial_version
Apr 21, 2026
Merged

elliot-ausocean merged 3 commits into
mainfrom
03-26-sync-analyser_initial_version

Conversation

@elliot-ausocean

Copy link
Copy Markdown
Collaborator

This is the initial version of sync analyser, a program which can be used to objectively measure the synchronisation of two stereoscopic cameras. It includes a GUI where you can step through the frames manually, which is useful for manual analysis and debugging. There is also a script that can ingest an entire folder of stereoscopic video and output a csv containing the sync offsets, which is useful for testing with multiple trials.
Stereo Sync Analyzer_screenshot_26 03 2026

This is the initial version of sync analyser.
This change removes some unnecessary commands and makes the threshold
for the dot a constant that is easier to change.

@ao-david ao-david left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Very Cool!

Comment thread exp/sync-analyser/bulk_sync_analyser.py Outdated
Comment thread exp/sync-analyser/README.md Outdated
ts_r = right_cap.get(cv2.CAP_PROP_POS_MSEC)

if ts_l < ts_r - MAX_TIMECODE_DIFF:
ret_l, frame_l = left_cap.read()

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does this not change the sync calculations entirely? or is this accounted for somehow?

Comment thread exp/sync-analyser/sync_analyser.py Outdated
Comment thread exp/sync-analyser/sync_analyser.py Outdated
Comment thread exp/sync-analyser/sync_analyser.py
Comment thread exp/sync-analyser/sync_analyser.py Outdated
Comment thread exp/sync-analyser/sync_analyser.py Outdated
@elliot-ausocean
elliot-ausocean merged commit b112a4e into main Apr 21, 2026
1 check passed
@elliot-ausocean
elliot-ausocean deleted the 03-26-sync-analyser_initial_version branch April 21, 2026 05:34
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.

2 participants