#17062: Fix an optional doctest output after #16746
----------------------------------+--------------------------------
Reporter: cremona | Owner:
Type: defect | Status: new
Priority: minor | Milestone: sage-6.4
Component: packages: optional | Keywords:
Merged in: | Authors: John Cremona
Reviewers: | Report Upstream: N/A
Work issues: | Branch:
Commit: | Dependencies:
Stopgaps: |
----------------------------------+--------------------------------
After #16746, output of dicts is sorted by index, and this results in one
test in src/doc/en/bordeaux_2008/elliptic_curves.rst failing when the
optional database_cremona_ellcurve database is installed:
{{{
sage: C = CremonaDatabase()
sage: C[37]
{'allbsd': {'a1': [1,
5.98691729246392,
0.305999773834052,
0.0511114082399688,
1.0],
...
}}}
is what comes out, since the field with key 'allbsd' which only exists in
the optional larger database comes before key 'allcurves' which is in the
standard version. I propose replacing the test with
{{{
sage: C = CremonaDatabase()
sage: C[37]['allcurves']
{'a1': [[0, 0, 1, -1, 0], 1, 1],
'b1': [[0, 1, 1, -23, -50], 0, 3],
'b2': [[0, 1, 1, -1873, -31833], 0, 1],
'b3': [[0, 1, 1, -3, 1], 0, 3]}
}}}
and will make this change soon.
--
Ticket URL: <http://trac.sagemath.org/ticket/17062>
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/d/optout.
