#4264: change E.a_invariants() for an elliptic curve to return a tuple
---------------------------+------------------------------------------------
Reporter: was | Owner: was
Type: enhancement | Status: new
Priority: minor | Milestone: sage-3.2
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Changes (by cremona):
* priority: major => minor
Comment:
I have made a patch (not yet attached) which implements this. It was
easy to do what was suggested (and make the consequent cosmetic changes in
doctests from [...] to (...) ) but there were two similar but distinct
other issues:
* Several Sage functions (the {{{__init__}}} function in the
!EllipticCurve classes) expect the a-invs input parameters to be a list
and not a tuple. I tried changing them to accept tuples but it caused too
many difficulites with parsing different input possibilities so I reverted
that.
* In several places where elliptic curves in other systems are
initialised (e.g. mwrank, gp) lists are required for the parsing done by
the wrappers.
In all the above I sorted everything out by inserting list(...) around
{{{blah.ainvs()}}} or {{{blah.a_invariants()}}}, which works but is ugly.
Is there a better way? Even just having a new function a_list() to be
list(self.ainvs()) would be a bit cleaner. We already have the
unnecessary synonyms a_invariants() and ainvs().
I'll wait for reaction before going further. In particular, I have not
yet tested anything outside the elliptic_curves directory, e.g. the
tutorial.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4264#comment:2>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
