#7981: animate ignores options to show that are passed up from the plot command
----------------------------------------------------------------+-----------
Reporter: jason | Owner:
novoselt
Type: defect | Status:
needs_work
Priority: major | Milestone:
sage-4.6.1
Component: graphics | Keywords:
Author: Jason Grout, Andrey Novoseltsev | Upstream:
N/A
Reviewer: Tim Dumol, Marshall Hampton, Karl-Dieter Crisman | Merged:
Work_issues: |
----------------------------------------------------------------+-----------
Changes (by kcrisman):
* status: positive_review => needs_work
* reviewer: Tim Dumol, Marshall Hampton => Tim Dumol, Marshall Hampton,
Karl-Dieter Crisman
Comment:
Just out of curiosity, what is the 'backward-incompatible' change you
mention? Which extra positional arguments - like dpi? (Though Jason
also got rid of that - I wonder why?)
I guess I mean to ask whether this is a good such change; usually there is
a deprecation period. After all, doctests catch very few of our use cases
:) What is wrong with the usual `*args,**kwds` syntax?
As for `savenow`, it looks like with it being `False` we could still
create a Sage object. You are right that it seems a little redundant,
though!
Also, this needs a doctest (it's in the original patch) to show that
animate options actually work, at least in theory (if one looked at it and
ran the optional tests). So... needs work. Sorry :(
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7981#comment:13>
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.
