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.

Sign up for GitHub

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

Jump to bottom

optimize the sparse HNF, some startegy, but mostly gc friendly #1766

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

optimize the sparse HNF, some startegy, but mostly gc friendly #1766

wants to merge 3 commits into from
+779 −213

Conversation

fieker
Copy link
Collaborator

@fieker fieker commented Feb 21, 2025

  • add temporary scalar variables in SMat
  • use (and define) more ! functions
  • careful memory management in ZZRow
  • improve sub, transpose, ..
  • add strategy in HNF to alternate between full HNF and only produce some upper triangular matrix
  • better support for getting the trafo by appending the identity

@fieker
optimize the sparse HNF, some startegy, but mostly gc friendly
83e1afa 
 - add temporary scalar variables in SMat
 - use (and define) more ! functions
 - careful memory management in ZZRow
 - improve sub, transpose, ..
 - add strategy in HNF to alternate between full HNF and only produce
   some upper triangular matrix
 - better support for getting the trafo by appending the identity
@fieker
remove show
acac6ad
@thofma
Copy link
Owner

thofma commented Feb 22, 2025

there is some conflict

thofma
thofma reviewed Feb 23, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thofma thofma thofma left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fieker @thofma