Opened 12 years ago
Closed 11 years ago
#9122 closed enhancement (fixed)
conversions between simplicial and cubical complexes
| Reported by: | John Palmieri | Owned by: | John Palmieri |
|---|---|---|---|
| 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: |
Status badges
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 12 years ago by
| Status: | new → needs_review |
|---|
comment:2 Changed 12 years ago by
| Cc: | mhampton added |
|---|
comment:3 Changed 12 years ago by
| Status: | needs_review → needs_info |
|---|
comment:4 Changed 12 years ago by
| Status: | needs_info → 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 12 years ago by
| Attachment: | trac_9122-cubical-simplicial.patch added |
|---|
comment:5 Changed 11 years ago by
| Status: | needs_review → positive_review |
|---|
All homology tests passed and the documentation looks fixed, so I think this is OK now.
comment:6 Changed 11 years ago by
| Milestone: | sage-4.7 → sage-4.7.1 |
|---|---|
| Reviewers: | → Marshall Hampton |
comment:7 Changed 11 years ago by
| Merged in: | → sage-4.7.1.alpha2 |
|---|---|
| Resolution: | → fixed |
| Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
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.