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
Installing ChomP causes ChainComplex to return incorrect answers #16364
Comments
comment:2
I poked around a bit. In the first example, the issue seems to be that the wrapper is misparsing the correct output of "homchain". In the second example, "homchain" outputs the following abort message (due to integer overflow, I think) which is ignored by the wrapper. [Tech info: chain 24, addr 8, intgr 2.] |
comment:3
Possibly related: http://ask.sagemath.org/question/26550/make-ptest-best-way-to-fix-errors/ |
Upstream: Reported upstream. No feedback yet. |
comment:4
Several comments: first, there is a new version of CHomP: see https://github.com/shaunharker/CHomP, but it does not fix these problems (at least the second one: the first is a problem with Sage's interface with CHomP, so that's our fault). Second, because of the second problem, we obviously need to catch this kind of error to avoid producing nonsensical answers. (It would also be nice to fix CHomP. I'll email the developer.) |
comment:5
We probably should fix #12607 now that optional spkgs are always tested, and we can do a version bump of CHomP (along with making it a new-style spkg). |
comment:6
It would also be good to deal with #15202, but that's a more complicated project. |
comment:7
The first problem is solved by the branch at #12607, that needs review. |
Changed keywords from none to chomp |
comment:9
And the second looks indeed like an overflow error in a product inside CHomP. This works
How should we catch the errors of CHomP ? |
comment:10
I am taking care of the second problem also in ticket #12607 |
comment:12
Everything should be taken care of in #12607 |
Installing the ChomP package causes
ChainComplex
to give different (and basically wrong) answers. Here are two examples from the doctests:May be related to #12607 where installing ChomP causes certain 0 dimensional homology groups not to be reported.
Upstream: Reported upstream. No feedback yet.
CC: @jhpalmieri @kini
Component: packages: optional
Keywords: chomp
Issue created by migration from https://trac.sagemath.org/ticket/16364
The text was updated successfully, but these errors were encountered: