matplotlib-devel

[matplotlib-devel] A small improvement to plot directive

[Fernando P. <fpe...@gm...>]

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

Re: [matplotlib-devel] A small improvement to plot directive

[Andrew S. <str...@as...>]

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

Re: [matplotlib-devel] A small improvement to plot directive

[Fernando P. <fpe...@gm...>]

On Wed, Jan 27, 2010 at 6:06 PM, Andrew Straw <str...@as...> 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 .)

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

Re: [matplotlib-devel] A small improvement to plot directive

[Fernando P. <fpe...@gm...>]

On Thu, Jan 28, 2010 at 2:51 PM, Fernando Perez <fpe...@gm...> 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.

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