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

Fix bug when data/values ownership is incorrectly assigned. #124

Merged
merged 4 commits into from
Dec 18, 2023

Conversation

pelesh
Copy link
Collaborator

@pelesh pelesh commented Dec 16, 2023

Closes #123

@pelesh pelesh added the bug Something isn't working label Dec 16, 2023
@pelesh pelesh added this to the First Release milestone Dec 16, 2023
@pelesh pelesh requested a review from kswirydo December 16, 2023 19:33
@pelesh pelesh self-assigned this Dec 16, 2023
@pelesh pelesh force-pushed the system-solver-update-dev branch from 9d489e1 to 86e232a Compare December 16, 2023 22:16
Copy link
Collaborator

@kswirydo kswirydo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments.

resolve/SystemSolver.cpp Outdated Show resolved Hide resolved
resolve/matrix/Coo.cpp Show resolved Hide resolved
resolve/matrix/Coo.cpp Show resolved Hide resolved
resolve/matrix/Coo.cpp Show resolved Hide resolved
resolve/matrix/Csc.cpp Show resolved Hide resolved
tests/functionality/testKLU_GLU.cpp Outdated Show resolved Hide resolved
tests/functionality/testKLU_RocSolver_FGMRES.cpp Outdated Show resolved Hide resolved
tests/functionality/testKLU_RocSolver_FGMRES.cpp Outdated Show resolved Hide resolved
author Slaven Peles <[email protected]> 1702759728 -0500
committer Slaven Peles <[email protected]> 1702871403 -0500

parent ec87c03b1f29997940e23f24a7b81230d8aaadd1
author Slaven Peles <[email protected]> 1702759728 -0500
committer Slaven Peles <[email protected]> 1702871274 -0500

Add more diagnostics to Csr matrix.
@pelesh pelesh force-pushed the linalg-ownership-fix branch from 257ef6d to 7fa6682 Compare December 18, 2023 04:11
@kswirydo kswirydo merged commit 4ee92c9 into system-solver-update-dev Dec 18, 2023
9 checks passed
pelesh added a commit that referenced this pull request Dec 20, 2023
* Fix bug when data/values ownership is incorrectly assigned.

* parent ec87c03b1f29997940e23f24a7b81230d8aaadd1
author Slaven Peles <[email protected]> 1702759728 -0500
committer Slaven Peles <[email protected]> 1702871403 -0500

parent ec87c03b1f29997940e23f24a7b81230d8aaadd1
author Slaven Peles <[email protected]> 1702759728 -0500
committer Slaven Peles <[email protected]> 1702871274 -0500

Add more diagnostics to Csr matrix.

* Addressing comments in the PR.

* Fix broken build, add logger to Csc and Coo classes.
pelesh added a commit that referenced this pull request Dec 20, 2023
* Refactor residual norm getter in SystemSolver.

* Implement and test NSR computation in SystemSolver.

* Have iterative refinement inside the solve call.

* System solver example that works with CUDA and HIP.

* Fix cmake presets for Deception.

* Fix SystemSolver running GLU test.

* Use memspace as parameter consistently in FGMRES classes. (#125)

* Fix bug when data/values ownership is incorrectly assigned. (#124)

* Fix broken CPU build, add logger to Csc and Coo classes.

* Have fgmres and gs set memory space at runtime based on vector handler.

* Randomized solver called through SystemSolver class (#128)

* Support for iterative methods in SystemSolver.

* Working example of a randomized solver.

* Use more consistent naming scheme for functionality tests.

* Fix issue with building SystemSolver on CPU.
@pelesh pelesh deleted the linalg-ownership-fix branch December 21, 2023 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants