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

Require support for DFXP presentation and transformation profiles (#3… #565

Merged
merged 2 commits into from
Jan 30, 2018

Conversation

skynavga
Copy link
Collaborator

…64).

Closes #364.

@skynavga skynavga added this to the Editor's CR Work List milestone Jan 15, 2018
@skynavga skynavga self-assigned this Jan 15, 2018
Copy link
Contributor

@palemieux palemieux left a comment

Choose a reason for hiding this comment

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

Why is the line "The processor supports the DFXP Presentation profile as specified by G.5 DFXP Presentation Profile" needed if the DFXP Presentation profile is a subset of "TTML2 Presentation Profile"?

@skynavga
Copy link
Collaborator Author

Because (1) it must recognize the profile designator, and (2) for avoidance of doubt (in case something about the TTML2 profile superset turns out to contradict the DFXP profile).

@palemieux
Copy link
Contributor

Because (1) it must recognize the profile designator

Yes.

(in case something about the TTML2 profile superset turns out to contradict the DFXP profile).

Ok. What it would mean for the processor to support contradicting features in the same namespace? If such a contradiction exists, shouldn't it be noted in the feature specification itself?

@skynavga
Copy link
Collaborator Author

skynavga commented Jan 17, 2018 via email

@palemieux
Copy link
Contributor

In any case, there is no harm in calling out independent support for the DFXP profiles.

Weren't we trying to avoid a normative reference to TTML1?

@skynavga
Copy link
Collaborator Author

We were, but it is unavoidable given the resolution to mandate support, which means a normative reference.

@palemieux
Copy link
Contributor

@skynavga I think it would be easier to assume that DFXP is a subset of TTML2, and avoid a normative reference. Just a suggestion at this point, and I will approve the PR as-is.

@skynavga
Copy link
Collaborator Author

When reviewing the matter of TTML1 reference during this process, I had already decided we needed to make the reference normative due the other citations, e.g., in sections 3.3, 5.2.3.1, and 6.2.12.

Copy link

@andreastai andreastai left a comment

Choose a reason for hiding this comment

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

I have no specific view on this issue, so abstain.

@skynavga skynavga merged commit b8d4f7b into master Jan 30, 2018
@skynavga skynavga removed their assignment Jan 30, 2018
@skynavga skynavga deleted the issue-0364-require-dfxp-profiles branch March 9, 2018 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants