Re: [matplotlib-devel] New plot type idea -- EventRaster

[Todd <tod...@gm...>]

On Mon, Jan 14, 2013 at 1:28 AM, Todd <tod...@gm...> wrote:

> On Mon, Nov 12, 2012 at 6:35 PM, Damon McDougall <
> dam...@gm...> wrote:
>
>> On Mon, Nov 12, 2012 at 2:17 PM, Paul Ivanov <piv...@gm...>
>> wrote:
>> > On Mon, Nov 12, 2012 at 7:36 AM, Michael Droettboom <md...@st...>
>> wrote:
>> >> On 11/11/2012 11:51 PM, Todd wrote:
>> >>> Now that 1.2 is out, can we revisit this?  I would like to get it
>> >>> implemented for the next feature release.
>> >>>
>> >>
>> >> Absolutely.  I think the next step, once you have an implementation,
>> >> would be to submit a pull request and we can all help with a review.
>> >
>> > This hasn't been mentioned yet, but Todd will hopefully find our
>> > developer docs useful:
>> > http://matplotlib.org/devel/index.html
>> >
>> > In particular, there's a section on writing a new pyplot function:
>> >
>> http://matplotlib.org/devel/coding_guide.html#writing-a-new-pyplot-function
>>
>> Thanks for that, Paul.
>>
>> Todd, there's also a section on writing tests for matplotlib on the
>> page Paul pointed out. For a new feature there should be a couple of
>> tests to go with it to make sure everything passes sanity checks.
>>
>> Thanks for spending your time contributing!
>>
>>
>>
> I have completed the plot type, including unit tests and examples (I am
> not an artist so someone else can probably make the examples prettier).
> I've confirmed pep8 compliance and run the code through pyflakes and pylint
> in addition to the unit tests.
>
> It is divided into two parts: an EventCollection class, which is a
> subclass of LineCollection in collections.py, and an eventplot method in
> axes.py (and pyplot, through boilerplate.py) which returns a list of
> EventCollection objects.
>
> I am ready to submit a git pull request now.  However, it depends on
> another git pull request I submitted, titled "add get_segments method to
> collections.LineCollection<https://github.com/matplotlib/matplotlib/pull/1655>".
> I can confirm the changes in this pull request work properly, I have been
> using the new method extensively in my new plot type.
>
> Would it be possible to get this accepted so I can submit the new plot
> type for review?  I am sure there will be lots of changes and cleanups that
> will be required before the new plots can be accepted, while the
> get_segments method is a relatively minor and non-intrusive change.
>
> If you wish to look at the code before the pull request it is in the
> toddrjen/matplotlib github fork, in the eventplot branch.  The main changes
> are to collections.py and axes.py.  The tests are in test_axes.py and a new
> test file test_collections.py. The examples are in eventcollection_demo.py
> and eventplot_demo.py
>
> Thanks a lot for your time.
>

Thanks for getting get_segments in so quickly.

The pull request has been submitted, see "Add EventCollection and
eventplot<https://github.com/matplotlib/matplotlib/pull/1657>
"