Add pillow to doc build dependencies #7793

Open
dstansby opened this Issue Jan 10, 2017 · 4 comments

Projects

None yet

3 participants

@dstansby
Contributor

Just spent about 5 mins building the docs, only to get:

Warning, treated as error:
/Users/dstansby/matplotlib/doc/examples/pylab_examples/agg_buffer.rst:8: WARNING: Exception occurred in plotting agg_buffer
 from /Users/dstansby/matplotlib/doc/mpl_examples/pylab_examples/agg_buffer.py:
Traceback (most recent call last):
  File "<string>", line 13, in <module>
ImportError: No module named 'PIL'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/dstansby/matplotlib/lib/matplotlib/sphinxext/plot_directive.py", line 525, in run_code
    six.exec_(code, ns)
  File "<string>", line 15, in <module>
SystemExit: Pillow must be installed to run this example

Building HTML failed.

Would be good to have pillow checked before the build starts like the other dependencies.

@tacaswell
Member

There is https://github.com/matplotlib/matplotlib/blob/master/doc-requirements.txt which has this. I guess the docs have gotten out of date?

Should have the instruction to pip install -r ../doc-requirements.txt someplace.

@dstansby
Contributor

Also looks like the docs need graphviz to build too.

writing output... [ 58%] api/animation_api                                                                                                                                                                      
Warning, treated as error:
WARNING: dot command 'dot' cannot be run (needed for graphviz output), check the graphviz_dot setting

Building HTML failed.
@dstansby
Contributor

Huh, odd, it definitely checked for numpydoc, ipython, and colorspacious when I started trying to build (and force stopped the build at the beginning), but not pillow or graphviz.

@NelleV
Contributor
NelleV commented Jan 14, 2017

My bad… I forgot a couple of dependencies when I added the checks early on in the documentation build.
I'll fix that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment