New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generators for homology of simplicial complexes #30838
Comments
Branch: u/jhpalmieri/homology-generators |
Commit: |
New commits:
|
Author: John Palmieri |
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
comment:5
Before this branch:
Note that no generators are given at all in the simplicial complex case. In the other cases, the answers are given in terms of the chain groups. After:
|
This comment has been minimized.
This comment has been minimized.
Reviewer: Travis Scrimshaw |
comment:8
LGTM. |
comment:9
Thanks very much. |
Changed branch from u/jhpalmieri/homology-generators to |
This is a followup to #6100, and was reported on this ask question. If
K
is a simplicial complex, thegenerators
argument toK.homology(...)
does nothing:K.homology(...)
calls thehomology
method for generic cell complexes, which in turn calls_homology_
for simplicial complexes, and that method ignores thegenerators
keyword.There is an easy solution: in the generic
homology
method, ifgenerators
is True, then don't call_homology_
, just continue in that method. This will give information about chain complex generators. It is possible, although obscure, to extract the simplicial complex information from that.A better solution: actually do the obscure work, don't force the users to figure it out. That is:
K.chain_complex(generators=True)
, andK._n_cells_sorted()
to convert the chain complex generators (which are just vectors in the corresponding free module over the base ring) to actual chains in the simplicial complex.CC: @tscrim
Component: algebraic topology
Author: John Palmieri
Branch/Commit:
a5b41da
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/30838
The text was updated successfully, but these errors were encountered: