Skip to content

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Feb 22, 2018

By default, the bolus velocity is added as long as GM is turned on.

By default, the bolus velocity is added as long as GM is turned on.
@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

For the MPAS-O paper, we need the MOC with the bolus velocity included. It was decided at today's MPAS-O meeting that the default should probably be to plot the MOC including bolus, since this is what is typically included in publications where ocean models are compared with one another.

@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

Testing

I ran the analysis on the QU240 test case only so far. Here is the MOC from a baseline run (without bolus):
mocglobal_gmpas-qu240_years0002-0005

And here is from this run with bolus:
mocglobal_gmpas-qu240_years0002-0005

@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

Tomorrow, I'll run the analysis on an EC60to30 run to make sure things are working.

@xylar xylar requested review from jonbob and maltrud February 22, 2018 06:16
@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

@milenaveneziani, once I've tested, I'd like your help to make sure this is implemented in a way you're happy with.

@maltrud, @jonbob and @mark-petersen, I've included you as reviewers just so you can take a look at the results once they're posted tomorrow and tell me if you think they're sensible or nutty.

self.includeBolus = config.getboolean(self.sectionName, 'includeBolus')
if self.includeBolus:
# only add the bolus velocity if GM is enabled
self.includeBolus = self.namelist.getbool('config_use_standardgm')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

This PR should address #310

@milenaveneziani
Copy link
Collaborator

looks good to me @xylar.

@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

Great, let's see what the results look like tomorrow. We're kind of hoping that the bolus increases the MOC strength some because MPAS-O has a pretty weak MOC in our latest EC60to30 G-case simulations. Either way, it's the version of the MOC with bolus that we want to be comparing with others.

@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

Here are the results of the MOC with bolus:
http://portal.nersc.gov/project/m2833/mpas_analysis_output/20180209.GMPAS-IAF.T62_oEC60to30v3wLI.cori-knl.afterSalinityFix_moc_bolus/

Here are a series of before and after images of each MOC plot:
No bolus:
image
With bolus:
image

No bolus:
image
With bolus:
image

No bolus:
image
With bolus:
image

These are only years 8 through 13 of the spin-up with ice shelves. It would be more useful to redo these for the longer spin-up without ice shelves. But is seems clear that the bolus velocity is actually making the MOC weaker! That seems surprising but maybe it shouldn't -- eddy velocities are often counter to mean velocities. Still, we should make sure nothing is wrong...

@vanroekel
Copy link
Collaborator

@xylar I've checked out this branch and am running with and without bolus on 40 years of the current low res g-case. I'll post results when complete.

@vanroekel
Copy link
Collaborator

@xylar and @milenaveneziani is there an official naming convention for the MOC transect files? In the mpas-analysis code it looks for

gridRes_SingleRegionAtlanticWTransportTransects_masks.nc

but on anvil, all files follow

gridResTag_Atlantic_region_and_southern_transect.nc

Either name seems fine, I just want to make sure that we all follow the same convention.

@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

@vanroekel

is there an official naming convention for the MOC transect files? In the mpas-analysis code it looks for gridRes_SingleRegionAtlanticWTransportTransects_masks.nc

Yes, that's the required naming convention now. We used to supply the file name because there was only one but it became clear (after adding a mask for ice shelves) that we really needed a naming convention so all the needed mask files could be found automatically. Anvil must have data from a rather old copy of the files at NERSC.

We're working to get this stuff put on a common SVN repo where it will be checked out and updated on all supported machines automatically. Bear with us until that happens...

Copy link
Collaborator

@mark-petersen mark-petersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By visual inspection, this all looks correct. Thank you Xylar.

I was also expecting Bolus to increase the MOC, but you are right that the eddy transport often counters the mean.

@vanroekel
Copy link
Collaborator

@xylar I'll rename anvil files. Not a problem at all, just wanted to make sure I was on the same page for anvil.

@vanroekel
Copy link
Collaborator

@xylar one small issue with this PR is a directory path issue in the default config file. I have made the change and pushed here. Everything else worked great. Thank you!

Atlantic MOC With Bolus
mocatlantic_20180123 test-low-restoring gmpas-iaf t62_oecv3 anvil_years0160-0200

without
mocatlantic_20180123 test-low-restoring gmpas-iaf t62_oecv3 anvil_years0160-0200

@xylar xylar requested a review from vanroekel February 22, 2018 20:43
@xylar
Copy link
Collaborator Author

xylar commented Feb 22, 2018

@vanroekel, thanks for the fix.

That's a mighty weak MOC...

@maltrud
Copy link

maltrud commented Feb 22, 2018

i've been trying to confirm that the bolus amoc is counter to the eulerian mean, but haven't been able to find anything yet. i may just ping Gokhan. it just seems a bit weird that the global NH appears to be a bit stronger with bolus, but the NA is weaker. this implies that the NPac is also stronger with bolus. i just haven't looked at this kind of thing enough.

Copy link
Collaborator

@vanroekel vanroekel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR works great. But it would be nice to hear from Gokhan about the bolus and MOC before making this the default.

@xylar
Copy link
Collaborator Author

xylar commented Mar 2, 2018

@maltrud, any update on this one from your discussion with Gokhan?

@xylar
Copy link
Collaborator Author

xylar commented Mar 5, 2018

@maltrud, thanks for the email update. Everyone (@milenaveneziani and @maltrud in particular), is there a consensus on whether something is wrong here or if the results are just unexpected (but correct)? Any reason not to merge this PR?

Copy link
Collaborator

@milenaveneziani milenaveneziani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think so.

@xylar xylar removed request for jonbob and maltrud March 6, 2018 16:20
@xylar xylar merged commit fcfcc38 into MPAS-Dev:develop Mar 6, 2018
@xylar xylar deleted the add_bolus_to_moc branch March 6, 2018 16:21
@xylar
Copy link
Collaborator Author

xylar commented Mar 6, 2018

Thanks for the reviews @vanroekel, @mark-petersen and @milenaveneziani. I decided to go ahead and merge because I'm not sure if @maltrud is around. We can always do some fixing up later on if we discover an issue on the analysis side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants