#9400: modify the NumberField constructor to pass in optional integer B such
that
all the internal pari routines will replace the discriminant by its gcd
with B, making some things massively faster.
-----------------------------+----------------------------------------------
Reporter: was | Owner: davidloeffler
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.5.3
Component: number fields | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------------+----------------------------------------------
Comment(by jdemeyer):
It is clear from a very first look at the patch that this will massively
conflict with #9343 (why did nobody point this out to me earlier???).
Personally, I would prefer to postpone the last two points of the
description, i.e.
* implement hashing for number field ideals that isn't a stupid string
repr, hence vastly faster
* make number field ideals *not* print in reduced form; this will look
uglier, but is massively faster and more sensible for any real
applications, as people learned constantly at sage days 22.
and do them after #9343 is merged (I also want to change the hashing of
PARI objects, see #9667). Also, I'm personally not completely convinced
about the best way to print NumberFieldIdeals (see also my post at sage-
devel).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9400#comment:5>
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.
