#767: [with patch, positive review] animate has cryptic error message when
imagemagick is not installed
----------------------------------+-----------------------------------------
Reporter: moretti | Owner: boothby
Type: defect | Status: new
Priority: trivial | Milestone: sage-3.1.3
Component: notebook | Resolution:
Keywords: animate, ImageMagick |
----------------------------------+-----------------------------------------
Changes (by mabshoff):
* summary: [with patch, needs review] animate has cryptic error message
when imagemagick is not installed => [with
patch, positive review] animate has cryptic
error message when imagemagick is not installed
Comment:
Positive review on the patch. One small issue is that the animate.py
doctest fails if convert is not installed:
{{{
[EMAIL PROTECTED]:/scratch/mabshoff/release-cycle/sage-3.1.3.alpha2$ ./sage -t
devel//sage/sage/plot/animate.py
sage -t devel/sage/sage/plot/animate.py
**********************************************************************
File "/scratch/mabshoff/release-cycle/sage-3.1.3.alpha2/tmp/animate.py",
line 59:
sage: a.show()
Expected nothing
Got:
<BLANKLINE>
Error: ImageMagick does not appear to be installed. Saving an
animation to a GIF file or displaying an animation requires
ImageMagick, so please install it and try again.
<BLANKLINE>
See www.imagemagick.org, for example.
**********************************************************************
}}}
One way around this would be to make the doctest optional.
Thoughts?
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/767#comment:5>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
