Ticket #5168 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patch, positive review] matrix0.pyx: fix doctest for commutator

Reported by: jhpalmieri Owned by: was
Priority: trivial Milestone: sage-3.3
Component: linear algebra Keywords: matrix0, commutator
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

Here is the extent of the docstring for the commutator method in matrix0.pyx:

        Return the commutator self*other - other*self.

        EXAMPLES:
            sage: A = Matrix(QQ[['t']], 2, 2, range(4))

Fix the doctest so that it actually computes a commutator.

Attachments

5168.patch Download (900 bytes) - added by jhpalmieri 4 years ago.

Change History

Changed 4 years ago by jhpalmieri

comment:1 Changed 4 years ago by cwitty

  • Summary changed from [with patch, needs review] matrix0.pyx: fix doctest for commutator to [with patch, positive review] matrix0.pyx: fix doctest for commutator

New doctests look good, and they pass.

(Good catch on noticing the original bug, too.)

Positive review.

comment:2 Changed 4 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in Sage 3.3.alpha6.

Cheers,

Michael

Note: See TracTickets for help on using tickets.