w3c / png

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

Call for Consensus: publish Candidate Recommendation #453

Closed ProgramMax closed 2 months ago

ProgramMax commented 3 months ago

This GitHub issue is to establish consensus on publishing a Candidate Recommendation. W3C Working Group participants should reply with affirmation, dissent, or may abstain.

Should we publish a Candidate Recommendation?

Process Consensus is a core value to the W3C. I have assigned this issue to all PNG WG participants who have their GitHub profile listed. Said Abou-Hallawa and Fares Alhassen do not have a GitHub account listed. I will try to email them.

A WG participant may register a Formal Objection if they would like Director consideration.

WG participants may respond with a simple yes/no or similar. They may also elaborate. If dissenting, elaboration is encouraged in order to identify issues and find solutions. The W3C process on managing dissent states that we will try to address concerns as much as is reasonable and should move on once that is done.

Call to action Working group participants should reply with their stance on publishing a Candidate Recommendation.

annevk commented 3 months ago

My main concern is that we still have very few tests, which might mean certain aspects have to be revisited as it becomes more clear what is actually implemented.

ProgramMax commented 3 months ago

@annevk We have tests for each green/red box listed in the Implementation Report. It covers each change that is testable (some cannot be).

Are there some we missed?

svgeesus commented 3 months ago

My main concern is that we still have very few tests, which might mean certain aspects have to be revisited as it becomes more clear what is actually implemented.

True, but remember this is the Third Edition. At this point, except for ambiguities pointed out in errata reports, we are not testing things that are unchanged from the First or Second editions.

I certainly agree that those should be tested too, but adding those tests is not the current priority.

svgeesus commented 3 months ago

Yes, lets publish an updated CR.

ProgramMax commented 3 months ago

I also vote yes.

digitaltvguy commented 3 months ago

I vote yes.

chrisn commented 3 months ago

+1 to publishing an updated CR

leo-barnes commented 2 months ago

I abstain. @hober and @annevk has been following the changes much more closely than me and can vote for Apple.

shallawa commented 2 months ago

I abstain too. @hober and @annevk has been following the changes much more closely than me and can vote for Apple.

svgeesus commented 2 months ago

@palemieux @lrosenthol @ccameron-chromium

svgeesus commented 2 months ago

as it becomes more clear what is actually implemented.

@annevk Speaking of, there are six APNG tests which fail in WebKit but pass in Blink and in Gecko who should we be talking to to have those bugs reported and fixed?

ProgramMax commented 2 months ago

@annevk Any chance you'll be able to join the meeting tomorrow to discuss what test coverage we should improve? Or if you cannot join tomorrow, can you give more info here?

simontWork commented 2 months ago

+1 to publishing an updated CR

palemieux commented 2 months ago

+1 to publishing the CR

ProgramMax commented 2 months ago

The votes thus far are: 6 yes 2 abstain 1 concern, which we're trying to address

5 people have not yet voted. @hober @podborski (who I forgot to assign to originally, sorry), @lrosenthol Fares (who I'll ping in email again), @ccameron-chromium

I'll give this a bit more time.

podborski commented 2 months ago

I also abstain. @hober and @annevk has been following the changes much more closely than me and can vote for Apple.

annevk commented 2 months ago

Apologies, my concern was not meant to be blocking.

+1 to publishing the CR (also on behalf of @hober).

ProgramMax commented 2 months ago

Ah. Good to hear. :D That changes the equation a bit. That makes it 8 yes, 3 abstain, and 3 not yet voted. With that, I'll consider consensus reached. We'll publish a new Candidate Recommendation.

ProgramMax commented 2 months ago

Fares replied to me via email after this. His stance is abstain.

lrosenthol commented 2 months ago

+1 to publishing

ccameron-chromium commented 2 months ago

ccameron abstains

svgeesus commented 2 months ago

CR Snapshot update request