Ticket #3501 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch, with positive review] charpoly of zero matrix over a cyclotomic field fails

Reported by: craigcitro Owned by: craigcitro
Priority: major Milestone: sage-3.0.4
Component: linear algebra Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

This is broken: 

sage: Matrix(CyclotomicField(13),3).charpoly()

The attached patch fixes it.

Attachments

trac-3501.patch Download (1.3 KB) - added by craigcitro 5 years ago.

Change History

Changed 5 years ago by craigcitro

comment:1 Changed 5 years ago by cpernet

  • Summary changed from [with patch, needs review] charpoly of zero matrix over a cyclotomic field fails to [with patch, with positive review] charpoly of zero matrix over a cyclotomic field fails

This patch looks good and fixes the bug. I have run the doctest on 3.0.4alpha1, and they pass.

comment:2 Changed 5 years ago by mabshoff

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

Merged in Sage 3.0.4.alpha1

Note: See TracTickets for help on using tickets.