Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Performance Timeline #644

Closed
caribouW3 opened this issue Jun 7, 2021 · 6 comments
Closed

Performance Timeline #644

caribouW3 opened this issue Jun 7, 2021 · 6 comments

Comments

@caribouW3
Copy link

Ya ya yawm TAG!

I'm requesting a TAG review of Performance Timeline.

This is actually a request to confirm or amend previous review, as requested in the CR transition
w3c/transitions#339 (comment)

This specification was originally developed in levels, Level 1 in REC in 2013,
Level 2 has had a wide review in 2016 before entering CR
https://www.w3.org/TR/2016/CR-performance-timeline-2-20161208/
There is no fundamental change to the goals of the spec. The processing model is more detailed and
data structure updated.
We plan to drop levels and use a living standard development model (from process2020).

We'd prefer the TAG provide feedback as (please delete all but the desired option):

🐛 open issues in our GitHub repo for each point of feedback

💬 leave review feedback as a comment in this issue and @-notify [github usernames]

Thank you!

@torgo torgo added this to the 2021-06-28-week milestone Jun 23, 2021
@torgo torgo added Progress: in progress Progress: pending external feedback The TAG is waiting on response to comments/questions asked by the TAG during the review labels Jul 27, 2021
@cynthia
Copy link
Member

cynthia commented Jul 27, 2021

Thanks for bringing this to our attention (and sorry for the delay!) - we were curious if we could get a delta of the parts that we are supposed to review, so we don't nitpick at stuff that has already been shipped as CR. Thanks!

@kenchris
Copy link

kenchris commented Aug 10, 2021

As this is a Level 2, pointing out what changes were actually made would really help us. We might have comments to the existing parts of Level 1 but as those are shipping, looking at those parts is not the best use of our time.

@torgo
Copy link
Member

torgo commented Aug 10, 2021

Hi @caribouW3 sorry but I think we really need to see an explainer here. The spec may be understood to the group working on it. However one of the points of "wide review" is that people who are unfamiliar with the spec review it (or at least review the delta) with fresh eyes. Can you please provide a little more info for us here? Otherwise we are going to close this as not satisfactory.

@npm1
Copy link

npm1 commented Aug 27, 2021

I'm working on an explainer and will probably have it ready by next week.

@npm1
Copy link

npm1 commented Aug 30, 2021

Explainer: https://github.com/w3c/performance-timeline/blob/gh-pages/explainer.md
Let me know if you have any questions or want me to expand on some part of it. Thanks.

@kenchris
Copy link

@cynthia and I looked though the changes and we don't have any concerns.

Thank you for the explainer!

@kenchris kenchris added Progress: propose closing we think it should be closed but are waiting on some feedback or consensus and removed Progress: pending external feedback The TAG is waiting on response to comments/questions asked by the TAG during the review Progress: in progress labels Sep 14, 2021
@cynthia cynthia closed this as completed Sep 14, 2021
@torgo torgo added Resolution: satisfied The TAG is satisfied with this design and removed Progress: propose closing we think it should be closed but are waiting on some feedback or consensus labels Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants