Opened 11 years ago
Closed 10 years ago
#9122 closed enhancement (fixed)
conversions between simplicial and cubical complexes
Reported by: | jhpalmieri | Owned by: | jhpalmieri |
---|---|---|---|
Priority: | minor | Milestone: | sage-4.7.1 |
Component: | algebraic topology | Keywords: | |
Cc: | mhampton | Merged in: | sage-4.7.1.alpha2 |
Authors: | John Palmieri | Reviewers: | Marshall Hampton |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
This patch implements conversions between simplicial and cubical complexes. It also implements the following: if you call
sage: SimplicialComplex(X)
it tests to see if X has a _simplicial_
method, and if it does, it calls that and returns the answer. So if anyone wants to convert their favorite Sage object to a simplicial complex, they just have to write a _simplicial_
method for it; then SimplicialComplex(blah)
will work. Same for a _cubical_
method and cubical complexes.
Attachments (1)
Change History (8)
comment:1 Changed 11 years ago by
- Status changed from new to needs_review
comment:2 Changed 10 years ago by
- Cc mhampton added
comment:3 Changed 10 years ago by
- Status changed from needs_review to needs_info
comment:4 Changed 10 years ago by
- Status changed from needs_info to needs_review
Oh, I think it's partly the fault of the patch: the docstrings should start with r"""
, not just """
. Try this new patch.
Changed 10 years ago by
comment:5 Changed 10 years ago by
- Status changed from needs_review to positive_review
All homology tests passed and the documentation looks fixed, so I think this is OK now.
comment:6 Changed 10 years ago by
- Milestone changed from sage-4.7 to sage-4.7.1
- Reviewers set to Marshall Hampton
comment:7 Changed 10 years ago by
- Merged in set to sage-4.7.1.alpha2
- Resolution set to fixed
- Status changed from positive_review to closed
Maybe I am building the docs incorrectly, but it seems kind of sad that the documentation for _simplicial_ and _cubical_ does not show up in the reference manual, and if you look at it in the notebook it is not currently rendered correctly.
Otherwise the functionality appears to be correctly implemented (although I have very little expertise in this area). The flaws in rendering are probably the fault of the notebook code rather than this patch.