Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-11 Thread Robert Bradshaw
On Mar 11, 2010, at 4:41 AM, Florent Hivert wrote: Hi While you are at it, To add a small epsilon to Minh's comments: it is fairly common for a reviewer to add a small reviewer patch fixing docstrings or adding some examples, etc. This could be a good alternative to getting frustrat

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-11 Thread Florent Hivert
Hi While you are at it, >> To add a small epsilon to Minh's comments: it is fairly common for a >> reviewer to add a small reviewer patch fixing docstrings or adding some >> examples, etc. This could be a good alternative to getting frustrated >> with the author for not making these change

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-11 Thread Dr. David Kirkby
Alex Ghitza wrote: On Thu, 11 Mar 2010 18:42:00 +1100, Minh Nguyen wrote: Third, in many cases, one can open a new ticket to improve the changes introduced by a patch from an existing ticket. In such cases, I think one can suggest this option to the patch author and leave it to them to either i

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-11 Thread Dr. David Kirkby
Minh Nguyen wrote: Hi Dima, 2) eventually, in order to prevent these things getting personal, it might be good to have a possibility to anonymise reviewing. Most of the time, reviewers are also people who contribute a lot to improving a patch. We don't want make such contributors anonymous,

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-11 Thread Dr. David Kirkby
Robert Bradshaw wrote: On Mar 11, 2010, at 12:04 AM, Alex Ghitza wrote: On Thu, 11 Mar 2010 18:42:00 +1100, Minh Nguyen wrote: Third, in many cases, one can open a new ticket to improve the changes introduced by a patch from an existing ticket. In such cases, I think one can suggest this opti

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-11 Thread Robert Bradshaw
On Mar 11, 2010, at 12:04 AM, Alex Ghitza wrote: On Thu, 11 Mar 2010 18:42:00 +1100, Minh Nguyen wrote: Third, in many cases, one can open a new ticket to improve the changes introduced by a patch from an existing ticket. In such cases, I think one can suggest this option to the patch autho

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-11 Thread Alex Ghitza
On Thu, 11 Mar 2010 18:42:00 +1100, Minh Nguyen wrote: > Third, in many cases, one can open a new ticket to improve the changes > introduced by a patch from an existing ticket. In such cases, I think > one can suggest this option to the patch author and leave it to them > to either incorporate the

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-10 Thread Minh Nguyen
Hi Dima, On Thu, Mar 11, 2010 at 5:10 PM, Dima Pasechnik wrote: > 1) it would we good to have a "moderator" who can step in in such > cases. Communication within any open source project is bound to be archived somewhere on the Internet. Contributors need to realize that any communication is d

Re: [sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-10 Thread William Stein
On Wed, Mar 10, 2010 at 10:10 PM, Dima Pasechnik wrote: > Dear all, > > I recently had a rather unpleasant experience reviewing a ticket that > shall remain unnamed. It went as follows. I suggested few Given that sentence it is trivial to figure out what ticket you're talking about: http://t

[sage-devel] anonymous reviewing of trac tickets? trac moderators?

2010-03-10 Thread Dima Pasechnik
Dear all, I recently had a rather unpleasant experience reviewing a ticket that shall remain unnamed. It went as follows. I suggested few improvements, asked few questions. Some suggestions were implemented, some plainly ignored, along with questions. I suggested few more improvements, asked (and