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

Duplicate 'colr' box definitions #83

Closed
palemieux opened this issue Dec 31, 2019 · 7 comments · Fixed by #180
Closed

Duplicate 'colr' box definitions #83

palemieux opened this issue Dec 31, 2019 · 7 comments · Fixed by #180

Comments

@palemieux
Copy link
Contributor

palemieux commented Dec 31, 2019

The colr entry at CSV/item-properties.csv duplicates that at CSV/sample-entries-boxes.csv. The defining specification for the former explicitly states that The colour information 'colr' descriptive item property has the same syntax as the ColourInformationBox as defined in [the latter]

@palemieux palemieux changed the title 'colr' box defined Duplicate 'colr' box definitions Dec 31, 2019
@dwsinger
Copy link
Collaborator

dwsinger commented Jan 7, 2020

Right, that box is valid as (a) a sample-entry extension and (b) an item property, which are formally separate namespaces. So it's registered into both. What is the issue?

@palemieux
Copy link
Contributor Author

which are formally separate namespaces.

Oh. This not clear. Where are the "Item Property" and "Sample-entry" namespaces defined?

@dwsinger
Copy link
Collaborator

dwsinger commented Jan 7, 2020

well, item properties are included in the item property container (only). sample entry extensions are in sample entries (only). we use the same 4CC since it's the same syntax.

@palemieux
Copy link
Contributor Author

item property container

Do you mean as as elements of ItemPropertyContainerBox as specified in ISO/IEC 23008-12?

sample entry extensions are in sample entries (only)

Do you mean as elements of extensions of SampleEntry as specified in ISO/IEC 14496-12?

@dwsinger
Copy link
Collaborator

dwsinger commented Jan 7, 2020

yes

@palemieux
Copy link
Contributor Author

Ok. I suggest adding these clarifications in the corresponding .csv files and/or below the relevant heading in the generated pages, e.g. http://mp4ra.org/#/item_properties

@dwsinger
Copy link
Collaborator

we'll look at more specific links between the tables and the defining context.

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 a pull request may close this issue.

2 participants