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 Mix Presentation OBU human-readable annotation fields #290

Open
jwcullen opened this issue Feb 24, 2023 · 2 comments
Open

Improve Mix Presentation OBU human-readable annotation fields #290

jwcullen opened this issue Feb 24, 2023 · 2 comments
Assignees
Milestone

Comments

@jwcullen
Copy link
Collaborator

This probably will need more discussion from others to flesh out. But here are some initial thoughts / suggestions:

  1. These should be some form of unicode (probably UTF-8) to support human-readable strings in languages that have non-ascii characters.
  2. We should consider cases where streams would be merged. Submixes could be combined together. What happens to annotations? Also do we need more than 1 substream in simple / base profile?
  3. We should break out useful "dimensions" in the mix_presentation_annotations and mix_presentation_element_annotations class defined in the spec.
  4. Add a section to recommend displaying these fields as appropriate to the user.
  5. Consider which portions need annotations (mix, submix audio element, submix). Currently only the former 2 have annotations.

Potential ideas for separate fields within the *_annotations classes.

  • Language Code
  • Channel layout (or maybe not because this would be duplicate information and annotations should just be informative)
  • Content type an enum or a string (commentary, audio, dialogue, music, ...). I think there is an ISOBMFF field we could get inspriation from.
  • Mix rank (lowest rank is tiebreaker as a "default")
  • Potentially other fields or put some room in here for extensibility?
@tdaede tdaede self-assigned this Feb 28, 2023
@tdaede
Copy link
Collaborator

tdaede commented Apr 3, 2023

We decided to leave this as-is for now.

@tdaede tdaede added the postpone label Apr 3, 2023
@cconcolato
Copy link
Contributor

Related to #404

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants