#7981: animate ignores options to show that are passed up from the plot command
-----------------------------------------------+----------------------------
   Reporter:  jason                            |       Owner:  was         
       Type:  defect                           |      Status:  needs_review
   Priority:  major                            |   Milestone:  sage-4.6.1  
  Component:  graphics                         |    Keywords:              
     Author:  Jason Grout, Andrey Novoseltsev  |    Upstream:  N/A         
   Reviewer:  Tim Dumol                        |      Merged:              
Work_issues:                                   |  
-----------------------------------------------+----------------------------
Changes (by newvalueoldvalue):

  * status:  needs_work => needs_review
  * author:  Jason Grout => Jason Grout, Andrey Novoseltsev


Comment:

 I have just wrote a patch fixing the issue for `save` (which is quite
 annoying in conjunction with SageTeX) and then found this ticket. Since
 the original patch does not apply cleanly on sage-4.6.alpha1, I posted my
 patch, but I have incorporated Jason's changes to `show` to eliminate
 double processing of options.

 I have changed call parameters to `save` since it is not documented why
 one would ever need `savenow=False`. If I want to save later, shouldn't I
 call `save` later?-) Also it does not make sense in my opinion to use any
 extra positional arguments in this function except for the file name. I
 realize that this is a backward-incompatible change, but all long tests
 pass with this patch.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7981#comment:9>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
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.

Reply via email to