#5428: [with patch, needs review] Doctest failure in
devel/sage/doc/en/bordeaux_2008/method_of_graphs.rst
---------------------+------------------------------------------------------
Reporter: jsp | Owner: mabshoff
Type: defect | Status: new
Priority: blocker | Milestone: sage-3.4
Component: doctest | Keywords:
---------------------+------------------------------------------------------
Comment(by GeorgSWeber):
Mmmh, using "sorted(rels)" instead of "rels" certainly points in the right
direction.
But IMHO, the fix(es) should not be done inside
"sparse_2term_quotient(rels, n, F)", where the first argument "rels" is
processed, but way before in the calling functions.
More precisely, the function "modS_relations(syms)" should not do "return
rels" but should do "return sorted(rels)", and in the function
"relation_matrix_wtk_g0(...)" we should do "rels =
sorted(rels.update(modI_relations(syms, sign)))", and possibly there are
one or two other places more like this.
But could we not postpone this to a point release 3.4.x, and open another
ticket for that?!? (The Bordeaux ReST file would have to be touched a
second time, but that's trivial.)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5428#comment:19>
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
-~----------~----~----~----~------~----~------~--~---
