#5660: [with patch, positive review] count_points(1) for elliptic curves over
finite fields is stupid
-----------------------------+----------------------------------------------
Reporter: was | Owner: davidloeffler
Type: enhancement | Status: closed
Priority: major | Milestone: sage-4.1.2
Component: elliptic curves | Resolution: fixed
Keywords: | Reviewer: Alex Ghitza
Author: John Cremona | Merged: Sage 4.1.2.alpha0
-----------------------------+----------------------------------------------
Changes (by cremona):
* cc: JCooley (added)
Comment:
Replying to [comment:7 AlexGhitza]:
> This is because the generic code for schemes is behaving badly and
doesn't sort the list of points before returning it.
>
> I will fix this in #6810, but for now the above doctest should just be
changed to {{{[(0 : 0 : 1), (0 : 1 : 0), (0 : 2 : 1), (1 : 0 : 1), (1 : 2
: 1), (2 : 0 : 1), (2 : 2 : 1)]}}}, which is the correct behaviour. I've
added a trivial patch that does this.
>
Thanks, Alex, you are quite right -- and it is my fault for not testing
more before posting the patch.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5660#comment:9>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
