#9483: implicit_plot3d fails on symbolic inputs that can't be automatically
differentiated
-----------------------------------+----------------------------------------
Reporter: cwitty | Owner: cwitty
Type: defect | Status: needs_info
Priority: major | Milestone: sage-4.5.2
Component: graphics | Keywords:
Author: Carl Witty | Upstream: N/A
Reviewer: Karl-Dieter Crisman | Merged:
Work_issues: |
-----------------------------------+----------------------------------------
Changes (by kcrisman):
* status: needs_review => needs_info
* reviewer: => Karl-Dieter Crisman
Comment:
Okay, one issue is that this depends on some other ticket which may or may
not get reviewed.
And I think I see why this fix works, sort of - when smooth is True,
MarchingCubesTriangles calls the gradient, though presumably smooth was
already False - this seems weird to me. But why not make the change at
{{{
def __init__(self, f, xrange, yrange, zrange,
contour=0, plot_points="automatic",
region=None, smooth=True, gradient=None,
vertex_color=None,
**kwds):
}}}
? I guess someone could still pass in 'smooth', though it seems like then
this should be deprecated as an option - or we should have prominent TODO
to add it. But also
{{{
if smooth and gradient is None:
}}}
it seems like we should just get rid of this whole thing, since smooth
will never be True.
Anyway, because I don't quite understand those things, I'm putting
`needs_info`, but it seems good overall - and I apologize for any
ignorance on my part.
On the plus side, it does get rid of the traceback as advertised! For
some reason I don't have 3d plotting on the computer I'm checking this one
(old Java?), but that's still pretty good.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9483#comment:4>
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.
