-
Notifications
You must be signed in to change notification settings - Fork 134
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 use_mpp_io from diag manager #940
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.
Looks good from what I can see.
@thomas-robinson This doesn't actually have any duplication from #928 since I wasn't sure whether or not to get rid of the use_mpp_io namelist flag (if anyone still uses it). Of course, I could incorporate these changes into #928 if we feel like they should go together. |
Thanks @GFDL-Eric I think you should keep #928 as is, and let this be part of the diag manager modernization. Then we can remove the rest of |
@thomas-robinson sounds good! |
Description
Removes the variable
use_mpp_io
from the diag manager.Fixes # (issue)
How Has This Been Tested?
CI
Checklist:
make distcheck
passes