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

Add default global scale value if missing in the annotation hook #647

Merged

Conversation

eloisebrosseau
Copy link
Contributor

Add default global scale value if missing

Summarize your change.

A default global scale value is calculated based on the aspect ratio of the first source if it couldn't be obtained from the OTIO.

Describe the reason for the change.

When no available_image_bounds is present in the OTIO, no global scale value was being calculated. This was causing an issue where the annotations couldn't be displayed in RV during a Live Review Session.

Describe what you have tested and on which operating system.

Successfully test on MacOS

Copy link
Contributor

@bernie-laberge bernie-laberge left a comment

Choose a reason for hiding this comment

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

LGTM
Thanks!

@eloisebrosseau eloisebrosseau enabled auto-merge (squash) December 12, 2024 19:57
@eloisebrosseau eloisebrosseau merged commit e015eb2 into AcademySoftwareFoundation:main Dec 12, 2024
10 checks passed
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