-
Notifications
You must be signed in to change notification settings - Fork 6
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
Contribute the conformance file provided by Samsung #54
Comments
Hi Dmitri -- I don't understand this is just a sample file that is actually
hosted on the HDR10+ LLC website. It doesn't need to be part of conformance
it's simply a deep link from the Wiki to a sample file.
Best Regards,
Bill
…On Mon, May 22, 2023 at 6:07 PM Dimitri Podborski ***@***.***> wrote:
We currently have a file in our wiki
<https://github.com/AOMediaCodec/av1-hdr10plus/wiki> which is not yet
included into conformance suite.
@quantizationbit <https://github.com/quantizationbit> could you try to
create a new branch either directly in the main repo, or in your fork and
follow the guidelines from
https://github.com/AOMediaCodec/av1-hdr10plus/tree/main/conformance and
see if you can add that file to the conformance suite?
—
Reply to this email directly, view it on GitHub
<#54>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGTCDONRKUECSOODCV6G3DXHQEU7ANCNFSM6AAAAAAYLEW6VY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@quantizationbit the question is: Can we also add this file to the conformance suite? |
Yes it is fine to add
…On Wed, May 31, 2023 at 3:30 PM Cyril Concolato ***@***.***> wrote:
@quantizationbit <https://github.com/quantizationbit> the question is:
Can we also add this file to the conformance suite?
It would be automatically processed by the tools that we currently use for
building the compliance report (ComplianceWarden ...). This would be
helpful to detect regression in ComplianceWarden.
It would also be useful for validating some rules that are currently not
validated by any streams (chroma sample position = 2).
—
Reply to this email directly, view it on GitHub
<#54 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGTCDNRBSU6D3Y2H5S2DSDXI7BBZANCNFSM6AAAAAAYLEW6VY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@quantizationbit is there a license for that file? the conformance software allows to add a text license to the metadata file associated with the conformance file. If yes, could you give me a pointer? |
@podborski did you get an answer on the license? Is this still blocking this issue? |
No I can no longer access emails so we previously decided to delete that
file and replace with a short encode. I'll reforward the email.
…On Tue, Nov 28, 2023 at 2:50 PM Cyril Concolato ***@***.***> wrote:
@podborski <https://github.com/podborski> did you get an answer on the
license? Is this still blocking this issue?
—
Reply to this email directly, view it on GitHub
<#54 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGTCDPH7ZXMEYHMJPFFMLDYGZTCXAVCNFSM6AAAAAAYLEW6V2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQHA4DENBYGM>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
@quantizationbit do you have any update on that short encode? |
We currently have a file in our wiki which is not yet included into conformance suite.
@quantizationbit could you try to create a new branch either directly in the main repo, or in your fork and follow the guidelines from https://github.com/AOMediaCodec/av1-hdr10plus/tree/main/conformance and see if you can add that file to the conformance suite?
The text was updated successfully, but these errors were encountered: