#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 was):
> It is clear from a very first look at the patch that this will massively
conflict with #9343 (why
I'm sure this will be easy to merge with #9343. It's probably best to get
9343 in first, since it is much more difficult, then rebase the current
patch against it.
> I would prefer to postpone: hashing, printing
The current hashing and printing setup is complete and total crap, and
needs to be removed ASAP. It renders number fields completely useless for
any nontrivial applications that involve prime ideals.
> Also, I'm personally not completely convinced about the best way to
print
I saw that. I think the best solution is exactly what I implemented in
the attached patch. Hash based on gens (trivial, as a hash should be).
Print in reduced form only in small trivial cases (by default), but allow
the user to easily up the cutoff if they want, for some reason.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9400#comment:6>
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.
