Conversation
climbfuji
left a comment
There was a problem hiding this comment.
It looks like you are only installing package ectrans?
Co-authored-by: Dom Heinzeller <[email protected]>
Co-authored-by: Dom Heinzeller <[email protected]>
@climbfuji Good question. I've asked, and @fmahebert told me majority of the work around this compiler is focused on I've started a dialog with @fmahebert and @l90lpa to get some more clarification. |
|
I pulled too much out for testing! I've added the original packages back into the spec and added |
|
This is more of a question for @fmahebert on requirements; Should we consider adding in Thoughts? At minimum it would be nice to get |
|
@eap I (selfishly) view this PR's goal to be fixing issues in the current nvhpc spack-stack environment that are blocking our short-term code demonstration goals. Thus, I would recommend against the scope expansion of adding any new packages to the build as part of this PR. To expand a bit — the demonstrations we're targeting will not involve skylab, workflows, or any of the soca/fv3 related code. Thus, adding the packages you've suggested is not immediately necessary, and risks introducing delays if we discover anything that doesn't build cleanly with nvhpc. In the longer term, it would certainly be nice to have more support for nvhpc across the system by working towards what you suggest. But I view that as a long-term project of building support (as and when warranted by JCSDA/partner needs), and beyond the scope of this PR. Does that sound reasonable? |
eap
left a comment
There was a problem hiding this comment.
@fmahebert > Absolutely reasonable, I just wanted to ask the question. Adding in ewok and other skylab dependencies would be a good followup but it looks like this is working as intended for the demo.
Summary
Update to the site and template for the
jedi-mpas-nvidia-devfeature.Testing
This has been done on an AWS GPU instance (g5.4xlarge) following instructions from @l90lpa.
Applications affected
This will only affect the JEDI MPAS Nvidia compiler work that @fmahebert completed.
Dependencies
N/A
Issue(s) addressed
Fixes #1563
Checklist