#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: |
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.