#4388: elliptic curves: basis_matrix command totally broken
---------------------------+------------------------------------------------
Reporter: was | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.2
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Comment (by cremona):
Comment: I noticed this when I reworked the whole of period_lattice.py
recently. But the function basis_matrix only exists because
PeriodLattice_ell derives from PeriodLattice and hence from
FreeModule_generic_pid. But I don't think it makes a lot of sense to ask
for a basis matrix in a case like this, when the thing is a Z-module but
it does not sit in an ambient Q-vector space.
If people agree, we should at least add the function but have it raise a
sensible error.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4388#comment:1>
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
-~----------~----~----~----~------~----~------~--~---
