Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

plot3d transformations should take symbolic vectors #12532

Closed
jasongrout opened this issue Feb 18, 2012 · 8 comments
Closed

plot3d transformations should take symbolic vectors #12532

jasongrout opened this issue Feb 18, 2012 · 8 comments

Comments

@jasongrout
Copy link
Member

This works:

var('r,theta,z,rho,phi')
cylindrical=[r*cos(theta), r*sin(theta), z]
plot3d(3,(theta,0,pi/2),(z,0,pi/2),transformation=cylindrical)

but this doesn't (but should work)

cylindrical(r,theta,z)=[r*cos(theta), r*sin(theta), z]
plot3d(3,(theta,0,pi/2),(z,0,pi/2),transformation=cylindrical)

The problem is we pay too much attention to the type passed into the transformation argument. We should also allow symbolic vectors, as well as lists and tuples.

CC: @kcrisman

Component: graphics

Author: Veronica Suaste

Reviewer: Miguel Marco

Merged: sage-5.10.beta4

Issue created by migration from https://trac.sagemath.org/ticket/12532

@sagetrac-veronica
Copy link
Mannequin

sagetrac-veronica mannequin commented May 14, 2013

Attachment: trac_12532_transformation_symbolic_vector.patch.gz

Now the transformation also can be done with a symbolic vector

@miguelmarco
Copy link
Contributor

comment:2

Looks good.

@miguelmarco
Copy link
Contributor

Reviewer: mmarco

@kcrisman
Copy link
Member

Author: Veronica Suaste

@kcrisman
Copy link
Member

Changed reviewer from mmarco to Miguel Marco

@kcrisman
Copy link
Member

comment:5

PS to mmarco - your web link on the main Trac page gives Página no encontrada.

@miguelmarco
Copy link
Contributor

comment:6

I know, i quited that position two years ago, and i have no personal webpage right now. I have to take care of that.

@jdemeyer
Copy link

Merged: sage-5.10.beta4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants