On Sun, Sep 2, 2012 at 5:05 PM, Stéfan van der Walt stefan@sun.ac.zawrote:

On Sep 2, 2012 7:15 PM, "Tony Yu" tsyu80@gmail.com wrote:

...

While looking through the sphinx config file, I noticed that we have

config settings for the `plot_directive` extension and also include the sphinx extension in `doc/ext/`. A quick grep of the doc directory suggests that we don't actually use the plot directive anywhere. Is it worth keeping for future use or should I remove that as part of this doc clean up?

There's probably some legacy cruft left over--few people ever read that file, so feel free to clean it up.

Is the plot directive non needed to build figures for the user guide?

Thanks! Stéfan

...

-Tony

It's not needed for the user guide, but I later noticed that it's used in a couple of the docstrings (e.g. API docs of hough transform). I'll just leave the plot directive as is, since it is actually being used.

I did manage to fix a number of Sphinx warnings. See PR 295https://github.com/scikits-image/scikits-image/pull/295/files .

-Tony