#6141 closed enhancement (fixed)
[with patch, positive review] simplicial complexes: change 'facets' from an attribute to a method
| Reported by: | jhpalmieri | Owned by: | jhpalmieri |
|---|---|---|---|
| Priority: | minor | Milestone: | sage-4.0.1 |
| Component: | algebraic topology | Keywords: | |
| Cc: | Merged in: | 4.0.1.alpha0 | |
| Authors: | John Palmieri | Reviewers: | David Perkinson |
| Report Upstream: | N/A | Work issues: | |
| Branch: | Commit: | ||
| Dependencies: | Stopgaps: |
Status badges
Description
See this thread from sage-devel.
Attachments (1)
Change History (4)
Changed 14 years ago by
| Attachment: | facets.patch added |
|---|
comment:1 Changed 14 years ago by
| Summary: | [with patch, needs review] simplicial complexes: change 'facets' from an attribute to a method → [with patch, positive review] simplicial complexes: change 'facets' from an attribute to a method |
|---|
comment:2 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
While this does represent a backwards incompatible change, I think it is better to resolve it now while the code is newer.
Merged in 4.0.1.alpha0.
comment:3 Changed 14 years ago by
| Authors: | → John Palmieri |
|---|---|
| Merged in: | → 4.0.1.alpha0 |
| Reviewers: | → David Perkinson |
Note: See
TracTickets for help on using
tickets.
The patch makes a simple change. The attribute self.facet is changed to self._facet in all files in the homology directory, and a facets() method is added. I checked the new code, ran the doctests in homology, and tried a few examples of my own. Everything was OK.
I was using Sage Version 4.0.alpha0, Release Date: 2009-05-15 under Fedora 10.