#16397: Symbolic cmp
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:
           Type:  defect             |       Status:  new
       Priority:  major              |    Milestone:  sage-6.4
      Component:  symbolics          |   Resolution:
       Keywords:                     |    Merged in:
        Authors:                     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/rws/symbolic_cmp                 |  af5f800fdc616022d503b5e845ec81ebfef7c25e
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by rws):

 * commit:  62be17eb6eccffcddcc61e4f18a2123ec0c364d8 =>
     af5f800fdc616022d503b5e845ec81ebfef7c25e


Comment:

 Trying to push this ticket a bit. I also found this `cmp` call:
 {{{
  File "matrix_integer_dense.pyx", line 604, in
 sage.matrix.matrix_integer_dense.Matrix_integer_dense.__richcmp__
 (build/cythonized/sage/matrix/matrix_integer_dense.c:9503)
   File "element.pyx", line 873, in sage.structure.element.Element._richcmp
 (build/cythonized/sage/structure/element.c:8891)
   File "element.pyx", line 855, in
 sage.structure.element.Element._richcmp_
 (build/cythonized/sage/structure/element.c:8615)
   File "element.pyx", line 902, in sage.structure.element.Element._richcmp
 (build/cythonized/sage/structure/element.c:9316)
   File "element.pyx", line 949, in
 sage.structure.element.Element._richcmp_c_impl
 (build/cythonized/sage/structure/element.c:9645)
   File "matrix_dense.pyx", line 126, in
 sage.matrix.matrix_dense.Matrix_dense._cmp_c_impl
 (build/cythonized/sage/matrix/matrix_dense.c:3092)
   File "expression.pyx", line 3066, in
 sage.symbolic.expression.Expression.__cmp__
 (build/cythonized/sage/symbolic/expression.cpp:16865)
 }}}
 Do I understand it right that this should call `math_sorted` instead of
 `cmp`?
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=215cff4e33f7a63e2da4cdde45c7f491046271ac
 215cff4]||{{{Merge branch 'develop' into t/16397/symbolic_cmp}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=af5f800fdc616022d503b5e845ec81ebfef7c25e
 af5f800]||{{{16397: use print_sorted() in two further places; fix
 doctest}}}||

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

Reply via email to