-
-
Notifications
You must be signed in to change notification settings - Fork 523
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
Deprecate Sage's CHomP interface #33777
Comments
This comment has been minimized.
This comment has been minimized.
Branch: u/jhpalmieri/deprecate-chomp |
New commits:
|
Commit: |
comment:4
How does CHomP compare speed-wise to what Sage can natively do? That might be a reason to continue to support it, perhaps as an experimental package. |
comment:5
Since I can't build CHomP on OS X, it's hard for me to compare. If I have a chance, I'll see if it works on a virtual linux machine and compare there. |
comment:6
I should be able to run it on my machine. Can you give me a few tests? |
comment:7
If you install CHomP, the first thing to try is renaming or symlinking the executable
I don't actually know if the "new" executables use the same syntax as the old ones, so I don't know if these symlinks will work with Sage. This is part of the point: our interface is broken, and if we aren't maintaining it (as we haven't been for 5 years, apparently), then maybe we don't have the bandwidth to maintain it and we should deprecate it. Or we can fix it this time and hope that the software remains the same. Even better, we could fix it and produce an experimental CHomP package so that it gets tested every now and then. Even better, we could try to build some sort of library-level interface, rather than just using CHomP's executables. I don't have the time to do any of these except the deprecation part, and given that it's been broken for a long time and we haven't heard complaints, I would guess that it's not super high priority. I also wonder if there is other homology software that we should be looking at instead. See
|
comment:8
An update with the
So the timing is not consistent: |
comment:10
Replying to @jhpalmieri:
Maybe one can build CHomP on macOS with "real" gcc/g++, not with Clang. |
comment:11
Replying to @dimpase:
no, it doesn't work, either. And dependence on old Boost is even more iffy... |
comment:12
I can't build CHomp on Fedora either - probably it needs old Boost. Same error as on macOS:
|
comment:13
Here is the situation: CHomP is currently broken. The "lite" version works, at least on OS X, but has limited functionality: it builds executables for simplicial complexes and cubical complexes, but not chain complexes. The simplicial complex stuff works, but I don't understand the cubical part. There is limited documentation: the only thing I can find is the
So maybe I think we have a few choices:
Opinions? |
comment:14
Oh, and when I say that So if you want to compute homology of a simplicial complex with torsion, no version of |
comment:15
Does |
comment:16
Ah, more apparent abandonware. I can find a version of Kenzo from 2008 at https://www-fourier.ujf-grenoble.fr/~sergerar/Kenzo/, and a version last modified several years ago at https://github.com/gheber/kenzo. The documentation is dated 1999, and according to that, it implements chain complexes and simplicial sets, not cubical complexes, not simplicial complexes. There have been some efforts to update it and improve its interface with Sage (#29879, #30400 comment:48) based on the branch at https://github.com/miguelmarco/kenzo/, but the branch and those efforts appear to have stalled. In any case, I think that's a parallel issue. |
comment:17
OK, looks good. |
Reviewer: Dima Pasechnik |
comment:18
Thank you! |
Changed branch from u/jhpalmieri/deprecate-chomp to |
…nterface <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> - Deprecated in sagemath#33777 (2022). ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#37872 Reported by: Matthias Köppe Reviewer(s):
…nterface <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> - Deprecated in sagemath#33777 (2022). ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#37872 Reported by: Matthias Köppe Reviewer(s):
Sage's CHomP interface is out of date: the executables now have different names and may use a different syntax. Also, CHomP has not been updated recently — the most recent changes at https://github.com/marciogameiro/CHomP (see also http://chomp.rutgers.edu/software.html) are 5 years ago — and it doesn't build on OS X. I'm not sure about other platforms. It used to be the case that CHomP was necessary for certain calculations, such as finding generators of homology groups, but Sage can now do all of that on its own. So let's deprecate the CHomP interface to Sage.
Component: algebraic topology
Author: John Palmieri
Branch/Commit:
e0a9c87
Reviewer: Dima Pasechnik
Issue created by migration from https://trac.sagemath.org/ticket/33777
The text was updated successfully, but these errors were encountered: