#12798: list_plot3d plots extraneous points at z=0 and doesn't take color or
rgbcolor as keywords
--------------------------------------------------------------+-------------
Reporter: ppurka | Owner:
jdemeyer
Type: defect | Status:
needs_work
Priority: major | Milestone:
sage-5.2
Component: graphics | Resolution:
Keywords: list_plot3d, sd40.5 | Work issues:
Report Upstream: N/A | Reviewers:
Karl-Dieter Crisman, Punarbasu Purkayastha
Authors: Punarbasu Purkayastha, Karl-Dieter Crisman | Merged in:
Dependencies: | Stopgaps:
--------------------------------------------------------------+-------------
Comment (by ppurka):
@jdemeyer: can you test the
[attachment:trac_12798-dont_pass_nan_to_ParametricSurface.patch new patch]
I have put up. I prevent `nan`s from being passed on to the
`ParametricSurface` class. Since I based it on 5.1beta5 (don't have
5.2alpha0 at home right now), you may apply the previous two patches
(either ours, or yours) and then apply this new patch.
Why I opted for this - modifying `ParametricSurface` has a high
probability of introducing more bugs than fixing the one already present.
If we modify it, then one option is to put checks for nan or inf in line
588 (as of the 5.1beta5) of `parametric_surface.pyx`
{{{
res.x, res.y, res.z = self.f(uu, vv)
}}}
But doing so implies that we also need to modify `urange, vrange, n` and
`m` in that file, and I believe that this requires a major editing of the
code.
It seemed better/safer to actually modify `list_plot3d` instead of
`ParametricSurface`.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12798#comment:43>
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.
