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: |
Status badges
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.
