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

NVPL LAPACK Solver Support on ARM #701

Merged
merged 1 commit into from
Aug 9, 2024
Merged

NVPL LAPACK Solver Support on ARM #701

merged 1 commit into from
Aug 9, 2024

Conversation

aayushg55
Copy link
Collaborator

@aayushg55 aayushg55 commented Aug 8, 2024

  • NVPL LAPACK support for all decompositions on ARM

  • Simplified SVD interface to hide column-major logic

  • More comprehensive type and shape checks for all decompositions

  • Batched tests and complex tests for remaining decompositions

  • Fixed sign bug in determinant

  • Split solver into separate files

Misc:

  • Switched all HostExecutor references to const references

Notes:

  • The SVD and Eigen methods (gesvd and syevd) are currently not fully optimized in NVPL and give a warning

@aayushg55 aayushg55 marked this pull request as ready for review August 8, 2024 17:58
@cliffburdick
Copy link
Collaborator

/build

* NVPL LAPACK support for all decompositions on ARM

* Simplify SVD interface to hide column-major logic

* More comprehensive type and shape checks for all decompositions

* Batched tests and complex tests for remaining decompositions

* Fixed sign bug in determinant

* Split solver into separate files
@cliffburdick
Copy link
Collaborator

/build

2 similar comments
@cliffburdick
Copy link
Collaborator

/build

@cliffburdick
Copy link
Collaborator

/build

@cliffburdick cliffburdick merged commit 9070391 into main Aug 9, 2024
1 check failed
@cliffburdick cliffburdick deleted the nvpl_lapack branch August 9, 2024 03:34
@cliffburdick
Copy link
Collaborator

/build

1 similar comment
@cliffburdick
Copy link
Collaborator

/build

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

Successfully merging this pull request may close these issues.

2 participants