#16804: points() has problems with some lists of complexes containing real (i.e.
imag(x)==0) values.
-------------------------------------+-------------------------------------
Reporter: charpent | Owner:
Type: defect | Status: needs_work
Priority: minor | Milestone: sage-7.3
Component: graphics | Resolution:
Keywords: graphics, plot, | Merged in:
complex | Reviewers: Vincent Delecroix
Authors: Javier Honrubia | Work issues:
González | Commit:
Report Upstream: N/A | d48da09f3e8107712654b6ff73df8bba25bb6ebf
Branch: | Stopgaps:
u/jhonrubia6/points___has_problems_with_some_lists_of_complexes_containing_real__i_e__imag_x___0__values_|
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by vdelecroix):
* status: needs_review => needs_work
Comment:
There should be a line break at the end of the function
`xydata_from_point_list` to separate it from `plot`.
In your context, it makes no sense to use reduce. You should use either
`all` or `any`.
I do not feel very confortable with the syntax
{{{
sage: points(CC(0,1)) # only one point !
sage: points(1) # one point but does not work!
}}}
As the method is plural, I guess we should just forbid this case. What do
you think? Depending on your answer you might want to fix
{{{
sage: points(CC(0,1)) # fine
sage: points(CDF(0,1)) # fine
sage: points(complex(1j)) # used to work
}}}
And ideally, numpy complex numbers should also be supported. You would
better use `numbers` as adviced in [comment:8 comment:8].
--
Ticket URL: <http://trac.sagemath.org/ticket/16804#comment:31>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
