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

Improve translation for AAF metadata properties from OTIO #60

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

Conversation

ghadd
Copy link

@ghadd ghadd commented Feb 26, 2025

Improved translation of AAF metadata stored in the OTIO back to AAF by generalizing the default descriptor creation and processing.

  • Resolved overlap of the properties that apply to CDCI descriptor only with RGBA descriptor and vice-versa.
  • Generalized instantiation of rational properties using the native AAF Rational type.
  • Generalized instantiation of other properties trying to enforce conversion to the native type.
  • Add harness for the properties that are not supported by the descriptor, they are applied to.

The issues were encountered while doing AAF-OTIO-AAF roundtrip with files that yield an RGBA descriptor.

Copy link

linux-foundation-easycla bot commented Feb 26, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: ghadd / name: Daniil Timachov (3e0ffd4)

@ghadd ghadd force-pushed the fix/default-descriptor-translation branch from 0d0e676 to dd6ba33 Compare February 26, 2025 13:42
Improved translation of AAF metadata stored in the OTIO back to AAF by generalizing the default descriptor creation and processing.

- Resolved overlap of the properties that apply to CDCI descriptor only with RGBA descriptor and vice-versa.
- Generalized instantiation of rational properties using the native AAF Rational type.
- Generalized instantiation of other properties trying to enforce conversion to the native type.
- Add harness for the properties that are not supported by the descriptor, they are applied to.

Signed-off-by: Daniil Timachov <daniiltimachov@gmail.com>
@ghadd ghadd force-pushed the fix/default-descriptor-translation branch from dd6ba33 to 3e0ffd4 Compare February 26, 2025 13:49
@jminor jminor requested review from markreidvfx and timlehr March 2, 2025 05:40
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