w3c / PNG-spec

Maintenance of the PNG specification
https://w3c.github.io/PNG-spec/
Other
43 stars 10 forks source link

Call for Consensus: publish Candidate Recommendation #453

Closed ProgramMax closed 1 week ago

ProgramMax commented 2 weeks 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 2 weeks 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 2 weeks 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 2 weeks 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 2 weeks ago

Yes, lets publish an updated CR.

ProgramMax commented 2 weeks ago

I also vote yes.

digitaltvguy commented 2 weeks ago

I vote yes.

chrisn commented 2 weeks ago

+1 to publishing an updated CR

leo-barnes commented 2 weeks ago

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

shallawa commented 2 weeks 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 weeks ago

@palemieux @lrosenthol @ccameron-chromium

svgeesus commented 2 weeks 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 1 week 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 1 week ago

+1 to publishing an updated CR

palemieux commented 1 week ago

+1 to publishing the CR

ProgramMax commented 1 week 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 1 week ago

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

annevk commented 1 week ago

Apologies, my concern was not meant to be blocking.

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

ProgramMax commented 1 week 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 1 week ago

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

lrosenthol commented 1 week ago

+1 to publishing

ccameron-chromium commented 1 week ago

ccameron abstains

svgeesus commented 2 days ago

CR Snapshot update request