#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
-~----------~----~----~----~------~----~------~--~---

Reply via email to