Opened 11 years ago
Closed 11 years ago
#13220 closed defect (fixed)
bug in morphisms of chain complexes
Reported by: | jhpalmieri | Owned by: | jhpalmieri |
---|---|---|---|
Priority: | minor | Milestone: | sage-5.2 |
Component: | algebraic topology | Keywords: | sd40 |
Cc: | Merged in: | sage-5.2.rc0 | |
Authors: | John Palmieri | Reviewers: | Marshall Hampton |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
The attached patch fixes a bug when defining morphisms of chain complexes: the code was supposed to check that a product of two matrices was zero, but used
A * B.is_zero()
rather than
(A * B).is_zero()
As a result, some valid maps of chain complexes were flagged as invalid. The patch fixes this.
Attachments (1)
Change History (5)
comment:1 Changed 11 years ago by
Status: | new → needs_review |
---|
Changed 11 years ago by
Attachment: | trac_13220-chain-complex.patch added |
---|
comment:2 Changed 11 years ago by
Reviewers: | → Marshall Hampton |
---|---|
Status: | needs_review → positive_review |
comment:3 Changed 11 years ago by
Milestone: | sage-5.3 → sage-5.2 |
---|
comment:4 Changed 11 years ago by
Merged in: | → sage-5.2.rc0 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.