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