> I'm not sure if non-core-developers are allowed to post MEPs, but I just
did

Yes please... The more the merrier!
Aren't you a core dev anyway Tony? :-) You've certainly contributed some
really valuable features, and even if you don't have access to "the big
green button" in my eyes you feedback and input are just as valuable.

The gallery was one of the key features that pulled me into mpl, but it
definitely needs a bit of TLC. The timing is quite convenient for me as I
am about to (in the next month or so) embark on implementing a gallery for
a project I'm working on that makes heavy use of matplotlib (cartopy:
http://scitools.org.uk/cartopy/docs/latest/).

One thing I was considering doing in my implementation was allowing
multiple tags for each example by adding some extra module level
information (in a list called "__tags__" perhaps) rather than focussing on
a directory structure to provide the tag as the current implementation of
the gallery does (and as far as I can see, the MEP recommends this too).

In the past I have also worked on a gallery extension which uses the
docstring of the example to provide a richer explanation of what is going
on (example:
http://scitools.org.uk/iris/docs/latest/examples/graphics/COP_1d_plot.html).
An alternative approach which has worked well for me is the walked through
examples found in scikits-learn (their website is down, but most of the
examples are good in that regard) which could be done via an iPython
notebook for the annotations.


> Do you think that all the work should be done on a parallel branch and
merged in one big PR?

Personally, I'm not in favour of that. We would have to make sure that the
pull requests related to this change don't start to stack up, but I think
we can definitely do this sequentially once the appropriate machinery is in
place.

Thanks for bringing this subject up,

All the best,

Phil




On 10 December 2012 23:00, Tony Yu <tsy...@gmail.com> wrote:

>
>
> On Mon, Dec 10, 2012 at 5:51 PM, Paul Hobson <pmhob...@gmail.com> wrote:
>
>> On Mon, Dec 10, 2012 at 2:18 PM, Tony Yu <tsy...@gmail.com> wrote:
>>
>>> Hi all,
>>>
>>> I'm not sure if non-core-developers are allowed to post MEPs, but I just
>>> did ;). MEP 12 outlines the reorganization of the example gallery and
>>> subsequent clean up of the examples:
>>>
>>>     https://github.com/matplotlib/matplotlib/wiki/MEP12
>>>
>>> In my opinion, there are two open questions in the MEP:
>>>
>>>     * Section names (may seem trivial to some, but I think it's really
>>> important)
>>>     * Guidelines for cleaning up examples
>>>
>>> I've added proposed section names and clean-up guidelines to the MEP,
>>> but I'm sure these will evolve over time.
>>>
>>> Best,
>>> -Tony
>>>
>>>
>> Here here! I've been waiting for a really nasty, cold, and rainy weekend
>> here in Portland to make some PRs at least cleaning up the code contained
>> within the examples. Just hasn't happened yet ;)
>>
>>  Tony, if you make your own branch for this, I'd be happy to contribute.
>> -paul
>>
>
> Awesome! I'm hoping we can quickly converge on some clean-up guidelines
> and section headings.
>
> I was envisioning a single PR that adds new gallery sections. After that
> examples would be updated over a long period of time in multiple PRs that
> change a few examples at a time. Do you think that all the work should be
> done on a parallel branch and merged in one big PR?
>
> Best,
> -Tony
>
>
> ------------------------------------------------------------------------------
> LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
> Remotely access PCs and mobile devices and provide instant support
> Improve your efficiency, and focus on delivering more value-add services
> Discover what IT Professionals Know. Rescue delivers
> http://p.sf.net/sfu/logmein_12329d2d
> _______________________________________________
> Matplotlib-devel mailing list
> Matplotlib-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>
>
------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Reply via email to