-
Notifications
You must be signed in to change notification settings - Fork 53
Add analysis tasks for plotting SOSE T and S #288
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
TestingMy laptop: Edison:
|
|
@stephenprice, I'm sure you'll be quite busy at AGU. If you have time, maybe run a quick test somewhere just to make sure the latest analysis works on ALCC runs. |
|
@mark-petersen, depending on how busy you are this week, maybe you could try out this branch as a way of getting your feet wet with the latest analysis? Let me know if you don't have time and I'll remove you as a reviewer. |
|
It's not necessary but I can.
…On Dec 12, 2017 9:16 PM, "Milena Veneziani" ***@***.***> wrote:
@xylar <https://github.com/xylar>: this needs to be rebased after #289
<#289>, right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#288 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AD_EeFR68XbDsobT0LCGBWKkj_gyca3tks5s_t8wgaJpZM4Q8gf2>
.
|
|
oh, ok, sorry, it seemed some files were already changed in #289. Never mind. |
|
@xylar I am trying to test this branch on my cori run: I could run this PR branch on edison or cori - do both work? Which config template should I start from for this G case RRS30to10? Sorry, I haven't done this in a long time. Thanks! |
|
@mark-petersen: I do not believe we have a config template for the 30to10, but probably the closest example is this one: |
259f52b to
f03ec5d
Compare
|
@stephenprice, I know you're busy but if you have time to test this on the run of your choice sometime in the next few days, that would be really helpful. @milenaveneziani, same to you if you have time. Since @mark-petersen won't be available, I'm going to take him off as a reviewer for now. |
|
@xylar: I tested this branch on the |
|
One thing I should note is that I ran |
|
yes, I can confirm that the following 'generate' flags do not work: |
|
@milenaveneziani, thanks for reviewing the PR. I agree, the output you posted looks good to me. Very interesting to see the differences between E3SM and SOSE there. |
|
Regarding To do what you wanted to do, I would do, |
...to ocean.PlotClimatologyMapSubtask
Adds support for *not* explicitly defining the ticks for an Antarctic plot (so ticks are determined automatically).
They can be plotted over various seasons and at various depths.
The directory currently has scripts for preprocessing Rignot melt rates and SOSE T and S monthly climatologies.
The configs from edison will work on cori, so they haven't been added explicitly.
This merge also updates 2 other config files with land-ice cavities to include the options for changing seasos and depths for the SOSE comparison plots.
|
Okay, this PR has languished for long enough. I'm going to go ahead and merge. @mark-petersen and @stephenprice, if you find issues afterward, we can address them with a follow-up PR. |


They can be plotted over various seasons and at various depths.
A utility for computing MPAS-Ocean zMid has been added.
mpas_analysis.ocean.PlotClimatologyMapSubtasknow supports plotting a field at a specified depth.Scripts for pre-processing Rignot et al (2013) and SOSE observations have been added in the
preprocess_observationsdirectory.