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
Clarify what the expectations are for advancing to CR #214
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proposed a couple of modifications for review.
index.html
Outdated
<dd><strong>Note</strong>: Candidate Recommendations are expected to be acceptable as Recommendations. Announcement of a | ||
<dd><strong>Note</strong>: Advancing to Candidate Recommendation indicates either that | ||
the technical report is expected to advance to Recommendation without substantive changes, | ||
or that no further improvement is deemed possible without additional implementation experience or feedback from the wider community. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Slightly troubled by the word "possible" here - it may be that the WG wants to draw the line at a suitable place and go ahead, with wide review feedback deferred to a later iteration of the technical report. Maybe replace this line with:
and that the Working Group has agreed that no further changes are required in this putative version of the Recommendation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm. I'd like to leave the door open to "let's leave this undefined in this level", but not to "this is known to be wrong, but whatever, let's ship anyway".
I think my phrasing may be a bit too strict, and yours a bit too loose.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with allowing "let's leave this undefined" being a reasonable deliberate choice that should not in itself block transition (but could be a blocker if there is unresolved feedback that raises it as an issue, for example).
Happy to review other wording proposals to try to hit the sweet spot.
After the last call of the process cg, my understanding of the status of this pull request is that:
|
I'd rather not lose the sense of "Candidate Recommendations are expected to be acceptable as Recommendations", and I wonder if we should have "The text of a Candidate Recommendation is expected to be of the quality of, and acceptable as, the text of a Recommendation itself." or the like? |
I understand what you're trying to achieve with that phrasing, but I am not sure that it works.
Assuming you don't want to just drop this, how about that:
|
@dwsinger the test for this is that it passes the Director's, and the AC's review, right, because those are the reviews that happen to get from CR to PR and then Rec? The Director reviews the transition to CR, but the AC isn't likely to comment until PR. We either need to close that loop so that your goal can be verified before transitioning to CR, or open up the very real possibility of getting feedback along the lines "this should never have been a CR" after the spec is in CR and has already met its CR exit criteria. The process needs to allow for the right feedback to happen at the right time, or accept that such feedback cannot be meaningfully processed. What would you do here? |
To get accepted as a CR, the spec needs not only to pass the Director's personal review, but also to demonstrate wide review. That seems like a perfectly appropriate time for anyone to point out if a spec is such a mess that it shouldn't be allowed to go forward. |
@frivoal right, so in that case does adding this statement of expectation help anyone? It seems like it's more an instruction to reviewers on the level of quality they should be aiming for, rather than a part of the process per se. |
I think we need, at least, to set an expectation, that the team and others can point at: if, magically, implementation experience and interop and all the things that happen post-CR go amazingly swimmingly, you should basically have the PR/Rec. text in hand. The spec needs to be of specification-quality, basically, or as close as it can get without the CR-period experience. And we need to say that that is the expectation. |
In a sense, yes. reviewers should be able to say "nope, your document is not spec.-ready and it's required, and you knew it was required because it's written in the process". I think if it's unstated we'll get cases trying to slip by, and slip by they will. |
@dwsinger your comments seem to be in support of the phrasing I proposed in #214 (comment), but you don't say so explicitly. Does that work for you, or should we try some more? |
@dwsinger , @nigelmegitt I've updated the PR with a blend of the phrases we have been discussing. I hope the latest version covers your expectations. To make it easier to read than looking at a diff, here are the two key paragraphs (do look at the diff if you want to see them in context)
|
This is fine. Thanks for persisting |
@dwsinger Should I just merge? Do you want to issue a CfC? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. |
Wrt
I really liked @dwsinger’s phrasing in this comment:
So maybe something like “A Candidate Recommendation is expected to be as well-written, detailed, self-consistent, and technically complete as a Recommendation, and acceptable as such ...”? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving as is, but with a proposed editorial tweak - I would approve either with or without the tweak.
@@ -1961,6 +1964,13 @@ <h4 id="maturity-levels">6.1.2 Maturity Levels</h4> | |||
abandoned without producing a Recommendation.</dd> | |||
</dl> | |||
|
|||
<p>The decision to advance to a new maturity level must be made solely based on the technical maturity of the work. | |||
<strong>Note:</strong> Should faster advancement to meet scheduling considerations be desired, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was my wording originally, but on re-reading, and thinking about the change of may
to can
, I realise that the readability could be improved to remove the slightly archaic (though strictly correct) construction and the accidental use of a conformance keyword, from:
Should faster advancement to meet scheduling considerations be desired
to
If faster advancement to meet scheduling considerations is desired,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about "to advance faster" ? If nobody wants to do that, it doesn't matter, and if they do we don't need to complicate the text.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm relatively indifferent between the existing phrasing and the alternative one proposed by @nigelmegitt . As for the one suggested by @chaals, would not object to it, but I like it less. Sure, it's shorted, but it hides the fact that advancing faster is a matter of choice.
@dwsinger commented a while back
This suggests a pandora's box of subjectiveness that I'm loathe to open in the text of the Process. By the time a WG feels their spec is sufficiently mature to become a CR all reviewers with concerns should be expected to propose -- or at least outline -- changes that would resolve their concern. "Not spec-ready" is only a useful comment when accompanied by sufficient text for the WG to understand what changes are being asked of it. 6.2.2 and 6.4 hold the normative prose for reviewers to make such comments and have them considered during transition deliberations. 6.1.2 is the wrong place to be providing advice to Working Groups, their editors, and reviewers. I don't support the proposed additions to 6.1.2; I only support dropping the Note as an historic artifact as it apparently now causes confusion. Additional informative advice such as that proposed could usefully be incorporated into Working Group Effectiveness. |
Removing the note altogether was one of the proposals I suggested in #172, so it would satisfy the issue as far as I'm concerned. I actually think the modification as proposed in this pull request is better than that, but complete removal would also be acceptable to me. |
Agreed. A claim that something is not spec.-ready would need to be supported by evidence (for example, the existence of a TBD in the document, or editorial notes indicating uncertainty, and so on).
If you want to move the requirements and advice to 6.4, which is also about Candidate Recommendation, I am fine with that.
So, you think that we should return to the old days where groups issued Last Calls (now CRs) on documents that could not be considered as publishable? I don't, and I don't understand why you would. |
I want any new proposed normative requirements to be in 6.4 and I recommend that advice be in a separate document. Statements such as "The decision to advance to a new maturity level must be made solely based on the technical maturity of the work" are claimed in this thread to not be constraining but have the same type of Process consistency issue that opened #172.
No, certainly not. |
@swickr OK, then retain:
Move to guide:
Omit or move to guide:
The retentions could be in 6.4, which is about CR. Is this your proposal? If not, please be specific. (Honestly, we're supposed to be in yes/no decision now; the time for this level of tinkering is supposed to be past.) |
is over-constraining as @wseltzer previously objected. Section 6.1.2 is the definition of maturity levels, not advice on how to reach them. I propose a different approach to remain definitional while respecting the spirit of the pull request: To the end of the definition of Working Draft add
In the definition of Candidate Recommendation replace
with
and move the remainder to a guide. I would be OK with "well-written, detailed, self-consistent, and technically complete" in place of "of sufficient quality and maturity" |
You seem to understand something from Wendy here that no-one else does, so can you explain? You (plurally) want to be able to advance technically immature work? |
No, the decision to -- or not to -- advance may be made on other considerations. For example, the group may decide not to advance work that is technically mature, in order to synchronize it with other work that is not yet mature. |
"Only sufficiently technically mature work should be advanced," is different in meaning from "The decision to advance to a new maturity level must be made solely based on the technical maturity of the work." I'd find the first acceptable, not the second. |
The original statement says nothing at all about why you make a decision not to advance, only about the criteria to advance. Nonetheless, if you can re-phrase to something which has the same intended meaning, I expect to get consensus. |
@dwsinger clearly our interpretations of the phrase and what's said by implication differ. I suggest "Only sufficiently technically mature work should be advanced." Otherwise, I suggest we leave it out entirely. |
try again. retain: "A Candidate Recommendation is expected to be as well-written, detailed, self-consistent, and technically complete as a Recommendation, and acceptable as such if and when the requirements for further advancement are met." replace Move to guide: "Note: Should faster advancement to meet scheduling considerations be desired, this can be achieved by reducing the scope of the technical report to a subset that is adequately mature and deferring less stable features to other technical reports." is advice that states the somewhat-obvious and could be in the guide. Omit or move to guide, or leave as note with the retentions: "Note: Advancing to Candidate Recommendation indicates that no further improvement is expected without additional implementation experience and testing." The retentions could be in 6.4, which is about CR, or left in the section that the original phrase is in. |
@dwsinger 's proposal in #214 (comment) is acceptable to me. I'd prefer keeping the notes in instead of moving then the the guide or dropping, as I think such notes improve the understandability of a document more than terseness does, but if there is an actual objection to having them, and no objection if they're removed, I can live with that. |
+1 @dwsinger's #214 (comment) |
I have updated the pull request with this change, since it seems that it is at the core of Wendy's objection and essential to Ralph's, and is non controversial for those who already supported this PR, as the rephrasing continues to match what we were attempting to say. As for moving the text to another section and dropping (in favor of the guide) one or both of the notes, I'm awaiting further instructions from the chair. It seems to me that these changes are not essential to overcome the objection from Wendy and Ralph, and that other people were fine with the text as it is, so I'd rather not change it at this stage, but I have no objection to doing so if the chair judges that doing so is the best way to get to consensus. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest changes look good to me.
The rephrasing addresses my core objection and with that change I accede to a consensus for including this advisory material in the Process rather than in /Guide. |
Looks good to me, too. |
Comments have been addressed to satisfaction.
The CfC about the 2019 process is closed and the chair has concluded we do have consensus. Merging. |
Closes #172