On Fri, Jun 18, 2010 at 8:35 PM, Ralf Gommers <span dir="ltr"><<a href="mailto:ralf.gommers@googlemail.com">ralf.gommers@googlemail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br><div class="gmail_quote"><div class="im">On Sat, Jun 19, 2010 at 5:49 AM, David Goldsmith <span dir="ltr"><<a href="mailto:d.l.goldsmith@gmail.com" target="_blank">d.l.goldsmith@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div>On Fri, Jun 18, 2010 at 2:38 PM, Pauli Virtanen <span dir="ltr"><<a href="mailto:pav@iki.fi" target="_blank">pav@iki.fi</a>></span> wrote:<br></div><div class="gmail_quote"><div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">


Fri, 18 Jun 2010 13:44:11 -0700, David Goldsmith wrote:<br>
[clip]<br>
<div>> I don't understand: if they're going to commit the changes, why do they<br>
> need to be able to mark that they're going to commit the changes?  To<br>
> help them remember which ones they've screened as possessing nothing<br>
> "absurd" in case they can't commit the changes immediately after they've<br>
> decided to commit the changes?<br>
<br>
</div>The point is that you typically commit a huge batch of docstring changes<br>
at once, and reading through a long patch listing makes your eyes glaze<br>
over really fast.<br></blockquote></div><div><br>OK, I understand that, and it makes sense during the regular course of the year when docstring changes aren't happening as frequently, but, something to consider, perhaps through the course of the Summer Marathon, "OK to apply"s should be merged once per week or some such? <br>

</div></div></blockquote></div><div><br>It always makes sense. "OK to apply" should only be used by (a) someone who is about to commit to svn or (b) someone who is 200% sure what this means.<br><br>A few weeks ago I was committing all scipy changes and found many "OK to apply" ones that couldn't actually be committed, for various reasons. This meant I had to go back and recheck everything, including ones I had checked as OK myself before.<br>

<br></div><div class="im"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="gmail_quote"><div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

So a rough sanity check is much easier to do in the web system, and the<br>
burden can be distributed across multiple people if necessary. Currently<br>
ok-to-apply is married with the Reviewer permissions.<br>
<br>
At least this is what I used and intended the feature for. I'm not sure<br>
if anyone else actually understands it the same way, especially as this<br>
is not written down anywhere :)<br>
<div><br>
> > Typically the way to just indicate that stuff is "done", is to mark is<br>
> > as "Needs review", at the moment.<br>
><br>
> I guess then I'm really unclear as to the need for the "OK to apply"; my<br>
> understanding was that it was there for the editor to signal to the<br>
> commitor that, even thought the docstring is technically *not* ready for<br>
> review (e.g., it's still missing an Example, say, or a needed<br>
> Reference), it still represents a big enough improvement over what's in<br>
> SVN that, in the editor's opinion, it is "OK to apply".  If "Needs<br>
> review" is necessary and sufficient for something to be applied, then<br>
> why do we need the extra "OK to apply"?<br>
<br>
</div>It was intended mostly as a reviewer/committer-level tool, at least<br>
originally, which is why it's not active with Editor permissions. Since<br>
anyone can in principle come and edit the wiki, I thought something like<br>
this would come useful.<br>
<br>
I haven't been following the edits lately, so I guess its your call as<br>
the present active guy to decide who gets which privileges :)<br></blockquote></div><div><br>Not necessarily (certainly not in that I don't have the permissions to grant such permissions; I don't even have permissions to commit changes): it depends on the purpose of the attribute - if it's closer to what you say, then I agree, it should be a reviewer/committor (though I didn't think those were one and the same) who controls this; if, on the other hand, the purpose is closer to what I say, then, at minimum, we need to think more about how we do this.<br>

</div></div></blockquote></div><div><br>It's not only about content, but also for example knowing for which docstrings pydocweb can not generate correct patches at the moment.</div></div></blockquote><div><br>Ah, in that case, please rescind my "OK to apply" permissions.  Thanks!<br>
<br>DG<br>
</div></div><br>