Skip to content

Restructured documentation files#299

Merged
pelesh merged 5 commits intofrontmatter-devfrom
shaked/organize_docs
Jun 21, 2025
Merged

Restructured documentation files#299
pelesh merged 5 commits intofrontmatter-devfrom
shaked/organize_docs

Conversation

@shakedregev
Copy link
Copy Markdown
Collaborator

@shakedregev shakedregev commented Jun 4, 2025

Description

The goal of this PR is to make the documentation more user intuitive. It allows users and developers to find information under relevant titles.

Closes #298

See visualization.

@pelesh

Proposed changes

I restructured the docs to put user information first, and developer information second. Within these groups, information relevant to larger subset is given earlier.

Checklist

Only documentation. Tests irrelevant.

  • All tests pass. Code tested on
    • CPU backend
    • CUDA backend
    • HIP backend
  • Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • The new code follows Re::Solve style guidelines.
  • There are unit tests for the new code.
  • The new code is documented.
  • The feature branch is rebased with respect to the target branch.

@shakedregev shakedregev requested a review from pelesh June 4, 2025 22:47
@shakedregev shakedregev requested a review from adhamsi June 5, 2025 18:48
@shakedregev
Copy link
Copy Markdown
Collaborator Author

@adham-ibrahim7 take a look at this new version and say if you think it's more understandable and if there's anything missing.

Copy link
Copy Markdown
Collaborator

@adhamsi adhamsi left a comment

Choose a reason for hiding this comment

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

I think it would be nice to include explicitly in the User Guide the commands required to build on a CUDA/HIP machine. That is, all the ccmake steps that you showed me that I cannot now recall. Unless that's trivial.

What is the usage of cmake/ReSolveFindHipLibraries.cmake or the other files like it? That was not clear to me.

I saw #268 and I believe you mentioned it to me. I think the documentation should mention these build scripts, and for now, state that one is being composed for Frontier and will be included in a future update. It would be nice eventually to have a single list of commands copy-paste ready to build on Frontier.

@pelesh
Copy link
Copy Markdown
Collaborator

pelesh commented Jun 9, 2025

What is the usage of cmake/ReSolveFindHipLibraries.cmake or the other files like it? That was not clear to me.

This is explained here. Perhaps we could expand upon this a bit. Typically, user should not need to worry much about these files (for as long as there are no bugs in them 😄).

@pelesh
Copy link
Copy Markdown
Collaborator

pelesh commented Jun 9, 2025

I think it would be nice to include explicitly in the User Guide the commands required to build on a CUDA/HIP machine. That is, all the ccmake steps that you showed me that I cannot now recall. Unless that's trivial.

I saw #268 and I believe you mentioned it to me. I think the documentation should mention these build scripts, and for now, state that one is being composed for Frontier and will be included in a future update. It would be nice eventually to have a single list of commands copy-paste ready to build on Frontier.

I think CMake presets should be the way to go about this. Some instructions how to use ccmake or CMake graphical user interface would be helpful, as well.

@pelesh pelesh merged commit ec7f8cd into frontmatter-dev Jun 21, 2025
1 of 4 checks passed
pelesh added a commit that referenced this pull request Jun 21, 2025
---------

Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: Slaven Peles <peless@ornl.gov>
@pelesh pelesh deleted the shaked/organize_docs branch July 24, 2025 20:50
shakedregev added a commit that referenced this pull request Oct 3, 2025
---------

Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: Slaven Peles <peless@ornl.gov>
shakedregev added a commit that referenced this pull request Oct 3, 2025
* fix some more memory leaks and add ubsan support to `CMakeLists.txt` (#320)

* Fix bug in KLU class affecting non-symmetric linear systems (#323)

* Update Spack and setup CI with asan and ubsan variants (#322)

* Point Spack to upstream v0.23.1.

* Update Frontier modules after Spack upgrade.

* Frontier build with +asan+ubsan.

* Temporarily point Spack to nkoukpaizan-fork to add asan and ubsan.

* +asan+ubsan in GH Actions.

* CMake targets for asan and ubsan.

* ~asan~ubsan Spack builds on Frontier by default.

* Try running CI tests within spack build-env to address the issue of the consumer test not finding the appropriate compiler.

* Alias for spack environment in CI.

* Revert "Alias for spack environment in CI."

This reverts commit fd263ae.

* Using `${OLDPWD}` to identify Spack environment.

* Add warnings flags to CMake configuration. (#325)

---------

Co-authored-by: pelesh <peless@ornl.gov>

* fix the means by which cmake finds the suitesparse include dir (#327)

* Fixed GLU to work on non-symmetric matrices. 



Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Synchronize devices after HIP functions (#336)

* synchronize devices after HIP functions

* Apply pre-commmit fixes

* Update resolve/matrix/MatrixHandlerHip.cpp

* fixed CUDA

* fixed CUDA

* Apply pre-commmit fixes

* Update resolve/matrix/MatrixHandlerCuda.cpp

* Revert "Update resolve/matrix/MatrixHandlerCuda.cpp"

This reverts commit 30e41bb.

* Revert "Apply pre-commmit fixes"

This reverts commit 0bcd8f7.

* Revert "fixed CUDA"

This reverts commit 62623fa.

* Revert "fixed CUDA"

This reverts commit b9b249b.

---------

Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Added the ability to reset a workspace without completely destroying it. (#343)

* added method to reset workspace

* changed tests

* added reset workspace to CUDA

* now with comments

* Apply pre-commmit fixes

* Update resolve/workspace/LinAlgWorkspaceCpu.cpp

---------

Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Enforce F being 0 and remove duplicate extract factors code 

* added check for F!=0

* enforce F being 0 and clean up factor extraction code duplication

* Apply pre-commmit fixes

* Update resolve/LinSolverDirectKLU.cpp

* cleaned up comments

* Apply pre-commmit fixes

* Update resolve/LinSolverDirectKLU.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Properly address CSR and CSC discrepancies across all solvers. 

Fixed discrepancies with minimal overhead.

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: pelesh <peless@ornl.gov>

* Put sorting within KLU instead of within other solver logic. (#359)

* Adding asymmetric tests (from the data folder) for KLU, CuSolver, and RocSolver.  (#364)

* tests that are supposed to pass are passing

* fixed typos, removed unnecessary file

* Apply pre-commmit fixes

* renamed matrix files (not added before accidentally)

* removed commented out code

* fixed failing tests

* Apply pre-commmit fixes

* fixed commented out code

* straggling comment

* Update tests/functionality/CMakeLists.txt

* Update tests/functionality/CMakeLists.txt

* Update tests/functionality/CMakeLists.txt

* fixed data path for lusol

* consumer

* fixed typo

* changing to sys refactor, which requires a path

* Fix for resolve_consumer to take in the matrices and right hand sides (#365)

* Passing matrices and rhs to resolve_consumer.

* Minor typo cuda-->hip

* adding generic test matrices

* modified numbering to suffix

* tests failing with well conditioned matrix

* updating to better conditioned matrices

* SPD test passes consistently, others don't

* still testing

* changed matrices

* updated matrices

* changed test mats

* changed matrices

* symmetric test passing, asymmetric passing when run separately, but not through make test

* changed matrices

* changed tolerance to machine precision, tests pass!

* Apply pre-commmit fixes

* updated consumer

* fixed non IR related comments

* removed IR check, it passes with IR with the better scaled matrices

* Apply pre-commmit fixes

* Update tests/functionality/CMakeLists.txt

* tests passing

* Apply pre-commmit fixes

* fixed prints

* set convergence conditions

* setting convergence condition, but it's overridden

* set default to relative residual

* added default conv_cond

* Apply pre-commmit fixes

* Update examples/resolve_consumer/CMakeLists.txt

* addressed comment

* reverted change that made CI test not pass

* made matrices consistent, tolerance too stringent

* tests passing, renamed files to be consistent

* addressed all comments, only tested on Frontier

* addressed comments

* removed straggling print

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>

* Shaked/rework examples csr (#367)

* fixed examples

* kalmarek example

* moved to experimental, kalmarek example works

* removed commented code

* Apply pre-commmit fixes

* Update examples/experimental/CMakeLists.txt

* addressed change requests

* changed cmake

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Fixed experimental examples to use CSR only.

* removed unnecessary Csc reference from examples

* removed build files

* fixed segfault

* fixed glu update

* fixed all experimental examples

* somehow this segmentation fault fix didn't save

* fixed fgmres

* Apply pre-commmit fixes

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Fix warning on r_KLU_rocsolverrf_asym6x6 

* Fix warning on r_KLU_rocsolverrf_asym6x6.

---------

Co-authored-by: nkoukpaizan <nkoukpaizan@users.noreply.github.com>
Co-authored-by: Shaked Regev <35384901+shakedregev@users.noreply.github.com>

* Removed all CSC code from CSR solvers. (#375)

* removed all CSC code

* Apply pre-commmit fixes

* Update tests/functionality/testRefactor.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Removed rocSparse "fast" solver from our options (#377)

Removed all "fast" solver code.

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Updated changelog and PR template (#381)

* Update emails in README.md

* Add code of conduct.

* Move code of conduct.

* Add badges

* Change action name to CPU Tests

* Update README.md

* Add logo to the README file.

* Add logo to readthedocs.

* Manage images for light/dark schemes in README.

* Add logo to Doxygen docs.

* Fix path to logo in Doxygen.

* Update logos

* Add svg logo images.

* Update README.md

* Update doxygen landing page.

* Monor changes to Doxygen landing page

* Updated coding and developer guidelines (#242)

Added details about implicit coding guidelines that were not specified.
Added suggested branching workflow.
Added PR reviewing and opening and issue guidelines.
Fixed typos
Restructured documentation sections 
Co-authored-by: pelesh <peless@ornl.gov>

* Update readthedocs links in README.md

Updated links in README to pint to readthedocs pages generated from `frontmatter-dev` branch. This needs to be reverted before merging.

* documentation clarification (#267)

* fixing typos

* fixed typos

* Restructured documentation files (#299)

---------

Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: Slaven Peles <peless@ornl.gov>

* Apply pre-commmit fixes

* Use Re::Solve instead of ReSolve and update Doxygen logo.

* Simplify doxygen stylesheet, allow dark mode

* [skip ci] Revert sidebar-only mode.

* [skip ci] Another shot on sidebar only doxy theme

* ReSolve -> Re::Solve

* Apply pre-commmit fixes

* Copy Doxyfile.in -> Doxyfile and then modify.

* updated contributors

* straggling changes

* Use correct Doxyfile

* Undo doxyfile template changes

* rebased frontmatter-dev

---------

Co-authored-by: superwhiskers <whiskerdev@protonmail.com>
Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>
Co-authored-by: pelesh <peless@ornl.gov>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: nkoukpaizan <nkoukpaizan@users.noreply.github.com>
Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: pelesh <pelesh@users.noreply.github.com>
shakedregev added a commit that referenced this pull request Oct 3, 2025
---------

Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: Slaven Peles <peless@ornl.gov>
shakedregev added a commit that referenced this pull request Oct 3, 2025
* fix some more memory leaks and add ubsan support to `CMakeLists.txt` (#320)

* Fix bug in KLU class affecting non-symmetric linear systems (#323)

* Update Spack and setup CI with asan and ubsan variants (#322)

* Point Spack to upstream v0.23.1.

* Update Frontier modules after Spack upgrade.

* Frontier build with +asan+ubsan.

* Temporarily point Spack to nkoukpaizan-fork to add asan and ubsan.

* +asan+ubsan in GH Actions.

* CMake targets for asan and ubsan.

* ~asan~ubsan Spack builds on Frontier by default.

* Try running CI tests within spack build-env to address the issue of the consumer test not finding the appropriate compiler.

* Alias for spack environment in CI.

* Revert "Alias for spack environment in CI."

This reverts commit fd263ae.

* Using `${OLDPWD}` to identify Spack environment.

* Add warnings flags to CMake configuration. (#325)

---------

Co-authored-by: pelesh <peless@ornl.gov>

* fix the means by which cmake finds the suitesparse include dir (#327)

* Fixed GLU to work on non-symmetric matrices.

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Synchronize devices after HIP functions (#336)

* synchronize devices after HIP functions

* Apply pre-commmit fixes

* Update resolve/matrix/MatrixHandlerHip.cpp

* fixed CUDA

* fixed CUDA

* Apply pre-commmit fixes

* Update resolve/matrix/MatrixHandlerCuda.cpp

* Revert "Update resolve/matrix/MatrixHandlerCuda.cpp"

This reverts commit 30e41bb.

* Revert "Apply pre-commmit fixes"

This reverts commit 0bcd8f7.

* Revert "fixed CUDA"

This reverts commit 62623fa.

* Revert "fixed CUDA"

This reverts commit b9b249b.

---------

Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Added the ability to reset a workspace without completely destroying it. (#343)

* added method to reset workspace

* changed tests

* added reset workspace to CUDA

* now with comments

* Apply pre-commmit fixes

* Update resolve/workspace/LinAlgWorkspaceCpu.cpp

---------

Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Enforce F being 0 and remove duplicate extract factors code

* added check for F!=0

* enforce F being 0 and clean up factor extraction code duplication

* Apply pre-commmit fixes

* Update resolve/LinSolverDirectKLU.cpp

* cleaned up comments

* Apply pre-commmit fixes

* Update resolve/LinSolverDirectKLU.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Properly address CSR and CSC discrepancies across all solvers.

Fixed discrepancies with minimal overhead.

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: pelesh <peless@ornl.gov>

* Put sorting within KLU instead of within other solver logic. (#359)

* Adding asymmetric tests (from the data folder) for KLU, CuSolver, and RocSolver.  (#364)

* tests that are supposed to pass are passing

* fixed typos, removed unnecessary file

* Apply pre-commmit fixes

* renamed matrix files (not added before accidentally)

* removed commented out code

* fixed failing tests

* Apply pre-commmit fixes

* fixed commented out code

* straggling comment

* Update tests/functionality/CMakeLists.txt

* Update tests/functionality/CMakeLists.txt

* Update tests/functionality/CMakeLists.txt

* fixed data path for lusol

* consumer

* fixed typo

* changing to sys refactor, which requires a path

* Fix for resolve_consumer to take in the matrices and right hand sides (#365)

* Passing matrices and rhs to resolve_consumer.

* Minor typo cuda-->hip

* adding generic test matrices

* modified numbering to suffix

* tests failing with well conditioned matrix

* updating to better conditioned matrices

* SPD test passes consistently, others don't

* still testing

* changed matrices

* updated matrices

* changed test mats

* changed matrices

* symmetric test passing, asymmetric passing when run separately, but not through make test

* changed matrices

* changed tolerance to machine precision, tests pass!

* Apply pre-commmit fixes

* updated consumer

* fixed non IR related comments

* removed IR check, it passes with IR with the better scaled matrices

* Apply pre-commmit fixes

* Update tests/functionality/CMakeLists.txt

* tests passing

* Apply pre-commmit fixes

* fixed prints

* set convergence conditions

* setting convergence condition, but it's overridden

* set default to relative residual

* added default conv_cond

* Apply pre-commmit fixes

* Update examples/resolve_consumer/CMakeLists.txt

* addressed comment

* reverted change that made CI test not pass

* made matrices consistent, tolerance too stringent

* tests passing, renamed files to be consistent

* addressed all comments, only tested on Frontier

* addressed comments

* removed straggling print

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>

* Shaked/rework examples csr (#367)

* fixed examples

* kalmarek example

* moved to experimental, kalmarek example works

* removed commented code

* Apply pre-commmit fixes

* Update examples/experimental/CMakeLists.txt

* addressed change requests

* changed cmake

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Fixed experimental examples to use CSR only.

* removed unnecessary Csc reference from examples

* removed build files

* fixed segfault

* fixed glu update

* fixed all experimental examples

* somehow this segmentation fault fix didn't save

* fixed fgmres

* Apply pre-commmit fixes

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Fix warning on r_KLU_rocsolverrf_asym6x6

* Fix warning on r_KLU_rocsolverrf_asym6x6.

---------

Co-authored-by: nkoukpaizan <nkoukpaizan@users.noreply.github.com>
Co-authored-by: Shaked Regev <35384901+shakedregev@users.noreply.github.com>

* Removed all CSC code from CSR solvers. (#375)

* removed all CSC code

* Apply pre-commmit fixes

* Update tests/functionality/testRefactor.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Removed rocSparse "fast" solver from our options (#377)

Removed all "fast" solver code.

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Updated changelog and PR template (#381)

* Update emails in README.md

* Add code of conduct.

* Move code of conduct.

* Add badges

* Change action name to CPU Tests

* Update README.md

* Add logo to the README file.

* Add logo to readthedocs.

* Manage images for light/dark schemes in README.

* Add logo to Doxygen docs.

* Fix path to logo in Doxygen.

* Update logos

* Add svg logo images.

* Update README.md

* Update doxygen landing page.

* Monor changes to Doxygen landing page

* Updated coding and developer guidelines (#242)

Added details about implicit coding guidelines that were not specified.
Added suggested branching workflow.
Added PR reviewing and opening and issue guidelines.
Fixed typos
Restructured documentation sections
Co-authored-by: pelesh <peless@ornl.gov>

* Update readthedocs links in README.md

Updated links in README to pint to readthedocs pages generated from `frontmatter-dev` branch. This needs to be reverted before merging.

* documentation clarification (#267)

* fixing typos

* fixed typos

* Restructured documentation files (#299)

---------

Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: Slaven Peles <peless@ornl.gov>

* Apply pre-commmit fixes

* Use Re::Solve instead of ReSolve and update Doxygen logo.

* Simplify doxygen stylesheet, allow dark mode

* [skip ci] Revert sidebar-only mode.

* [skip ci] Another shot on sidebar only doxy theme

* ReSolve -> Re::Solve

* Apply pre-commmit fixes

* Copy Doxyfile.in -> Doxyfile and then modify.

* updated contributors

* straggling changes

* Use correct Doxyfile

* Undo doxyfile template changes

* rebased frontmatter-dev

---------

Co-authored-by: superwhiskers <whiskerdev@protonmail.com>
Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>
Co-authored-by: pelesh <peless@ornl.gov>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: nkoukpaizan <nkoukpaizan@users.noreply.github.com>
Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: pelesh <pelesh@users.noreply.github.com>
pelesh added a commit that referenced this pull request Oct 7, 2025
---------

Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: Slaven Peles <peless@ornl.gov>
pelesh added a commit that referenced this pull request Oct 7, 2025
* fix some more memory leaks and add ubsan support to `CMakeLists.txt` (#320)

* Fix bug in KLU class affecting non-symmetric linear systems (#323)

* Update Spack and setup CI with asan and ubsan variants (#322)

* Point Spack to upstream v0.23.1.

* Update Frontier modules after Spack upgrade.

* Frontier build with +asan+ubsan.

* Temporarily point Spack to nkoukpaizan-fork to add asan and ubsan.

* +asan+ubsan in GH Actions.

* CMake targets for asan and ubsan.

* ~asan~ubsan Spack builds on Frontier by default.

* Try running CI tests within spack build-env to address the issue of the consumer test not finding the appropriate compiler.

* Alias for spack environment in CI.

* Revert "Alias for spack environment in CI."

This reverts commit fd263ae.

* Using `${OLDPWD}` to identify Spack environment.

* Add warnings flags to CMake configuration. (#325)

---------

Co-authored-by: pelesh <peless@ornl.gov>

* fix the means by which cmake finds the suitesparse include dir (#327)

* Fixed GLU to work on non-symmetric matrices.

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Synchronize devices after HIP functions (#336)

* synchronize devices after HIP functions

* Apply pre-commmit fixes

* Update resolve/matrix/MatrixHandlerHip.cpp

* fixed CUDA

* fixed CUDA

* Apply pre-commmit fixes

* Update resolve/matrix/MatrixHandlerCuda.cpp

* Revert "Update resolve/matrix/MatrixHandlerCuda.cpp"

This reverts commit 30e41bb.

* Revert "Apply pre-commmit fixes"

This reverts commit 0bcd8f7.

* Revert "fixed CUDA"

This reverts commit 62623fa.

* Revert "fixed CUDA"

This reverts commit b9b249b.

---------

Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Added the ability to reset a workspace without completely destroying it. (#343)

* added method to reset workspace

* changed tests

* added reset workspace to CUDA

* now with comments

* Apply pre-commmit fixes

* Update resolve/workspace/LinAlgWorkspaceCpu.cpp

---------

Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Enforce F being 0 and remove duplicate extract factors code

* added check for F!=0

* enforce F being 0 and clean up factor extraction code duplication

* Apply pre-commmit fixes

* Update resolve/LinSolverDirectKLU.cpp

* cleaned up comments

* Apply pre-commmit fixes

* Update resolve/LinSolverDirectKLU.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Properly address CSR and CSC discrepancies across all solvers.

Fixed discrepancies with minimal overhead.

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: pelesh <peless@ornl.gov>

* Put sorting within KLU instead of within other solver logic. (#359)

* Adding asymmetric tests (from the data folder) for KLU, CuSolver, and RocSolver.  (#364)

* tests that are supposed to pass are passing

* fixed typos, removed unnecessary file

* Apply pre-commmit fixes

* renamed matrix files (not added before accidentally)

* removed commented out code

* fixed failing tests

* Apply pre-commmit fixes

* fixed commented out code

* straggling comment

* Update tests/functionality/CMakeLists.txt

* Update tests/functionality/CMakeLists.txt

* Update tests/functionality/CMakeLists.txt

* fixed data path for lusol

* consumer

* fixed typo

* changing to sys refactor, which requires a path

* Fix for resolve_consumer to take in the matrices and right hand sides (#365)

* Passing matrices and rhs to resolve_consumer.

* Minor typo cuda-->hip

* adding generic test matrices

* modified numbering to suffix

* tests failing with well conditioned matrix

* updating to better conditioned matrices

* SPD test passes consistently, others don't

* still testing

* changed matrices

* updated matrices

* changed test mats

* changed matrices

* symmetric test passing, asymmetric passing when run separately, but not through make test

* changed matrices

* changed tolerance to machine precision, tests pass!

* Apply pre-commmit fixes

* updated consumer

* fixed non IR related comments

* removed IR check, it passes with IR with the better scaled matrices

* Apply pre-commmit fixes

* Update tests/functionality/CMakeLists.txt

* tests passing

* Apply pre-commmit fixes

* fixed prints

* set convergence conditions

* setting convergence condition, but it's overridden

* set default to relative residual

* added default conv_cond

* Apply pre-commmit fixes

* Update examples/resolve_consumer/CMakeLists.txt

* addressed comment

* reverted change that made CI test not pass

* made matrices consistent, tolerance too stringent

* tests passing, renamed files to be consistent

* addressed all comments, only tested on Frontier

* addressed comments

* removed straggling print

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>

* Shaked/rework examples csr (#367)

* fixed examples

* kalmarek example

* moved to experimental, kalmarek example works

* removed commented code

* Apply pre-commmit fixes

* Update examples/experimental/CMakeLists.txt

* addressed change requests

* changed cmake

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Fixed experimental examples to use CSR only.

* removed unnecessary Csc reference from examples

* removed build files

* fixed segfault

* fixed glu update

* fixed all experimental examples

* somehow this segmentation fault fix didn't save

* fixed fgmres

* Apply pre-commmit fixes

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

* Update examples/experimental/r_KLU_cusolverrf_redo_factorization.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Fix warning on r_KLU_rocsolverrf_asym6x6

* Fix warning on r_KLU_rocsolverrf_asym6x6.

---------

Co-authored-by: nkoukpaizan <nkoukpaizan@users.noreply.github.com>
Co-authored-by: Shaked Regev <35384901+shakedregev@users.noreply.github.com>

* Removed all CSC code from CSR solvers. (#375)

* removed all CSC code

* Apply pre-commmit fixes

* Update tests/functionality/testRefactor.cpp

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Removed rocSparse "fast" solver from our options (#377)

Removed all "fast" solver code.

---------

Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>

* Updated changelog and PR template (#381)

* Update emails in README.md

* Add code of conduct.

* Move code of conduct.

* Add badges

* Change action name to CPU Tests

* Update README.md

* Add logo to the README file.

* Add logo to readthedocs.

* Manage images for light/dark schemes in README.

* Add logo to Doxygen docs.

* Fix path to logo in Doxygen.

* Update logos

* Add svg logo images.

* Update README.md

* Update doxygen landing page.

* Monor changes to Doxygen landing page

* Updated coding and developer guidelines (#242)

Added details about implicit coding guidelines that were not specified.
Added suggested branching workflow.
Added PR reviewing and opening and issue guidelines.
Fixed typos
Restructured documentation sections
Co-authored-by: pelesh <peless@ornl.gov>

* Update readthedocs links in README.md

Updated links in README to pint to readthedocs pages generated from `frontmatter-dev` branch. This needs to be reverted before merging.

* documentation clarification (#267)

* fixing typos

* fixed typos

* Restructured documentation files (#299)

---------

Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: Slaven Peles <peless@ornl.gov>

* Apply pre-commmit fixes

* Use Re::Solve instead of ReSolve and update Doxygen logo.

* Simplify doxygen stylesheet, allow dark mode

* [skip ci] Revert sidebar-only mode.

* [skip ci] Another shot on sidebar only doxy theme

* ReSolve -> Re::Solve

* Apply pre-commmit fixes

* Copy Doxyfile.in -> Doxyfile and then modify.

* updated contributors

* straggling changes

* Use correct Doxyfile

* Undo doxyfile template changes

* rebased frontmatter-dev

---------

Co-authored-by: superwhiskers <whiskerdev@protonmail.com>
Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>
Co-authored-by: pelesh <peless@ornl.gov>
Co-authored-by: shakedregev <shakedregev@users.noreply.github.com>
Co-authored-by: Shaked Regev <shakedvregev@gmail.comregev@gmail.com>
Co-authored-by: nkoukpaizan <nkoukpaizan@users.noreply.github.com>
Co-authored-by: Adham Ibrahim <37982706+adham-ibrahim7@users.noreply.github.com>
Co-authored-by: pelesh <pelesh@users.noreply.github.com>
pelesh added a commit that referenced this pull request Oct 7, 2025
* Update emails in README.md

* Add code of conduct.

* Add badges

* Change action name to CPU Tests

* Update README.md

* Add logo to the README file.

* Add logo to readthedocs.

* Manage images for light/dark schemes in README.

* Add logo to Doxygen docs.

* Update doxygen landing page.

* Monor changes to Doxygen landing page

* Updated coding and developer guidelines (#242)

* Restructured documentation files (#299)

* ReSolve -> Re::Solve


---------

Co-authored-by: shakedregev <shakedvregev@gmail.com>
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.

3 participants