BIDS machine shop & matplotlib

Hello matplotlib-devs !

As some of you may know, I am involved with St?fan van der Walt in an
initiative around Berkeley campus known as the "BIDS Machine Shop". It
consists of trying to bridge the gap in software development and best
practises around campus by introducing undergraduates to research
software development and open source communities.
St?fan and I will manage students on different research projects and
open source packages, including matplotlib and scikit-image. I'll be
managing the matplotlib side of the machine shop, and you should see
(I hope) a bunch of contributions from our students popping up on
github.

I am going to create a label "Machine Shop" and a project "Machine
Shop" for the duration of the semester (maybe longer if this
experiment is a success). I'll assign "easy fix" tickets to our
students. I expect them to do peer-review before opening the PRs for
broader reviews. Please, wait for the tag "need review" to be set on
those PR before reviewing theme. Once they fixed these issues, I'll
assign them to more complex tasks.

If you have questions/suggestions, don't hesitate to chime in.

Thanks,
Nelle

PS: we should have a website up and running soon with more details.

Nelle,

I think the "needs review" tag is automatically added. I might be wrong
though.
-p

···

On Fri, Sep 30, 2016 at 9:32 AM, Nelle Varoquaux <nelle.varoquaux at gmail.com> wrote:

Hello matplotlib-devs !

As some of you may know, I am involved with St?fan van der Walt in an
initiative around Berkeley campus known as the "BIDS Machine Shop". It
consists of trying to bridge the gap in software development and best
practises around campus by introducing undergraduates to research
software development and open source communities.
St?fan and I will manage students on different research projects and
open source packages, including matplotlib and scikit-image. I'll be
managing the matplotlib side of the machine shop, and you should see
(I hope) a bunch of contributions from our students popping up on
github.

I am going to create a label "Machine Shop" and a project "Machine
Shop" for the duration of the semester (maybe longer if this
experiment is a success). I'll assign "easy fix" tickets to our
students. I expect them to do peer-review before opening the PRs for
broader reviews. Please, wait for the tag "need review" to be set on
those PR before reviewing theme. Once they fixed these issues, I'll
assign them to more complex tasks.

If you have questions/suggestions, don't hesitate to chime in.

Thanks,
Nelle

PS: we should have a website up and running soon with more details.
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel at python.org
Matplotlib-devel Info Page

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/matplotlib-devel/attachments/20160930/a7f35de9/attachment.html&gt;

Nelle,

I think the "needs review" tag is automatically added. I might be wrong
though.

Yes. I plan on removing it by hand everytime one of my students open a
PR not ready to be merged. It is going to be painful, but I don't see
any other way?

I actually think it is slightly problematic to have it added
automatically. A bunch of PRs are not ready to be merged or review,
and not many people remove the tag after reviewing if changes need to
be applied. That means most of our PRs have this tag, so it is hard to
identify the one that actually need to be reviewed.

Chers,
N

···

On 30 September 2016 at 11:15, Paul Hobson <pmhobson at gmail.com> wrote:

-p

On Fri, Sep 30, 2016 at 9:32 AM, Nelle Varoquaux <nelle.varoquaux at gmail.com> > wrote:

Hello matplotlib-devs !

As some of you may know, I am involved with St?fan van der Walt in an
initiative around Berkeley campus known as the "BIDS Machine Shop". It
consists of trying to bridge the gap in software development and best
practises around campus by introducing undergraduates to research
software development and open source communities.
St?fan and I will manage students on different research projects and
open source packages, including matplotlib and scikit-image. I'll be
managing the matplotlib side of the machine shop, and you should see
(I hope) a bunch of contributions from our students popping up on
github.

I am going to create a label "Machine Shop" and a project "Machine
Shop" for the duration of the semester (maybe longer if this
experiment is a success). I'll assign "easy fix" tickets to our
students. I expect them to do peer-review before opening the PRs for
broader reviews. Please, wait for the tag "need review" to be set on
those PR before reviewing theme. Once they fixed these issues, I'll
assign them to more complex tasks.

If you have questions/suggestions, don't hesitate to chime in.

Thanks,
Nelle

PS: we should have a website up and running soon with more details.
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel at python.org
Matplotlib-devel Info Page

I agree. Automatic adding of this tag is pointless, not helpful, as far
as I can see.

Eric

···

On 2016/09/30 8:20 AM, Nelle Varoquaux wrote:

I actually think it is slightly problematic to have it added
automatically. A bunch of PRs are not ready to be merged or review,
and not many people remove the tag after reviewing if changes need to
be applied. That means most of our PRs have this tag, so it is hard to
identify the one that actually need to be reviewed.