#2100: sensible defaults for aspect ratio
-------------------------------------------------------+--------------------
Reporter: jason | Owner: was
Type: enhancement | Status:
needs_review
Priority: major | Milestone:
sage-4.6.2
Component: graphics | Keywords:
Author: Jason Grout, Karl-Dieter Crisman | Upstream: N/A
Reviewer: Andrey Novoseltsev, Karl-Dieter Crisman | Merged:
Work_issues: complete documentation |
-------------------------------------------------------+--------------------
Changes (by newvalueoldvalue):
* status: needs_work => needs_review
* reviewer: Andrey Novoseltsev => Andrey Novoseltsev, Karl-Dieter
Crisman
* author: Jason Grout => Jason Grout, Karl-Dieter Crisman
Comment:
The problem with the `plot_field3d` turns out to be the following very bad
behavior of plotting vectors after this patch
{{{
sage: v = vector([1.,2.,3.])
sage: v.plot() # all is well
sage: plot(v) # passes in aspect_ratio to a 3d graphics, which already has
such a thing, and get nasty traceback
}}}
The culprit was telling `plot_vector_field3d` to do `plot(v)` for each
vector instead of `v.plot()` as it should have. However, this exposes
something else really dumb - namely, that `plot.py` turns vectors into
tuples before it plots them
{{{
from sage.structure.element import is_Vector
if kwds.get('parametric',False) and is_Vector(funcs):
funcs = tuple(funcs)
if hasattr(funcs, 'plot'):
G = funcs.plot(*args, **original_opts)
# if we are using the generic plotting method
else:
}}}
this presumably avoids the fact that
{{{
sage: w = vector([x^2,3,x^3])
sage: plot(w,(x,0,1))
---------------------------------------------------------------------------
NotImplementedError
}}}
but still this is a problem. So after fixing this issue, I'm opening a
ticket for making vector plotting more sensible.
I think this change might need review (?) so I'm setting to 'needs
review'.
To buildbot and reviewer: apply trac_2100-aspect-ratio-rebase.patch and
trac_2100-3d-vector.patch. Depends on #7981, #8632, #10244, and #10143.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/2100#comment:30>
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.
