-
Notifications
You must be signed in to change notification settings - Fork 47
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
Site config updates and bugfixes for NRL systems (Atlantis, Nautilus) #1449
Merged
Conversation
This file contains 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
3d45023
to
0996876
Compare
…s for Intel/OneAPI, add missing slurm module for GNU OpenMPI
…uplicate packages
d1cd1c7
to
ef2e1ec
Compare
…ix_libsci.sh after environment builds
climbfuji
commented
Jan 16, 2025
AlexanderRichert-NOAA
approved these changes
Jan 16, 2025
areinecke
approved these changes
Jan 16, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
General
~libbsd
forexpat
package to avoid duplicate packages when building unified environment on NarwhalAtlantis
Update
compilers.yaml
andpackages_gcc.yaml
: remove unneeded environment modifications, add missing module dependencies.Narwhal
Fixing the underlying issue for the wrong/old/not-even-default
libsci
libraries being linked by several packages is not straightforward and will require the HPE Cray admins to make modifications. Therefore, this PR provides a utility scriptutil/narwhal/fix_libsci.sh
as a workaround. This script needs to be run after each successful spack-stack environment build.Nautilus
Revert
gcc
from12.2.1
to11.2.1
to address a known problem with the version ofld
frombinutils
that ships withgcc@12
. This particular version leads to compilation errors of downstream packages (e.g.py-pandas
) when usingopenblas
as provider forblas
,lapack
,fftw-api
in spack-stack. See https://github.com/OpenMathLib/OpenBLAS/wiki/Faq#ELFoffset for more information.Testing
gcc
,intel
,oneapi
,oneapi-ifx
, ran NEPTUNE with these stacksoneapi
+libsci/23
(default islibsci/22
, the offending version in `/opt/cray, ran NEPTUNE with this stackgcc
+openblas
, ran NEPTUNE with this stack(NRL colleagues, please see https://github.nrlmry.navy.mil/NEPTUNE/neptune-atmos/pull/1096 for a testing-only PR (not to be merged) for these changes)
Applications affected
NEPTUNE and JEDI-NEPTUNE on Atlantis, Nautilus, Narwhal
Systems affected
Atlantis, Nautilus, Narwhal
Dependencies
None
Issue(s) addressed
Atlantis
Resolves #1445
Narwhal
Working towards #1447 (this is not a full solution)
Nautilus
Resolves #1446
Checklist
All dependency PRs/issues have been resolved and this PR can be merged.