Fix tests with singular 4.4.0p3 #38689
Conversation
|
Documentation preview for this PR (built with commit 84dd3e5; changes) is ready! 🎉 |
| sage: I.change_ring(P.change_ring(IntegerModRing(2 * 7))).groebner_basis() | ||
| [x + 9*y + 13*z, y^2 + 3*y, y*z + 7*y + 6, 2*y + 6, z^2 + 3, 2*z + 10] | ||
| [x + ..., y^2 + 3*y, y*z + 7*y + 6, 2*y + 6, z^2 + 3, 2*z + 10] |
There was a problem hiding this comment.
The text above this test says that it's checking that the basis is non-trivial, so we could probably fix this one forever by simply comparing it to the trivial basis and looking for False
There was a problem hiding this comment.
The list of leading monomials is deterministic so I think it's good to keep testing it, even though it's not the main goal of the test.
|
Ok, thanks, LGTM then. |
|
I now think that the "p" in singular version means "pre release" (i.e. "alpha" or "beta"; instead of "patch" as I used to think) since they keep breaking API and everything in these. IOW, I'm proposing that we (distros, including sage-the-distro) stick with 4.4.0 until either 4.4.1 or 4.5.0 is released. |
sagemathgh-38689: Fix tests with singular 4.4.0p3 Fuzz some tests to make then pass with singular>=4.4.0p3, which returns different Gröbner bases in some cases URL: sagemath#38689 Reported by: Antonio Rojas Reviewer(s): Antonio Rojas, Michael Orlitzky
4 participants
Fuzz some tests to make then pass with singular>=4.4.0p3, which returns different Gröbner bases in some cases