Opened 15 years ago

Closed 15 years ago

#1593 closed defect (fixed)

[with doc patch, with positive review] m4ri -- the documentation of the echelon command only lists 1 algorithm but >= 2 algorithms are supported

Reported by: William Stein Owned by: Martin Albrecht
Priority: minor Milestone: sage-2.10.1
Component: documentation Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description


Attachments (1)

trac_1593.patch (1.3 KB) - added by Martin Albrecht 15 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 15 years ago by William Stein

Owner: changed from tba to Martin Albrecht

Also, there is a bug in algorithm = "classical", since it doesn't check for mutability and doesn't clear the cache.

Changed 15 years ago by Martin Albrecht

Attachment: trac_1593.patch added

comment:2 Changed 15 years ago by Martin Albrecht

Status: newassigned
Summary: m4ri -- the documentation of the echelon command only lists 1 algorithm but >= 2 algorithms are supported[with doc patch] m4ri -- the documentation of the echelon command only lists 1 algorithm but >= 2 algorithms are supported

the attached patch adds 'classical' to the docstring of echelonize. The remark about algorithm="classical" is invalid because the called method _echelon_in_place_classical does check for mutability and clears the cache. See matrix2.pyx for details.

comment:3 Changed 15 years ago by Michael Abshoff

Summary: [with doc patch] m4ri -- the documentation of the echelon command only lists 1 algorithm but >= 2 algorithms are supported[with doc patch, needs review] m4ri -- the documentation of the echelon command only lists 1 algorithm but >= 2 algorithms are supported

comment:4 Changed 15 years ago by Michael Abshoff

Summary: [with doc patch, needs review] m4ri -- the documentation of the echelon command only lists 1 algorithm but >= 2 algorithms are supported[with doc patch, with positive review] m4ri -- the documentation of the echelon command only lists 1 algorithm but >= 2 algorithms are supported

Patch looks good to me.

Cheers,

Michael

comment:5 Changed 15 years ago by Michael Abshoff

Resolution: fixed
Status: assignedclosed

Merged in Sage 2.10.1.alpha2

Note: See TracTickets for help on using tickets.