Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

encoderd: ensure consistent frames per segment #34462

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

deanlee
Copy link
Contributor

@deanlee deanlee commented Jan 24, 2025

Ensures each segment contains exactly SEGMENT_LENGTH * MAIN_FPS frames, even during camera re-alignment.

It replaces the frame_id-based logic with a frame counter, guaranteeing consistent segment length regardless of camera re-alignment or frame ID jumps.

@deanlee deanlee force-pushed the encoderd_ensure_frames_count_per_segment branch 2 times, most recently from c2f7b8d to 95f34cf Compare January 24, 2025 06:40
@deanlee deanlee marked this pull request as ready for review January 24, 2025 12:42
@deanlee deanlee force-pushed the encoderd_ensure_frames_count_per_segment branch from 95f34cf to 69a037e Compare January 30, 2025 08:21
Copy link
Contributor

github-actions bot commented Feb 9, 2025

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@github-actions github-actions bot added the stale label Feb 9, 2025
@deanlee deanlee force-pushed the encoderd_ensure_frames_count_per_segment branch from 69a037e to a6d7f14 Compare February 9, 2025 04:24
@github-actions github-actions bot removed the stale label Feb 10, 2025
@deanlee deanlee force-pushed the encoderd_ensure_frames_count_per_segment branch from a6d7f14 to 6eb3fa5 Compare February 11, 2025 11:13
@adeebshihadeh
Copy link
Contributor

It's not clear what exactly should be done in this case. Let me think about it some more.

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