#7872: Adding coordinate transformations to plot3d
--------------------------------------------------------+-------------------
Reporter: olazo | Owner: olazo
Type: enhancement | Status:
needs_work
Priority: minor | Milestone:
sage-4.3.3
Component: graphics | Keywords:
Author: Bill Cauchois, Oscar Gerardo Lazo Arjona | Upstream: N/A
Reviewer: Karl-Dieter Crisman | Merged:
Work_issues: |
--------------------------------------------------------+-------------------
Changes (by kcrisman):
* status: needs_review => needs_work
Comment:
> > Also, why are Cylindrical and Spherical imported in plot3d/all.py?
It's not clear to me why someone would want that available but not just
use e.g. spherical_plot3d or just import Spherical if they really needed
it?
>
> This is one of the major features of the transformation system. You can
use spherical_plot3d, which will graph a function r in terms of phi and
theta, or you can specify transformation=Spherical() which lets you choose
the dependent and independent variables.
>
> For example,
> {{{
> plot3d(..., transformation=Spherical('phi', ['r', 'theta']))
> }}}
> will graph a function phi in terms of r and theta. So basically you get
more flexibility by using Spherical(). Is there a way I could make that
clearer in the documentation?
>
Most of the changes seem fine. I am still confused about this, though.
Isn't
{{{
sage: spherical_plot3d(e^-y,(x,0,2*pi),(y,0,pi))
}}}
already allowing one to use a different name for the variables?
Ah, I think I see. There are six different possibilities for order of
independent variables and the dependent variable, and you want to allow
all of these - it that it? Maybe you should make it clearer that this is
what is going on by saying that it's not the names, but rather which one
is the dependent variable, you are changing - in fact, that the names look
the same only to preserve the usual terminology - and then adding to
{{{
sage: Spherical('theta', ['r', 'phi'])
Spherical coordinate system (theta in terms of r, phi)
}}}
by actually plotting something with this.
Also, in line 623, I think it should be
{{{
sage: var('r,u,v')
}}}
instead of r, u, u; I am surprised this doesn't give a doctest failure (I
didn't get a chance to run that right now).
Is it ok if I make it still 'needs work' to just clarify these? For most
users it will all be irrelevant, of course, but if we are really going to
import Spherical() at the top level, then it should be very clear what the
point is.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7872#comment:20>
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.
