-
Notifications
You must be signed in to change notification settings - Fork 632
Update CMAKE so that OASIS requires MPI #1490
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
Update CMAKE so that OASIS requires MPI #1490
Conversation
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.
I approve this PR, however I had an error with ww3_tc1 with a missing file "ounfmeta.inp" in ww3_tc1/input
Could you add it even if it's not linked to this PR. I've checked your matrix output files and it seems you are not running this regtest. Is there a reason why you didn't run it ?
|
@mickaelaccensi - Just submit a different PR since it's not related. |
I've created an issue #1492 |
|
It didn't change from before on our system? I didn't see any errors - but we'll re-run tests and double check. |
|
@mickaelaccensi I've checked my output logs, and I don't get any error messages, etc? But we'll follow up in the other issue on further testing. Thanks for reviewing this Pr. |
|
I tested on Hercules-Intel; no errors were observed in the log messages, and the matrix comparison results were consistent. Expected differences due to different switches in ww3_tp2.14 tests: The The matrix comparison summary: |
Pull Request Summary
Updates cmake related items so that MPI is required for OASIS.
Description
This is a minor update for cmake and regtests reuiqring MPI for OASIS and other similar updates.
Issue(s) addressed
Closes #1486
Commit Message
Update CMAKE so that OASIS requires MPI
Check list
Testing
The ww3_tp2.14 tests change because the netcdf file will have different switches. The contents otherwise is the same.
matrixCompFull.txt
matrixCompSummary.txt
matrixDiff.txt