Temporary PR to generate a full bikeshed diff compared to v1.1.0 #293

leo-barnes · 2024-10-25T18:20:15Z

Turns out there's no documented way of generating the "diff" document yourself, so this temporary PR will serve as the generator for it.

This is the v1.2.0 release candidate diff against v1.1.0.

Preview | Diff

Fix white spaces, indentation, newlines, missing closing HTML elements and mismatched opening/closing HTML elements.

Changed alpha_video.avif: - Applied Jon's fix to set the correct iref - Stripped out optional config OBUs from av1C - Added explicit NCLX with 1,13,1,0 - Set track_in_movie to true in track

1. Add lsel property with value 0xFFFF 2. Add NCLX colr box with value 1,13,1,0 (since this matches what the PNGs look like)

Microsoft/Chimera_*_cropped_*.avif: - Truncated crop to integers - Stripped optional config OBUs Link-U/kimono*.crop.avif: - Reordered property associations so 'ispe' comes before transform properties - Rounded up crop to integers (that matches the PNGs) - Added NCLX with values 1,13,9,0 - Stripped optional config OBUs

…70.avif to fix issue 151 Summer_in_Tomsk_720p_5x4_grid.avif: - Added NCLX 1,13,6,0 for all items - Added pixi for grid item - Strip optional config OBUs from av1C Chimera-AV1-10bit-480x270.avif: - Added NCLX 1,13,6,0 - Added pixi - Strip optional config OBUs from av1C - Set track_in_movie to true for track

Changes done: 1. Change order of item property associations to place ispe first 2. Add colr-nclx box with values 1,13,9,0 since this is what Sequence Header says 3. Strip out optional config OBUs from av1C

…AVIF files

Also fix crash in sanitizer for some profile 2 files testFiles/Link-U/hato.profile0.10bpc.yuv420.monochrome.no-cdef.no-restoration.avif testFiles/Link-U/hato.profile0.8bpc.yuv420.monochrome.no-cdef.avif testFiles/Link-U/hato.profile2.10bpc.yuv422.monochrome.no-cdef.no-restoration.avif testFiles/Link-U/hato.profile2.8bpc.yuv422.monochrome.no-cdef.avif Remove optional config OBUs Insert default colr box Fix incorrect pixi testFiles/Link-U/fox.profile2.10bpc.yuv422.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.odd-height.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.odd-width.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.odd-width.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.odd-width.odd-height.avif testFiles/Link-U/hato.profile2.10bpc.yuv422.no-cdef.no-restoration.avif testFiles/Link-U/hato.profile2.12bpc.yuv422.monochrome.no-cdef.no-restoration.avif testFiles/Link-U/hato.profile2.12bpc.yuv422.no-cdef.no-restoration.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.odd-height.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.odd-width.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.odd-width.odd-height.avif Update av1C: av1C[chroma_subsampling_x] does not match Sequence Header OBU. '0' != '1' Remove optional config OBUs Insert default colr box testFiles/Link-U/fox.profile0.10bpc.yuv420.avif testFiles/Link-U/fox.profile0.10bpc.yuv420.monochrome.avif testFiles/Link-U/fox.profile0.10bpc.yuv420.monochrome.odd-height.avif testFiles/Link-U/fox.profile0.10bpc.yuv420.monochrome.odd-width.avif testFiles/Link-U/fox.profile0.10bpc.yuv420.monochrome.odd-width.odd-height.avif testFiles/Link-U/fox.profile0.10bpc.yuv420.odd-height.avif testFiles/Link-U/fox.profile0.10bpc.yuv420.odd-width.avif testFiles/Link-U/fox.profile0.10bpc.yuv420.odd-width.odd-height.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.monochrome.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.monochrome.odd-height.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.monochrome.odd-width.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.monochrome.odd-width.odd-height.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.odd-height.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.odd-width.avif testFiles/Link-U/fox.profile0.8bpc.yuv420.odd-width.odd-height.avif testFiles/Link-U/fox.profile1.10bpc.yuv444.avif testFiles/Link-U/fox.profile1.10bpc.yuv444.odd-height.avif testFiles/Link-U/fox.profile1.10bpc.yuv444.odd-width.avif testFiles/Link-U/fox.profile1.10bpc.yuv444.odd-width.odd-height.avif testFiles/Link-U/fox.profile1.8bpc.yuv444.avif testFiles/Link-U/fox.profile1.8bpc.yuv444.odd-height.avif testFiles/Link-U/fox.profile1.8bpc.yuv444.odd-width.avif testFiles/Link-U/fox.profile1.8bpc.yuv444.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.monochrome.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.monochrome.odd-height.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.monochrome.odd-width.avif testFiles/Link-U/fox.profile2.10bpc.yuv422.monochrome.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.monochrome.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.monochrome.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.monochrome.odd-width.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.monochrome.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.odd-width.avif testFiles/Link-U/fox.profile2.12bpc.yuv420.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.monochrome.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.monochrome.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.monochrome.odd-width.avif testFiles/Link-U/fox.profile2.12bpc.yuv422.monochrome.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.monochrome.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.monochrome.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.monochrome.odd-width.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.monochrome.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.odd-height.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.odd-width.avif testFiles/Link-U/fox.profile2.12bpc.yuv444.odd-width.odd-height.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.monochrome.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.monochrome.odd-height.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.monochrome.odd-width.avif testFiles/Link-U/fox.profile2.8bpc.yuv422.monochrome.odd-width.odd-height.avif testFiles/Link-U/hato.profile0.10bpc.yuv420.no-cdef.no-restoration.avif testFiles/Link-U/hato.profile0.8bpc.yuv420.no-cdef.avif testFiles/Link-U/hato.profile2.12bpc.yuv422.monochrome.avif Remove optional config OBUs Insert default colr box

Closes #167. This is a rebase and fixup of PR #170. The section on box requirements for sequences has been removed until someone has taken the time to fully go through it.

This closes #182

This closes #179

Add tmap and altr text

Introduce Sample Transform Derived Image Items in AVIF. Add the Definition, Syntax, Semantics and Constraints sections.

Make it obvious that AVIF also supports 16-bit.

Increase unary operator range to 64+ to leave room for more operand token types.

Clarify that non-essential item properties may use other versions. This closes #247.

along other supported features.

Moved box type definition anchors to the root spec listed in table of boxes Added links to all box types in table of boxes Sorted all anchors, removed duplicates Removed duplicated lists of boxes

To make sure the first sentence of each derived image item section is rendered in a similar way.

Same as for Operating Point Selector Property sections.

Use box names instead of some property 4CC types. Add a comment about creating the stack in the Syntax section.

- Change optional boxes table to have same layering as required boxes table. - Change text on box order - Change text on `dinf`

Section 9.1 also applies to an AVIF file containing image items and image sequences, so remove "only" from "containing only image items".

Otherwise, it seems that it is the specifications that shall be respected. Also make the second instance of this sentence more similar to the first instance.

The 'colr' box specifies color information.

In prevision of the major_brand field of the FileTypeBox not needing to be repeated in the compatible_brands field.

Fix use of br tags in SATO examples

y-guyon and others added 30 commits June 26, 2024 16:21

Begin new Working Group Draft version

ce1174e

Make indent and formatting consistent (#204)

8ebeb5f

Fix white spaces, indentation, newlines, missing closing HTML elements and mismatched opening/closing HTML elements.

Fix use of <dfn value> and some indenting.

fa97d56

Add changes since v1.1.0 entry

d32b750

Mark change as editorial

b0b1c4d

Rebase fixes

2bd17c2

Switch indentation to 4 spaces

79bd06f

Add asserts for shoulds and shalls

5809865

Fix issue #177

b510e72

Changed alpha_video.avif: - Applied Jon's fix to set the correct iref - Stripped out optional config OBUs from av1C - Added explicit NCLX with 1,13,1,0 - Set track_in_movie to true in track

Changed Xiph multilayer test content with the following changes:

0a49303

1. Add lsel property with value 0xFFFF 2. Add NCLX colr box with value 1,13,1,0 (since this matches what the PNGs look like)

Add primary item to Chimera-AV1-10bit-480x270.avif

e624c37

Update testFiles/Link-U/kimono*.avif files to make them spec compliant

30a2f4c

Changes done: 1. Change order of item property associations to place ispe first 2. Add colr-nclx box with values 1,13,9,0 since this is what Sequence Header says 3. Strip out optional config OBUs from av1C

Added sanitizer tool that can fix some commonly identified issues in …

aa906e0

…AVIF files

Add iso8 to sequence test files

b9ea3d8

Add list of required boxes for image items.

c1f956f

Closes #167. This is a rebase and fixup of PR #170. The section on box requirements for sequences has been removed until someone has taken the time to fully go through it.

Add list of required boxes and versions.

732e53d

This closes #182

Fix broken link to latest draft at the top of the document.

cc68618

This closes #179

Relax constraint on derivation chains

bbeec5f

Change 'ispe' width to correspond to UpscaledWidth

7a095c1

Clarify relationship between av1C and metadata OBUs

b2b2a23

Update list of other item properties

01e3ea3

Reword av1C vs pixi conformance

24508d6

Add tmap and altr text (#239)

3d2d175

Add tmap and altr text

Replace AV1 still_picture=1 shoulds with a note (#228)

2286f6c

Add Sample Transform Derived Image Items (#224)

70d14e6

Introduce Sample Transform Derived Image Items in AVIF. Add the Definition, Syntax, Semantics and Constraints sections.

Add Sample Transform rationale (#245)

b21779d

Make it obvious that AVIF also supports 16-bit.

Specify Sample Transform syntax versioning (#246)

b6758ca

Increase unary operator range to 64+ to leave room for more operand token types.

leo-barnes and others added 30 commits September 27, 2024 10:51

Clarify that non-essential item properties may use other versions (#250)

434dbe1

Clarify that non-essential item properties may use other versions. This closes #247.

Remove "value" from sato definition (#255)

1b5720e

Add Appendix A on sato examples (#253)

fa78f2b

Add refs, fix wording and format in clauses 1,2 (#257)

506c1e4

Add sato, alpha, depth, progressive in Scope (#258)

e0e048e

along other supported features.

Fix final review comments (#256)

727501a

Moved box type definition anchors to the root spec listed in table of boxes Added links to all box types in table of boxes Sorted all anchors, removed duplicates Removed duplicated lists of boxes

Use Markdown syntax coloring for code snippets (#259)

94c679e

Consistently use dfn noexport for derived items (#261)

f0b2f29

To make sure the first sentence of each derived image item section is rendered in a similar way.

Remove Sample Transform sections from TOC (#262)

b1a66af

Same as for Operating Point Selector Property sections.

Add references in the Sample Transform sections (#263)

b89eb19

Use box names instead of some property 4CC types. Add a comment about creating the stack in the Syntax section.

Indent notes as the list items they refer to (#266)

263536c

Remove inconsistent dots in 9.1.2 (#267)

08f65d4

Update change-list (#271)

d980c52

Use exact tokens as sato math expr example (#272)

439b57e

Change optional boxes table layering (#273)

751dffe

- Change optional boxes table to have same layering as required boxes table. - Change text on box order - Change text on `dinf`

Use precise dim/chan constraints in sato examples (#287)

7e5c8c7

Add hidden image item recommendation (#288)

d81c366

Minor edits in Section 8 Profiles

198515c

Remove an extraneous "only"

83d5188

Section 9.1 also applies to an AVIF file containing image items and image sequences, so remove "only" from "containing only image items".

Add the word "bits" to the amount of shift

20b9eaf

Add the word "added" to "parentheses for clarity"

a83f921

Clarify it is the order that shall be respected

2acad7e

Otherwise, it seems that it is the specifications that shall be respected. Also make the second instance of this sentence more similar to the first instance.

Add the word "color"

aa77c70

The 'colr' box specifies color information.

Use full names of HDR image item properties

486d3ff

Minor edits and punctuation fixes in sato examples

e352a2f

Remove mentions of ftyp compatible_brands (#289)

2a57aca

In prevision of the major_brand field of the FileTypeBox not needing to be repeated in the compatible_brands field.

Remove avio brand recommendation (#265)

9c389bb

Fix use of br tags in SATO examples (#292)

48dc1c6

Fix use of br tags in SATO examples

Update date

6820e20

"Empty" commit to trigger a bikeshed diff to get generated

0c0466b

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Temporary PR to generate a full bikeshed diff compared to v1.1.0 #293

Temporary PR to generate a full bikeshed diff compared to v1.1.0 #293

leo-barnes commented Oct 25, 2024 •

edited by pr-preview bot

Loading

Temporary PR to generate a full bikeshed diff compared to v1.1.0 #293

Are you sure you want to change the base?

Temporary PR to generate a full bikeshed diff compared to v1.1.0 #293

Conversation

leo-barnes commented Oct 25, 2024 • edited by pr-preview bot Loading

leo-barnes commented Oct 25, 2024 •

edited by pr-preview bot

Loading