#4392: [with new patch, needs review] smallest_integer() is broken
--------------------------------+-------------------------------------------
Reporter: cremona | Owner: cremona
Type: defect | Status: new
Priority: major | Milestone: sage-3.2
Component: number theory | Resolution:
Keywords: number field ideal |
--------------------------------+-------------------------------------------
Changes (by cremona):
* summary: [with patch, needs review] smallest_integer() is broken =>
[with new patch, needs review]
smallest_integer() is broken
Comment:
The third patch sage-trac4392-3.patch adds a new rather useful function
coordinates() to the class NumberFieldIdeal which expresses an element of
the field as a QQ-linear combination of the integral basis of the ideal.
(This does not work for relative ideals, but then what does?). This uses
a change-of-basis matrix which is cached. It results in simpler
implementations for the _contains_() and is_integral() functions, as well
as the smallest_integer() function where this started.
I checked that all doctests in sage/rings/number_field pass.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4392#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
-~----------~----~----~----~------~----~------~--~---
