#4722: BUG - number field K.hilbert_class_polynomial() is a *lie*
---------------------------+------------------------------------------------
Reporter: was | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.2.2
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Comment (by kohel):
Replying to [comment:1 was]:
> Who to blame? Either me or David Kohel, since this was done before Sage
was under revision control.
Not me. This is a wrapper for the Pari/gp function quadhilbert.
I find this unanswered question about what it returns:
http://pari.math.u-bordeaux.fr/archives/pari-users-0402/msg00000.html
Certainly it does not return the hilbert class polynomial (minimal
polynomial of the j-invariant), rather it returns a "nicer" polynomial
over QQ which generates the Hilbert class field over K.
I agree that a name change is in order to avoid this confusion, but
I don't have a suggestion other than
hilbert_class_field_[relative_]defining_polynomial
which is a bit long, but descriptive. Note that the hilbert_class_field
does not have this as a defining polynomial (hence the relative_),
since it is formed as a compositum of the quadratic and degree h
extensions rather than a relative extension.
The documentation should also be corrected to say that Schertz's method
is used (only) for D < 0. A reference to Schertz's methods and whatever
method (Stark units?) is used for D > 0 would be desirable.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4722#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
-~----------~----~----~----~------~----~------~--~---