Skip to content
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

wrapper for determinant, minpoly, etc. from linbox for sparse matrices #13915

Closed
sagetrac-Bouillaguet mannequin opened this issue Jan 6, 2013 · 8 comments
Closed

wrapper for determinant, minpoly, etc. from linbox for sparse matrices #13915

sagetrac-Bouillaguet mannequin opened this issue Jan 6, 2013 · 8 comments

Comments

@sagetrac-Bouillaguet
Copy link
Mannequin

sagetrac-Bouillaguet mannequin commented Jan 6, 2013

Superseeded by #23214.


Currently, computing the determinant, minimum polynomial, characteristic polynomial, etc of sparse matrices over the integers and finite fields switch them to a dense representation, then runs the dense algorithm.

This works, but is quite suboptimal. Linbox packs a few algorithms tailored for sparse matrices, e.g. iterative methods for the determinant, minpoly, etc. These methods have the advantage that they only read the matrix, and are memory efficient.

The aim of this ticket is to make these algorithms available in Sage.

CC: @ClementPernet

Component: linear algebra

Keywords: linbox, sd75

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/13915

@sagetrac-Bouillaguet
Copy link
Mannequin Author

sagetrac-Bouillaguet mannequin commented Feb 4, 2013

comment:1

Examples of things that are presently bad (where a sparse matrix is converted to a dense representation) :

  • kernel, determinant of sparse integers matrices
  • rank of integers matrices relies on echelonization, which very likely switch it to dense. Linbox has other ways of dealing with this.
  • elementary divisors and smith form of sparse integer matrices (idem).

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@sagetrac-Bouillaguet
Copy link
Mannequin Author

sagetrac-Bouillaguet mannequin commented Aug 22, 2016

Changed keywords from linbox to linbox, sd75

@sagetrac-Bouillaguet sagetrac-Bouillaguet mannequin modified the milestones: sage-6.4, sage-7.4 Aug 22, 2016
@tscrim
Copy link
Collaborator

tscrim commented Apr 29, 2018

comment:8

See #25257 for an initial attempt for rank.

@tscrim tscrim modified the milestones: sage-7.4, sage-8.2 Apr 29, 2018
@videlec
Copy link
Contributor

videlec commented May 1, 2018

comment:10

superseeded by #23214. I propose to close this as duplicate

@videlec

This comment has been minimized.

@videlec videlec removed this from the sage-8.2 milestone May 1, 2018
@tscrim
Copy link
Collaborator

tscrim commented May 1, 2018

comment:11

Concur.

@tscrim
Copy link
Collaborator

tscrim commented May 1, 2018

Reviewer: Travis Scrimshaw

@videlec
Copy link
Contributor

videlec commented May 18, 2018

comment:13

closing positively reviewed duplicates

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants