-
Notifications
You must be signed in to change notification settings - Fork 52
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 submodule in PreAndPostProcessingScripts following support for reading namelist and streams files #28
Comments
@milenaveneziani, do you think we are missing anything on the above list? |
Thanks for the updates @milenaveneziani and @xylar in the above list. |
@milenaveneziani and @xylar, following merging of #55, my understanding is that we can update |
That's my understanding as well. |
Note, I believe that we should be able to close this issue following resolution of https://github.com/ACME-Climate/PreAndPostProcessingScripts/pull/24. |
Until that PR is actually merged, I think this should stay open. |
Agreed, issues should only be closed after rectifying PRs are merged. |
yes, we should be close to fulfill ACME-Climate/PreAndPostProcessingScripts#24. |
ACME-Climate/PreAndPostProcessingScripts#24 was fulfilled, so closing this now. |
Following completion of recent improvements to generalize usage of MPAS-Analysis we will need to update the submodule for inclusion in ACME (https://github.com/ACME-Climate/PreAndPostProcessingScripts). The goal of this issue is to document PRs that we want to make sure are included as part of this update process.
A partial list of improvements (issues / PRs) to be included:
The text was updated successfully, but these errors were encountered: