A small improvement to plot directive

Development matplotlib-devel
1

Hi all,

would anyone mind if I commit the attached patch?

It's 100% backwards compatible and allows for turning plot directive
errors into fatal exceptions easily, so one can make sure that docs
either build correctly or not at all. This is useful for having
examples be a kind of test suite, in addition to any unit tests a
codebase may have.

Thanks,

f

plot_directive.diff (1.57 KB)

2

Fernando Perez wrote:

Hi all,

would anyone mind if I commit the attached patch?

It's 100% backwards compatible and allows for turning plot directive
errors into fatal exceptions easily, so one can make sure that docs
either build correctly or not at all. This is useful for having
examples be a kind of test suite, in addition to any unit tests a
codebase may have.

Thanks,

f
  
I'm +1 on this. We can have then have the buildbot doc builder enable
this when building the docs. (Which are output at
http://matplotlib.sourceforge.net/trunk-docs/ and
http://matplotlib.sourceforge.net/trunk-docs/Matplotlib.pdf , for
reference .)

-Andrew

3

Thanks, with your and John's (off-list) approvals, it's committed.
The patch that went in had more docs than what I posted here, but the
code is identical.

Cheers,

f

···

On Wed, Jan 27, 2010 at 6:06 PM, Andrew Straw <strawman@...36...> wrote:

I'm +1 on this. We can have then have the buildbot doc builder enable
this when building the docs. (Which are output at
http://matplotlib.sourceforge.net/trunk-docs/ and
http://matplotlib.sourceforge.net/trunk-docs/Matplotlib.pdf , for
reference .)

4

Oops, somehow my commit had never made it, I just noticed now I was
trying to use it from another machine.

Fixed now, r8110.

Cheers,

f

···

On Thu, Jan 28, 2010 at 2:51 PM, Fernando Perez <fperez.net@...149...> wrote:

Thanks, with your and John's (off-list) approvals, it's committed.
The patch that went in had more docs than what I posted here, but the
code is identical.