Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Issue with convert when installing docs #14756

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.

Sign up for GitHub

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

Jump to bottom
Closed
shanosborne opened this issue Jul 13, 2019 · 2 comments
Closed

Issue with convert when installing docs #14756

shanosborne opened this issue Jul 13, 2019 · 2 comments
Labels
Documentation
Milestone
v3.2.0

Comments

@shanosborne
Copy link

When building the docs with the make html command, if you already have a convert module somewhere else, you can get an error:

File "/Users/sosborne/miniconda3/envs/astroconda/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['convert', '--version']' returned non-zero exit status 1.

This should probably be fixed by checking if the convert module matplotlib is looking for is actually the correct module, and finding a work around if convert is from another package
@alexrudy
Copy link
Contributor

Took a quick look at this – I think that _get_executable_info should probably be more forgiving, or raise a smaller subset of errors.

e.g. this error got raised via animation.ImageMagickBase, which tries to catch a FileNotFoundError, but _get_executable_info can raise other errors quite reasonably.

@alexrudy alexrudy mentioned this issue Jul 14, 2019
Merged
6 tasks
@timhoffm
Copy link
Member

Not fully followed through, but it seems fixed by #14776. Please reopen if there is still something to do.

@QuLogic QuLogic added this to the v3.2.0 milestone Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation
Projects
None yet
Milestone
v3.2.0
Development

No branches or pull requests
5 participants
@QuLogic @alexrudy @timhoffm @dstansby @shanosborne