-
Notifications
You must be signed in to change notification settings - Fork 383
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
EAMxx: Update Compy's config file to enable test-all-scream #6982
base: master
Are you sure you want to change the base?
EAMxx: Update Compy's config file to enable test-all-scream #6982
Conversation
I can run the physics tests using this setup, but I get undefined reference to BLAS and LAPACK specific routines in HOMME if I try to generate baselines. I think we should merge this PR and I will address the remaining issues in a follow-up PR. |
Are you saying that |
I can run
I get the blas/lapack errors in the end. |
Unrelated to this PR: On Chrysalis, when I try to generate baselines with |
Edit: what I previously wrote is wrong. |
@singhbalwinder another thing: test-all-scream only runs baselines tests if you add |
Thanks, Luca for the suggestions.
Yes, that is right.
Yes, this is also true. I googled and found out that the problem might be with the I am changing this PR to draft as changing |
@singhbalwinder I closed the PR for the mkl change. In that PR, I suggested a fix to change mkl just for test-all-scream, which is what I think you wanted to do? If instead you want to update MKL for all use cases (including regular CIME runs), then please, reopen that PR, and we'll integrate it. |
Thanks, @bartgol for your suggestions . @rljacob has opened the other PR as nbfb only on Compy is not an issue. My other motivation to change mkl to 2020 was that mkl/2019u5 is buggy (see here under title "Intel Parallel Studio XE 2019 Update 5 MKL:"). Once that PR is in, I will make this PR available for more reviews and merging. |
Compy's config is updated to run test-all-scream script on Compy.