Early check for dot binary (from graphviz) when building the doc (fixes #8207)#8221
Conversation
|
It also look good to me, though the amount of dependencies we have in our documentation worries me. It means that new contributors struggle with something as simple as contributing documentation. |
|
@NelleV I wonder whether we could create two mpl-testing and mpl-docs packages on conda-forge which would pull in all required dependencies, including the non-python ones (for example, graphviz is packaged by conda). Then the instructions would just be to install the corresponding package from conda. |
|
@anntzer I don't use conda, so I have no opinion on whether this would be useful. An option would be to change the default for allowsphinxwarnings so that people can build the documentation with missing dependencies. That would reduce the barrier for contributions, but then it would be up to us (core contributors) to guide new contributors on fixing travis build. I do not think we are well equipped for that option. |
|
Alternatively, I know that imageio has been including code to download ffmpeg/avbin/freeimage at runtime (http://imageio.readthedocs.io/en/latest/envvariables.html), so that could be an option too (although it would certainly have its own issues, unless they have completely statically linked builds...). |
|
Thanks @afvincent ! |
Early check for dot binary (from graphviz) when building the doc (fixes #8207)
|
Backported to |
Fixes #8207.
The commit is shamelessly inspired from #8213.