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
determinant() of integer matrices of size in [51,63] broken #14032
Comments
comment:1
50 is the threshold for defaulting to |
comment:2
(never mind, forgot about caching) |
This comment has been minimized.
This comment has been minimized.
comment:4
There is an infinite recursion going on where we compute a determinant over |
This comment has been minimized.
This comment has been minimized.
Author: Jeroen Demeyer |
comment:6
Attachment: 14032_multimodular_det.patch.gz |
comment:7
Looks great! |
Reviewer: Volker Braun |
Merged: sage-5.7.beta2 |
The problem was an infinite recursion where we compute a determinant over ZZ by working mod p and we compute a determinant over GF(p) by lifting to ZZ...
This recursion got broken because the bound to use LinBox for matrices mod p changed in #12883. This patch fixes this bound.
It also does various clean-up of doctests and removes
_det_4x4_unsafe()
which only added an extra level of indirection (determinant() -> _det_4x4_unsafe() -> four_dim_det()
)Component: linear algebra
Author: Jeroen Demeyer
Reviewer: Volker Braun
Merged: sage-5.7.beta2
Issue created by migration from https://trac.sagemath.org/ticket/14032
The text was updated successfully, but these errors were encountered: