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

obs-outputs: Use packet time callback for frame-accurate chapter markers #11659

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

Conversation

derrod
Copy link
Member

@derrod derrod commented Dec 22, 2024

Description

Makes chapter markers frame-accurate by using packet timing callback introduced for BPM.

Motivation and Context

When there is a significant delay between a frame being rendered and submitted to the output, e.g. due to lookahead or audio buffering, the chapter markers would previously be too early in the file as there was no way to determine the packet's original rendering time. This change fixes that.

How Has This Been Tested?

Recorded a video at 30 fps with rc-lookahead=30, confirmed chapter markers are now inserted with a 1-second delay at the correct frame.

Types of changes

  • Tweak (non-breaking change to improve existing functionality)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@derrod derrod force-pushed the hybrid-mp4-accurate-chapters branch from 0b78409 to 9b70fb2 Compare December 23, 2024 08:51
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.

1 participant