-
Notifications
You must be signed in to change notification settings - Fork 40
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
Condensed/normaleqn linear system without CUDA/RAJA #557
Conversation
I think this is kosher. In the PR diff, I do not see how/if MA57 and/or xdycyd gets picked up when RAJA/HIOP_SPARSE is not present in the build... but I guess that logic is already in place |
@@ -40,10 +42,6 @@ set(hiopOptimization_INTERFACE_HEADERS | |||
hiopResidual.hpp | |||
) | |||
|
|||
if(HIOP_USE_RAJA) | |||
list(APPEND hiopOptimization_SPARSE_SRC hiopKKTLinSysSparseCondensed.cpp hiopKKTLinSysSparseNormalEqn.cpp) | |||
endif() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me this change leads to error: 'hiopVectorRajaPar' does not name a type
, e.g. in line 175 of src/Optimization/hiopKKTLinSysSparseCondensed.cpp when building with Ginkgo and CUDA but without RAJA.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! I was having a similar concern re: building w/o RAJA. Nai-Yuan and I will meet today and will address this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the issue has been fixed, it would be great if you could confirm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I can still build without RAJA but with Ginkgo and Cuda enabled.
Condensed/normaleqn linear system should work on CPU, using MA57 as the backend linear solver.
In the current
develop
branch, PR #549 changed the CI behaviors when neither RAJA nor CUDA is defined, i.e., hiop will switchCondensed/normaleqn
toxycyd
and solves the problem without any warning. In addition, #549 compiles the cpp files for condensed/normaleqn system only if raja is defined (see here).Since condensed/normaleqn system should work without raja, I correct the above issues in this PR.
This PR also guarantees the feature @fritzgoebel introduced in #549, i.e., enable cuda build without raja.
I believe now we can compile hiop with cuda and ginkgo, but without raja.