#5890: [with patch, needs review] clean up
schemes/elliptic_curves/ell_generic.py
---------------------------+------------------------------------------------
Reporter: AlexGhitza | Owner: AlexGhitza
Type: defect | Status: assigned
Priority: major | Milestone: sage-4.0
Component: number theory | Keywords: elliptic curve field
---------------------------+------------------------------------------------
Changes (by AlexGhitza):
* owner: was => AlexGhitza
* status: new => assigned
Comment:
The attached patch does some cleaning up, and it depends on #5765.
It moves all the twists-related methods from {{{ell_generic.py}}} to
{{{ell_field.py}}}, as well as the alias {{{base_field = base_ring}}}.
It also makes {{{change_ring}}} into an alias for {{{base_extend}}}, since
the two have the exact same functionality and equivalent code.
Lastly, the standalone function {{{Hasse_bounds}}} is moved from
{{{ell_generic.py}}} to {{{schemes/plane_curves/projective_curve.py}}},
which is a more natural place for it.
There are more things to do, but they are issues with the generic code for
curves rather than the code for elliptic curves, so I think they should be
fixed in {{{schemes/plane_curves}}} instead. I'll be looking into that
soon.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5890#comment:3>
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
-~----------~----~----~----~------~----~------~--~---
