#11758: Bug in global_height function
-----------------------------+----------------------------------------------
   Reporter:  dkrumm         |          Owner:  cremona      
       Type:  defect         |         Status:  new          
   Priority:  major          |      Milestone:  sage-4.7.2   
  Component:  number theory  |       Keywords:  global height
Work_issues:                 |       Upstream:  N/A          
   Reviewer:                 |         Author:               
     Merged:                 |   Dependencies:               
-----------------------------+----------------------------------------------

Comment(by dkrumm):

 Replying to [comment:7 mderickx]:
 > Replying to [comment:6 dkrumm]:
 > > I hope I've created the patch correctly. Sorry, I don't know what
 doctest is.
 >
 > A doctest is some test contained in the documentation of a function.
 Just do:
 > {{{
 > sage: K.<s> = QuadraticField(2)
 > sage: s.global_height?
 > }}}
 > You should now see documentation on how to use the global height. The
 examples like:
 > {{{
 >           sage: R.<x> = QQ[]
 >           sage: K.<a> = NumberField(x^4+3*x^2-17)
 >           sage: b = a/2
 >           sage: b.global_height()
 >           2.869222240687...
 >           sage: b.global_height(prec=200)
 >           2.8692222406879748488543678846959454765968722137813736080066
 > }}}
 > given there are called doctests since they are used for automated
 testing of the sage library (see
 http://www.sagemath.org/doc/developer/doctesting.html). All those examples
 are tested and should produce the shown output before a patch is accepted.
 In this case it means that the doctests have to change since the answers
 show are not correct. What john asked is if you could also add your
 example wich you posted in the description as a test (ofcourse with the
 new and imporved answers.

 Ok, thanks for the explanation. Unfortunately, I can't put in those
 examples yet, because my sage library won't build. So even though I've
 made the changes in the code, I can't get sage to recognize them. I have
 this problem on my personal computer, and also on the ones in my
 department.. do you think this problem with Lion and Xcode will be fixed
 any time soon?

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11758#comment:8>
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.

Reply via email to