#13101: enumerate_totallyreal_fields bug fix
----------------------------------------------+----------------------------
       Reporter:  jvoight                     |         Owner:
           Type:  defect                      |  davidloeffler
       Priority:  major                       |        Status:  needs_work
      Component:  number fields               |     Milestone:  sage-5.12
       Keywords:  sd51                        |    Resolution:
        Authors:  Robert Harron, John Voight  |     Merged in:
Report Upstream:  N/A                         |     Reviewers:  Alex Ghitza
         Branch:                              |   Work issues:
       Stopgaps:                              |  Dependencies:
----------------------------------------------+----------------------------

Comment (by jvoight):

 Did someone add

 {{{
 sage: enumerate_totallyreal_fields_all(8, 10^8)
 }}}

 to the doctest?  This was just an easy example that caused the stupid bug
 and I don't think it was ever intended for a doctest.  But then we also
 probably want to keep it for future testing purposes, right?

 There is an easy doctest in there already

 {{{
 enumerate_totallyreal_fields_rel(F, 2, 2000)
 }}}

 and for general purposes of testing, this should be enough, IMHO.  JV

--
Ticket URL: <http://trac.sagemath.org/ticket/13101#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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to