-
-
Notifications
You must be signed in to change notification settings - Fork 489
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
Enhanced linbox interface #22872
Labels
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
comment:3
I just noticed that there was a previous attempt to clean the interface in #21327 by C. Bouillaguet. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Changed keywords from none to linbox |
comment:21
update milestone 8.3 -> 8.4 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This was referenced May 14, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In
sage/libs/linbox/linbox.pxd
we currently have two classes that gives an interface for dense integer matrix (Linbox_integer_dense
) and sparse matrix over Z/nZ (Linbox_modn_sparse
). These interfaces are then used in different matrix implementations insage/matrix/
.This task ticket stands for improvement of the LinBox interface
sage/matrix
without the intermediate Cython classes. In particular, avoid calling the linbox-sage interface (on LinBox side) but directly invoque LinBox C++ code (as Cython now allows it) (done in cleaning of linbox for dense integer matrices #22924 and cleaning linbox declarations + reimplement modn interface #24544)A X = B
withB
a matrix without repeated callto LinBox column by column
spkg-install
with a reference to #17635 comment 67spkg-install
with a reference to update fplll to version 5.x #21221Concrete ticket list
matrix_integer_dense.pyx
element_givaro.pyx
fmpq_mat_t
for rational dense matricesunsigned int
andint64_t
behave very differently!CC: @ClementPernet @JohnCremona @sagetrac-dlucas
Component: interfaces
Keywords: linbox
Issue created by migration from https://trac.sagemath.org/ticket/22872
The text was updated successfully, but these errors were encountered: