w3c / png

Maintenance of the PNG specification
https://w3c.github.io/png/
Other
44 stars 11 forks source link

first draft of impl report #445

Closed svgeesus closed 5 months ago

svgeesus commented 5 months ago

Captures a static snapshot of the automatic wpt tests, plus results of manual tests.

Summarizes current state of non-browser implementations.

ProgramMax commented 5 months ago

I'm still reviewing this. But I assume the ~$index.html file shouldn't be added.

digitaltvguy commented 5 months ago

I added a MediaInfo example and changed the MaxFALL #.

svgeesus commented 5 months ago

But I assume the ~$index.html file shouldn't be added.

Huh. I don't have one my local directory. But yes.

Ah, I see it came in a later commit

digitaltvguy commented 5 months ago

Ahhh. Very cool. Thx

From: Chris Lilley @.> Date: Friday, April 5, 2024 at 12:39 PM To: w3c/PNG-spec @.> Cc: Seeger, Chris (NBCUniversal) @.>, Review requested @.> Subject: [EXTERNAL] Re: [w3c/PNG-spec] first draft of impl report (PR #445)

@svgeesus commented on this pull request.


In Implementation_Report_3e/index.htmlhttps://urldefense.com/v3/__https:/github.com/w3c/PNG-spec/pull/445*discussion_r1553983079__;Iw!!PIZeeW5wscynRQ!rjQrmFR13Fws4-p2HihGG356-KS5KNK1re7HkYisr_l4NVJQOYkEuXq696-eovI-2_b4LRjOqdd_7F-vrfjj2kmG5w$:

@@ -545,7 +545,7 @@

2. Coding Independent Code Points (CICP) from ITU-R H.273

                 </ul>

                 <p>Implemented in <a href="https://affinity.serif.com/en-us/photo/"><strong>Affinity<https://urldefense.com/v3/__https:/affinity.serif.com/en-us/photo/*22*3E*3Cstrong*3EAffinity__;JSUlJQ!!PIZeeW5wscynRQ!rjQrmFR13Fws4-p2HihGG356-KS5KNK1re7HkYisr_l4NVJQOYkEuXq696-eovI-2_b4LRjOqdd_7F-vrfhc46KC8Q$> Photo 2</strong></a></p>

                 <ul>

No, this is incorrect. Their own announcement (see screenshot in this implementation report) says 32 bits. Per component!

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/w3c/PNG-spec/pull/445*pullrequestreview-1983690720__;Iw!!PIZeeW5wscynRQ!rjQrmFR13Fws4-p2HihGG356-KS5KNK1re7HkYisr_l4NVJQOYkEuXq696-eovI-2_b4LRjOqdd_7F-vrfgu34sT7g$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AANIO6DE6UJM2WJ42LRMME3Y33HQVAVCNFSM6AAAAABFV6WO72VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSOBTGY4TANZSGA__;!!PIZeeW5wscynRQ!rjQrmFR13Fws4-p2HihGG356-KS5KNK1re7HkYisr_l4NVJQOYkEuXq696-eovI-2_b4LRjOqdd_7F-vrfhjuGneXw$. You are receiving this because your review was requested.Message ID: @.***>

digitaltvguy commented 5 months ago

Hi All,

I will not be able to attend Monday’s meeting as I will be @ NAB.

Chris’s – Is there anything more you need from me on the implementation report?

Best,

Chris