-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add pillow and graphviz to doc build dependencies #7793
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
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 |
Also looks like the docs need
|
Huh, odd, it definitely checked for |
My bad… I forgot a couple of dependencies when I added the checks early on in the documentation build. |
Pretty sure this is fixed now. |
At least on OSX,
|
I didn't think graphviz was pip-installable. Is the requirement documented somewhere? |
graphvis is not pip installable. |
I can confirm that |
Huh, how isn't For the record, I had run |
https://pypi.python.org/pypi/graphviz/0.5.2
so I guess there's no harm in adding this to |
|
Closed in #8544 if not before. |
Just spent about 5 mins building the docs, only to get:
Would be good to have pillow checked before the build starts like the other dependencies.
The text was updated successfully, but these errors were encountered: