-
Notifications
You must be signed in to change notification settings - Fork 374
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
Add scaling of DIB and DISMF by running mean of removed Antarctic ice runoff #6696
Add scaling of DIB and DISMF by running mean of removed Antarctic ice runoff #6696
Conversation
68df68a
to
f16058e
Compare
components/mpas-ocean/cime_config/testdefs/testmods_dirs/mpaso/scaled_dib_dismf/README
Outdated
Show resolved
Hide resolved
dc5515b
to
ddf3e95
Compare
@xylar Is there any testing that would be helpful for me to do? |
@cbegeman, I think we're mostly waiting on the longer run that @darincomeau is doing. I can't think of other testing that I didn't already report on E3SM-Ocean-Discussion#109, at least not anything that would be a good use of your time. |
Update, longer test run is in year 50 now: https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.dcomeau/E3SMv3_dev/20241018.v3.SORRME3r3.piControl.scaledDISMF.chrysalis/mpas_analysis/ts_0001-0050_climo_0041-0050/ Note ISMF is not plotting due to #6719, fixed in #6729 |
@jonbob, if you look at the surface fluxes, a lot of them show atmosphere blockiness. Is that expected? Here's an example: Not related to this PR but showing up in the analysis above. |
@xylar -- @cbegeman and I were looking at this last week. We rely on the nonlinear CAAS mapping files to smooth out some of the grid imprinting, but we found that snow and rain are removed from the list of fluxes that get this treatment. It's easily changed in user_nl_cpl if you want to test the impact. |
And I guess the first thing to do is to make sure we have the nonlinear mapping file for atm2ocn specified for this run |
@xylar and @jonbob I discussed this with @proteanplanet and we will not be changing the atmosphere mapping files because it would compromise conservation accuracy across the coupled system, which is a higher priority. Thus, we will be retaining the blockiness moving forward. |
Added information: The decision was made to switch off non-linear remapping following testing in the lead-up to v3, but just for mass fluxes as documented by Andrew Bradley in the Critical Path meeting on December 4, 2023: |
which grid set is this? I think we only automatically turn on CAAS for the v3 grid set. |
@rljacob -- it's using the new SOwISC12to30E3r3 grid |
@jonbob (and anyone else who wants to chime in), is there a reason we can't use higher order, conservative (linear) mapping files from TempestRemap for these fields? It would seem to me that would reduce blockiness without compromising conservation. I'm mostly just seeking to understand why some flux fields are blocky and others don't seem to be. |
@xylar -- we do use conservative mapping files from tempest for all flux fields. The nonlinear CAAS mapping files are also used for some, but not all, of those fluxes. We would probably see less blockiness by also using the nonlinear mapping files for snow and rain -- but apparently there has been a decision not to do that |
Some discussion here: https://acme-climate.atlassian.net/wiki/spaces/CM/pages/4016177153/2023-12-01+v3+Critical+Path+Meeting#Discussion-on-coupler-water-budgets |
The links @rljacob and I have provided above summarize the issue, but ultimately it boils down to several decimal places of accuracy in the global water budget; non-linear remapping leaks mass more than linear remapping. To answer your questions @xylar: A reason why certain fields in sea ice show the aliasing, but others do not, is that sea ice growth is particularly sensitive to snow conductivity. There are other reasons associated with the aging of snow, too, but rain does not play a role because it just runs off. |
Okay, sorry for the noise about this. It doesn't relate to this PR in any case, I just noticed it and it looked pretty ugly. |
I'm very glad you noticed it and questioned it. The more scrutiny E3SM receives, the better it gets. |
Read in areaIntegAnnMeanDataIcebergIceShelfFreshwaterFlux from these files
Read in areaIntegAnnMeanDataIcebergIceShelfFreshwaterFlux from these files
Most of the support is in place, just not the coupling between MPAS-Ocean and MPAS-Seaice to pass `runningMeanRemovedIceRunoff`
Since the alarm rings at the end of a daily interval, we need to accumulate the removed runoff before, not after, we reset the daily accumulation.
This is necessary becasue we want to update before streams get written.
We want this to be a valid entry that contributes to the running mean.
4cb3096
to
dd80901
Compare
Passes (with expected NML DIFFs):
Also successfully ran the new test on chrysalis:
|
Merge branch 'xylar/ocn-ice/add-scaled-dismf-dib' into next (PR #6696) Add scaling of DIB and DISMF by running mean of removed Antarctic ice runoff This merge adds a capability by which data iceberg (DIB) freshwater fluxes and data ice-shelf melt fluxes (DISMF) can be scaled by the running mean of the the amount of ice (solid) runoff coming from Antarctica (which is removed from the system by MPAS-Ocean). This merge includes: * A capability in MPAS-Ocean to compute daily averages of the area integrated removed ice runoff * A capability in MPAS-Ocean to compute the running mean of these daily means over a given period of time (currently 2 years = 730 days) * A capability to pass this running mean through the coupler (via the field rmean_rmv_ice_runoff in the infodata datastructure) from MPAS-Ocean to MPAS-Seaice * A requirement that the the total annual mean DIB and DISMF fluxes be computed and stored in the DIB and DISMF input files. These files are currently only available for the IcoswISC30E3r5 mesh. * Capabilities to compute a scale factor between the annual mean total data freshwater flux and the running mean of the removed ice runoff in both MPAS-Ocean and MPAS-Seaice * A capability to scale landIceFreshwaterFlux and landIceHeatFlux in MPAS-Ocean by the scale factor * A capability to scale bergFreshwaterFlux and bergLatentHeatFlux in MPAS-Seaice by the scale factor [NML] [BFB] stealth feature
merged to next |
🎉🎉🎉 Thanks @jonbob, @cbegeman and @darincomeau! |
merged to master and expected NML DIFFs blessed, except for pm-cpu which has yet to report note that we had a few failures on non-lcrc platforms due to the new iceberg climatology file not being staged on the inputdata server. Those should clear with the next round of testing, though more blesses may be required after that |
Fix issue with mpas-seaice restart_contents A new field runningMeanRemovedIceRunoff was added in PR #6696 that causes errors in some configurations on pm-cpu and frontier with the gnu compiler. This moves the new field from a shared "meta-stream" to the streams themselves because attached package wasn't getting inherited correctly. Fixes #6855 [BFB]
Fix issue with mpas-seaice restart_contents A new field runningMeanRemovedIceRunoff was added in PR #6696 that causes errors in some configurations on pm-cpu and frontier with the gnu compiler. This moves the new field from a shared "meta-stream" to the streams themselves because attached package wasn't getting inherited correctly. Fixes #6855 [BFB]
This merge adds a capability by which data iceberg (DIB) freshwater fluxes and data ice-shelf melt fluxes (DISMF) can be scaled by the running mean of the the amount of ice (solid) runoff coming from Antarctica (which is removed from the system by MPAS-Ocean).
This merge includes:
rmean_rmv_ice_runoff
in theinfodata
datastructure) from MPAS-Ocean to MPAS-SeaicelandIceFreshwaterFlux
andlandIceHeatFlux
in MPAS-Ocean by the scale factorbergFreshwaterFlux
andbergLatentHeatFlux
in MPAS-Seaice by the scale factor[NML]
[BFB] stealth feature
The capability is described in more detail in this design document
https://acme-climate.atlassian.net/wiki/spaces/PSC/pages/4210098268/Design+Document+Data+iceberg+and+ice-shelf+melt+flux+patterns+for+E3SM+runs
Relevant discussion can be seen at E3SM-Ocean-Discussion#109