#12798: list_plot3d plots extraneous points at z=0 and doesn't take color or
rgbcolor as keywords
--------------------------------------------------------------+-------------
Reporter: ppurka | Owner:
jason, was
Type: defect | Status:
needs_review
Priority: major | Milestone:
sage-5.1
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:
--------------------------------------------------------------+-------------
Changes (by ppurka):
* reviewer: Karl-Dieter Crisman => Karl-Dieter Crisman, Punarbasu
Purkayastha
Old description:
> 1. `list_plot3d` plots extra points at the horizontal x-y plane. This
> seems to stem from the default value of 0.0 that is assigned in the code.
> See [https://groups.google.com/d/topic/sage-devel/RAdolRdjSTk/discussion
> this post].
>
> 2. This function also does not take in `color` or `rgbcolor` as keywords.
> Rest of the plot commands including `plot`, `list_plot`, `plot3d` do take
> in these keywords.
>
> First attached patch fixes both these problems, second one fixes more doc
> and clarifies output.
> ----
> Apply
> [attachment:trac_12798-fix_extraneous_points_and_kwds_in_list_plot3d.patch]
> and [attachment:trac_12798-more-doc.patch] to `devel/sage`.
New description:
1. `list_plot3d` plots extra points at the horizontal x-y plane. This
seems to stem from the default value of 0.0 that is assigned in the code.
See [https://groups.google.com/d/topic/sage-devel/RAdolRdjSTk/discussion
this post].
2. This function also does not take in `color` or `rgbcolor` as keywords.
Rest of the plot commands including `plot`, `list_plot`, `plot3d` do take
in these keywords.
First attached patch fixes both these problems, second one fixes more doc
and clarifies output.
----
Apply
[attachment:trac_12798-fix_extraneous_points_and_kwds_in_list_plot3d.patch]
and [attachment:trac_12798-more-doc.2.patch] to `devel/sage`.
--
Comment:
Your patch looks good. The documentation was indeed lacking. I added just
one small comment (# long time) in the last example in your patch.
Positive review from me.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12798#comment:12>
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.
