matplotlib-devel Mailing List for matplotlib

Moving this to matplotlib-devel
On Thu, Jan 5, 2012 at 1:17 PM, Skipper Seabold <jss...@gm...> wrote:
> Is this information incorrect?
>
> http://matplotlib.sourceforge.net/devel/documenting_mpl.html#module-matplotlib.sphinxext.plot_directive
>
> This line seems to preclude ever getting to the below code to process
> the caption.
>
> https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/sphinxext/plot_directive.py#L601
>
> I think the proper fix is to make :caption: an option.
I implemented :caption: as an option [1], but this is at odds with the
original figure directive [2]. AFAICT either the check for content and
arguments has to be removed and all content after a filename is given
is assumed to be a caption, or caption has to become an option.
Skipper
[1] https://github.com/jseabold/matplotlib/commit/2f003e296a0c37bcbf6687702ddfd34ac12235b4
[2] http://docutils.sourceforge.net/docs/ref/rst/directives.html#figure

On 01/05/2012 02:22 AM, Benjamin Root wrote:
>
> >
> >
>
> I don't have an opinion on this yet, but I do want to note that if 
> this change were applied, then the lasso selector should probably also 
> be changed in order to be consistent.
I believe the lasso selector already does this (determined 
experimentally at least) -- this brings the RectangleSelector widget in 
line with that.
Mike