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

Unifying modn matrices #19076

Open
sagetrac-kartikv mannequin opened this issue Aug 24, 2015 · 2 comments
Open

Unifying modn matrices #19076

sagetrac-kartikv mannequin opened this issue Aug 24, 2015 · 2 comments

Comments

@sagetrac-kartikv
Copy link
Mannequin

sagetrac-kartikv mannequin commented Aug 24, 2015

This is a consolidation of several matrix related tickets, e.g. #2705, #10734, #11161, #12177, #12679, #13177, #13915, #15104, ... basically, the world of matrices over finite fields. The fundamental issues are the following

  • there is inconsistency over the size of fields that are allowed to use the "fast" matrix methods. Specifically, some places use mod_int and either a int_fast64_t or a long, while other places use int or int32_t specifically. Moreover, the size of the maximum modulus differs even when the sizes of the representations are the same.

  • because of this, there are many cases when fast methods could be used but aren't: the code generally just reverts to generic matrices and slow internal operations

  • the code totally ignores linbox's support for GMP-based matrix operations, which are not quite as fast as word-size ones but are still better than Sage internal operations

I intend to build a consolidated class framework for modular vectors and matrices, with system-optimized choice of crossover between small and large moduli and uniform use of linbox throughout. As part of this, I also intent to consolidate the existing code and update it to modern standards, e.g. move pxi files to pxd, add kwargs to all overloadable functions, etc.

CC: @kedlaya

Component: linear algebra

Keywords: matrix, sparse, modn, int

Author: Kartik Venkatram

Reviewer: rbeezer, jason

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

@sagetrac-kartikv sagetrac-kartikv mannequin added this to the sage-6.9 milestone Aug 24, 2015
@sagetrac-kartikv
Copy link
Mannequin Author

sagetrac-kartikv mannequin commented Aug 24, 2015

Reviewer: rbeezer, jason

@kedlaya
Copy link
Contributor

kedlaya commented Oct 25, 2017

comment:3

Another relevant ticket: #9888.

@mkoeppe mkoeppe removed this from the sage-6.9 milestone Dec 29, 2022
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

2 participants