-
-
Notifications
You must be signed in to change notification settings - Fork 565
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
add minimal interface for using ZZ[x]-matrices from flint #29894
Comments
Branch: u/chapoton/29894 |
Commit: |
New commits:
|
comment:2
Could we add at least one little direct use for this interface, say in computing the determinants as mentioned in the ticket description? |
comment:3
ok, I will try to think about it. I had some use case, but it turned out to be slower than the existing method. |
comment:4
That is interesting and surprising to me. Bill probably would like to know about that. The flint version might also be asymptotically faster. Plus, I am generally a fan of having multiple implementations available and giving the choice to the user (if for nothing else other than testing). |
comment:5
My test case was (for M upper triangular square matrix with 1 on the diagonal)
versus
and the inversion + product + charpoly over ZZ was much faster than the det over ZZ[x] |
comment:6
maybe useful for Alexander polynomial from Seifert matrix, see #29952 |
comment:7
Perhaps that is just a special case of the fact it is a upper unitriangular matrix? Anyways, I am not going to fret over not having a definitive use case at this time. It doesn't hurt anything to have the interface all setup. |
Reviewer: Travis Scrimshaw |
Changed branch from u/chapoton/29894 to |
So that one may use this later to compute determinants in this case, for example.
CC: @videlec
Component: interfaces
Author: Frédéric Chapoton
Branch/Commit:
1f52255
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/29894
The text was updated successfully, but these errors were encountered: