#13878: Fix failing assertion in linbox/matrix/permutation-matrix.h
--------------------------------------------+-------------------------------
Reporter: SimonKing | Owner: jason, was
Type: defect | Status: new
Priority: major | Milestone: sage-5.6
Component: linear algebra | Resolution:
Keywords: debug, linbox, assertion | Work issues:
Report Upstream: N/A | Reviewers:
Authors: | Merged in:
Dependencies: | Stopgaps:
--------------------------------------------+-------------------------------
Comment (by nbruin):
Replying to [comment:7 SimonKing]:
> What is needed to do to finalise your preliminary patch? Adding the
above example as a doc test would be half cheating, because this used only
to fail in debug mode. But still, one could include this example and
comment on it accordingly.
At first blush it seems this is an upstream bug, so this would only be a
workaround, not a proper fix. Furthermore, it seems this code seems to
work in an NDEBUG linbox, so perhaps the test is just overzealous. I
haven't checked linbox's documentation to see if they declare this input
illegal.
Other than that, there already are doctests that test this issue, so a
comment should be enough documentation for this fix.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13878#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 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.
