AOMediaCodec / av1-hdr10plus

Official AOM repository for the development of the specification for the use of AV1 and HDR10+.
https://aomediacodec.github.io/av1-hdr10plus/
16 stars 6 forks source link

Contribute the conformance file provided by Samsung #54

Open podborski opened 1 year ago

podborski commented 1 year ago

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?

quantizationbit commented 1 year ago

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 https://github.com/AOMediaCodec/av1-hdr10plus/issues/54, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGTCDONRKUECSOODCV6G3DXHQEU7ANCNFSM6AAAAAAYLEW6VY . You are receiving this because you were mentioned.Message ID: @.***>

cconcolato commented 1 year ago

@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).

quantizationbit commented 1 year ago

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 https://github.com/AOMediaCodec/av1-hdr10plus/issues/54#issuecomment-1571048233, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGTCDNRBSU6D3Y2H5S2DSDXI7BBZANCNFSM6AAAAAAYLEW6VY . You are receiving this because you were mentioned.Message ID: @.***>

podborski commented 1 year ago

@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?

cconcolato commented 7 months ago

@podborski did you get an answer on the license? Is this still blocking this issue?

quantizationbit commented 7 months ago

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 https://github.com/AOMediaCodec/av1-hdr10plus/issues/54#issuecomment-1830882483, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGTCDPH7ZXMEYHMJPFFMLDYGZTCXAVCNFSM6AAAAAAYLEW6V2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQHA4DENBYGM . You are receiving this because you were assigned.Message ID: @.***>