-
Notifications
You must be signed in to change notification settings - Fork 53
Support standalone MPAS runs #306
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
Conversation
TestingI ran a QU240 MPAS-Ocean test on my laptop from scratch. I followed the same procedure as in the new instructions I've added to the documentation. Once the run had reached 2 years, all analysis worked fine. I'll note that I had a little trouble producing an MOC mask file that worked with the grid I have, which has a different number of cells from the QU240v3 grid for some reason. Some thought will need to go into how to produce masks in a more automated way once we have a manual workflow for these masks that we're happy with. I have also verified that analysis of a QU240 E3SM run works fine, as before. |
@pwolfram, I'm not sure what the easiest way for you to review this might be. Hopefully, you can run the analysis on your SOMA uniform32km test case and, assuming everything works, that will suffice for testing these changes. |
@xylar, the SOMA run is on IC at |
58515eb
to
22851a9
Compare
@pwolfram, I was able to use e3sm-unified on Grizzly (
I'll try again when the SOSE data are there and when we're further along. So far, it looks like all the necessary data is there. Note: I'm having trouble with MPAS-Analysis using |
22851a9
to
d9934db
Compare
Further note on running at LANL: The latest HDF5 (1.10.1) glitches because some expected locks are not supported. To handle this issue, you must set:
after |
@pwolfram, I was able to run the analysis successfully but I haven't had a chance to look at the results. Here's the config file I used:
|
Could you try the same, move the results over to the NERSC portal, and post it here? That would be enough testing to allow us to merge this PR, I think. |
@darincomeau, can you review this based on your standalone MPAS-SeaIce testing? |
@xylar sure, no problem. |
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.
@xylar I successfully ran the analysis on 10 years of an MPAS-Seaice QU120km standalone on LANL IC, and posted the results here:
http://portal.nersc.gov/project/m2833/dcomeau/mpas_analysis_output/html/
Otherwise just a couple typos noted.
docs/mpascice.rst
Outdated
MPAS Sea Ice | ||
============ | ||
|
||
The Model for Predictiona Across Scales Sea Ice (MPAS Sea Ice, sometimes called |
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.
typo in 'Prediction'
docs/mpaso.rst
Outdated
MPAS Ocean | ||
========== | ||
|
||
The Model for Predictiona Across Scales Ocean (MPAS-O) is designed for the |
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.
same 'Prediction' typo
docs/mpascice.rst
Outdated
|
||
Several streams must be defined in the streams file, typically | ||
``streams.cice``, (even if they will not be written out -- | ||
``output_intervale=="none"``):: |
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.
typo in 'output_interval'
The filename_template attribute can now be parsed into a format that datetime.strptime can use to extract the year, month, etc. from a file name.
Instead of relying on monthly mean files having the E3SM file format, parse the year and month using the filename_template attribute from the stream.
This allows us to use ncclimo even when a standalone MPAS run has used different file-naming conventions than ncclimo supports.
Standalone MPAS runs need to be set up with the proper analysis members enabled and streams defined. These are now described in the documentation. Authors are now included at the end of the documentation.
d9934db
to
e0dc22c
Compare
@darincomeau, thanks for the review. The typos have been fixed. I just rebased onto the current develop. @pwolfram, when might you have simulations to use to check this PR? |
@xylar, as we are discussing offline the check is in progress now... |
The output is at http://portal.nersc.gov/project/m2833/pwolfram/mpas_analysis_output/html/. As the SOMA case is an idealized spherical cap, I'm not sure how seriously we should take this output. But, I think the key thing is that this demonstrates the opportunity for stand alone analysis to be implemented in MPAS-Analysis. |
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.
Verified code completed without error.
monthly (``"0000-01-00_00:00:00"``). | ||
|
||
Additional fields can be included in the ``timeSeriesStatsMonthlyOutput`` | ||
streams. These are the minimum that allow the analysis to run successfully. |
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'm assuming this is from the wiki we used in this process. Thanks for including it here!
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.
These will become pages in the official documentation on ReadTheDocs as soon as this is merged.
One minor piece of food for thought: Note, for whatever reason, if there is bad output in the output directory, there can be a crash (happened to me the first time). I had a suspicion this was the case and turned out to be lucky that a fresh output directory fixed the problem. I don't recall if we have a purge on the output directory or not but it might be a good feature to have if we already don't have it in MPAS-Analysis, especially for new users. |
Ha, ha!, the analysis does look pretty silly in this case. But presumably as we build up more capabilities, there could be useful plots here. Thanks for testing, @pwolfram. |
@pwolfram, yes, just run |
We haven't been testing standalone MPAS-O and MPAS-SeaIce runs on a regular basis. As a result, they were no longer working and this merge attempts to remedy that situation.
File names produced by standalone MPAS runs don't necessarily conform to E3SM standards, and are therefore not supported by
ncclimo
all versions of NCO. (ncclimo
in NCO 4.7.3 will include more general file-name parsing so this could be a partial solution to the problem.) The solution here is to create symlinks to the timeSeriesStatsMonthlyOutput files in the analysis output directory where the links have the expected E3SM file names.Additionally, there are 3 places in MPAS-Analysis where years and months are extracted from file names. Previously, this was done in a lazy way assuming the E3SM file-naming convention. With this merge, file names are instead parsed based on the
filename_template
attribute in the associated stream.Finally, the documentation has been updated to give instructions on how to set up analysis members, including output streams, in standalone MPAS runs to they are compatible with MPAS-Analysis. These instructions will need to be updated as new variables are used in analysis task. (A documentation page with a list of authors has also been added.)