-
Notifications
You must be signed in to change notification settings - Fork 97
ci: Spack docker containers #3039
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
Merged
Merged
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
e6bb8fa
Update tag, add PySolver snippet, adjust lai setting
bmhan12 9670cad
Generalize umpire dependency
bmhan12 ca8d0b7
Add LAI interface toggle
bmhan12 185cf23
Update LvArray submodule
bmhan12 e0959d2
Check Trilinos config path before include
bmhan12 3062a3d
Remove uberenv submodule and uberenv/spack configuration files from G…
bmhan12 3d95e9e
Modify docs
bmhan12 6bc17e7
gcc@12 initialization warning-error
bmhan12 ec87301
Specify path to ATS python3 executable
bmhan12 143afc1
Update docs
bmhan12 ff7708d
more docs
bmhan12 3c94c03
Update docs with info about LC script
bmhan12 861477e
Sherlock changes; adjust hypre/trilinos defaults, make lai setting ex…
bmhan12 ba54444
Merge branch 'develop' into feature/han12/docker_spack
bmhan12 adf9690
Merge branch 'develop' into feature/han12/docker_spack
rrsettgast 9bbceb1
update LvArray submodule
rrsettgast d46fc15
disable sherlock unit tests
rrsettgast File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
why the need to specify both
ENABLE_TRILINOSandGEOS_LA_INTERFACE?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.
The current way the linear algebra packages are configured in the CI jobs is by setting the
ENABLE_HYPREandENABLE_TRILINOSvariables as environment variables:GEOS/.github/workflows/build_and_test.yml
Lines 199 to 207 in bd70903
Then reading those environment variables into CMake and choosing a
GEOS_LA_INTERFACEfrom there:GEOS/host-configs/environment.cmake
Lines 16 to 43 in bd70903
Instead of reading in environment variables from
environment.cmake, the spack Docker containers each generate a host-config (spack-generated.cmake) to use instead, where theGEOS_LA_INTERFACEis by default set to Hypre. The snippet added here is for overriding the spack host-config defaults by defining those variables through CMake (passing them toconfig-build.py) instead of reading them through environment variables. This is done for CI jobs that haveGEOS_LA_INTERFACEset to Trilinos.