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

Consider defining a Metadata OBU for private data #514

Open
cconcolato opened this issue Jun 26, 2023 · 7 comments
Open

Consider defining a Metadata OBU for private data #514

cconcolato opened this issue Jun 26, 2023 · 7 comments
Labels

Comments

@cconcolato
Copy link
Contributor

One could store SMPTE timecodes for example, or other metadata.

@sunghee-hwang
Copy link
Collaborator

Can we postpone this to v2?

@sunghee-hwang
Copy link
Collaborator

sunghee-hwang commented Jun 27, 2023

If we do have, then the below syntaxes of Timecode OBU payload would be good enough?

class timecode_obu() {
unsigned int (32) smpte_timecode_bcd;
}

'smpte_timecode_bcd' indicates HH:MM:SS:FF format and is represented using binary-coded decimal. Each variable represents in the range from 0-9 within the half-byte boundary. Where, H: hour, M: minute, S: second and F: frame.

For example, 15 hours 05 minutes 48 seconds 11th frame is indicated as follows:

  • 1st byte (MSB): 0x15
  • 2nd byte: 0x05
  • 3rd byte: 0x48
  • 4th byte (LSB): 0x10
    NOTE: It can represent up to 100 frames (0x00 ~ 0x99) per second.

@cconcolato
Copy link
Contributor Author

There should be a way to carry generic metadata payloads, potentially defined elsewhere:

It could be simply reserving an OBU type, having an leb128 metadata type, as in AV1: https://aomedia.org/av1/specification/syntax/#general-metadata-obu-syntax

@sunghee-hwang
Copy link
Collaborator

Ok, I get it.
Then, I think that we have no action as long as there are enough number of reserved obu_types.
Am I right?

@cconcolato
Copy link
Contributor Author

Let's discuss in meeting to see if we want that for v1 or v2.

@tdaede
Copy link
Collaborator

tdaede commented Jun 27, 2023

Let's leave this for v2. (Regardless if we reserve a OBU type for this or not, it does not change decoder behavior wrt skipping reserved OBUs).

@cconcolato
Copy link
Contributor Author

For context, AOM is working on a potential new spec/registry of metadata, that could be useful to link with IAMF: https://aomediacodec.github.io/metadata-registry/

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

No branches or pull requests

3 participants