-
Notifications
You must be signed in to change notification settings - Fork 15
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
Remove waves and eke from MPAS-Analysis generate option #547
Remove waves and eke from MPAS-Analysis generate option #547
Conversation
Typical E3SMv3 WCYCL and CRYO configurations will not have the output necessary to run waves and eddy kinetic energy (EKE) analysis. It should be turned off to prevent confusing (though not fatal) errors in the setup stage of an MPAS-Analysis run.
@forsyth2, when you have a chance, could you suggest any testing I should do to verify that this fix actually works as expected? It should be sufficient to do one run of watercycle and another of cryo analysis on Chrysalis (possibly just with MPAS-Analysis enabled). Do you have any existing tests like that? |
@xylar Thanks for working on this.
|
@forsyth2, perfect! That more simplified option is what I was after as a starting point. The tip on updating |
@forsyth2, yep, those tests are still running but they worked exactly as expected. (Some tasks still fail during setup for the cryosphere configuration but that's because the run you use as a reference isn't really a cryosphere run, so some expected output is not available.) Do you want me to run the tests and update the expected files once the PR has been approved? Or do you prefer to do that yourself? |
If it's not too much trouble, that sounds good, thanks! I usually consider a Chrysalis run good enough for PRs (I run the test suite on Perlmutter and Compy before RCs). Note that there are a couple things that need fixing in the testing directions (#540). There also might be some permissions issues that pop up. If it proves to be too much of a hassle, let me know and I can run the tests myself. |
It looks a a short test has been done, but some expected files need to be updated. I will go ahead and merge for a new |
Typical E3SMv3 WCYCL and CRYO configurations will not have the output necessary to run waves and eddy kinetic energy (EKE) analysis. It should be turned off to prevent confusing (though not fatal) errors in the setup stage of an MPAS-Analysis run.