Opened 11 years ago

Closed 11 years ago

#5089 closed defect (fixed)

[with patch, positive review] add kernel method for sparse integer matrices

Reported by: jhpalmieri Owned by: was
Priority: minor Milestone: sage-3.3
Component: linear algebra Keywords: sparse integer matrix kernel
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

sage: M = matrix(ZZ, 2, 3, [1,2,3,4,5,6])
sage: M.kernel()

works fine, while

sage: M = matrix(ZZ, 2, 3, [1,2,3,4,5,6], sparse=True)
sage: M.kernel()

gives an error, TypeError: Argument K (= Integer Ring) must be a field.

The attached patch fixes this -- it adds a kernel method for sparse integer matrices, which just calls self.dense_matrix().kernel(...).

Attachments (1)

5089.patch (1.8 KB) - added by jhpalmieri 11 years ago.

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by jhpalmieri

comment:1 Changed 11 years ago by was

  • Summary changed from [with patch, needs review] add kernel method for sparse integer matrices to [with patch, positive review] add kernel method for sparse integer matrices

looks good.

comment:2 Changed 11 years ago by mabshoff

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

Merged in Sage 3.3.alpha2

Cheers,

Michael

Note: See TracTickets for help on using tickets.