#11836: gens_reduced() does not handle "large" ideals
-----------------------------+----------------------------------------------
Reporter: mirela | Owner: jdemeyer
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: number fields | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Marco Streng | Author: Jeroen Demeyer
Merged: | Dependencies: #11130
-----------------------------+----------------------------------------------
Comment(by mstreng):
Ok, sorry for the amount of posts. I'm done reading the patch, and it
looks good. So let me sum things up:
Could you...
* replace flag 1 by flag 2 in bnfisprincipal to fix one more bug, and
remove the confusing pari warnings all at the same time!
* find out why caching doesn't seem to work. Is the problem a difference
between hasattr and catching {{{AttributeError}}}? see [comment:16
mstreng]
* while you're doing the above, think about the minor things of
[comment:14 mstreng]?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11836#comment:18>
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.
