#9266: bug in global_integral_model for Elliptic Curves over Number Fields
-------------------------------+--------------------------------------------
Reporter: wuthrich | Owner: cremona
Type: defect | Status: new
Priority: minor | Milestone: sage-4.4.5
Component: elliptic curves | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Comment(by cremona):
Replying to [comment:2 wuthrich]:
> ''w'' is the algebraic integer (1+sqrt(5))/2 and it is the coefficient
of this integral Weierstrass equation. So this is a global_integral_model.
We should not check if the denominator in some basis is 1, but rather if
the coefficients are integers.
OK then so we should do
{{{
if not all([z.is_integral() for z in ai]):
}}}
I'm too busy writing lectures for SD22 to make the patch myself!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9266#comment:3>
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 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.
