Skip to content
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 new ne30 grids with active MALI #3069

Merged
merged 8 commits into from
Aug 8, 2019
Merged

Add new ne30 grids with active MALI #3069

merged 8 commits into from
Aug 8, 2019

Conversation

jhkennedy
Copy link
Contributor

Adds two new ne30 grids with an active (but not dynamic) MALI component, with a new combined Antarctic and Greenland ice sheet model. Importantly, without an active MALI component, ELM cannot compute the required variables needed analyze the (snow) surface mass balance. This will allow SMB analysis for the low res watercycle and cryosphere campaigns, which is critically important for:

  • new snow model physics being developed
  • updating the ELM-->MALI downscaling
  • prepping/validation for v3 runs with dynamic ice sheet model runs.

The grids are: ne30_oECv3_aisgis and ne30_oECv3wLI_aisgis

[BFB]

Joseph H Kennedy and others added 5 commits July 13, 2019 12:23
Copy link
Contributor

@matthewhoffman matthewhoffman left a comment

Choose a reason for hiding this comment

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

@jhkennedy , thanks for getting this PR in - this will be a big step forward in the SMB work to have the dual ice sheet grids available out of the box.

My review by inspection includes some questions about the mapping file definitions. Otherwise everything looks as I would expect. @jonbob might notice details I have not.

@jonbob
Copy link
Contributor

jonbob commented Jul 25, 2019

@jhkennedy - does this PR need the newer version of MALI (in PR 3032) before it can be merged?

@stephenprice
Copy link
Contributor

@jonbob -- I'm 99% sure these are entirely separate. That is, the features in this PR can operate independent of the changes being merged in as part of 3032.

@jhkennedy
Copy link
Contributor Author

@jonbob yep, @stephenprice got it -- they are (or at least should be) independent.

@jonbob
Copy link
Contributor

jonbob commented Jul 25, 2019

@jhkennedy - do you know if LND <=> GLC mapping requires the "mono" and "intbilin" methods? I have to admit I'm not clear on this, and @matthewhoffman is out of town

@matthewhoffman
Copy link
Contributor

matthewhoffman commented Jul 25, 2019 via email

@mt5555
Copy link
Contributor

mt5555 commented Jul 25, 2019

FYI: both maps are monotone. intbilin is more accurate but not conservative. mono is less accurate but conservative.

@jonbob
Copy link
Contributor

jonbob commented Jul 25, 2019

Thanks @mt5555 - I think it makes sense for the mappings to be consistent, so we'll discuss and fix this PR. But we may ask your advice on which maps to use.

@jonbob
Copy link
Contributor

jonbob commented Jul 30, 2019

@bishtgautam - can you please look at this comment above? I think the landice people are using an older file because it contains glc elevation class information that the newer one does not? If so, is there anyway to update the newer file to make it usable for their purposes? Thanks for your help!

@bishtgautam
Copy link
Contributor

bishtgautam commented Jul 30, 2019

I will regenerate the land surface for ne30np4 for 2000 to include glacier elevation class and get back to you.

@bishtgautam
Copy link
Contributor

@jhkennedy, The new land surface dataset that is available on blues/anvil at the following path:
/home/ccsm-data/inputdata/lnd/clm2/surfdata_map/surfdata_ne30np4_simyr2000_c190730.nc
Could you test it out? Thanks.

@jhkennedy
Copy link
Contributor Author

Great, thanks @bishtgautam! I'll kick off a couple of runs today.

@jonbob
Copy link
Contributor

jonbob commented Jul 31, 2019

Thanks @bishtgautam!

@matthewhoffman
Copy link
Contributor

@jhkennedy , check out the comment I added at the bottom of this thread: https://acme-climate.atlassian.net/wiki/spaces/PROBSLR/pages/1003323510/Figures+for+SciDAC+PI+meeting+July+16--18+2019?focusedCommentId=1004503418#comment-1004503418
I think it might explain the grid imprinting issue you were seeing. It would be great if we are able to resolve that in this PR. I think it might just require a change in the LND2GLC State mapping method.

@jhkennedy
Copy link
Contributor Author

@matthewhoffman I'll try it with the other mappings as well. For sure, I'd like to get that sorted here as well.

@jhkennedy
Copy link
Contributor Author

@matthewhoffman I think we've sorted the grid imprinting issues, see my reply here.

I think everything here is ready to go at this point

@matthewhoffman
Copy link
Contributor

@jhkennedy , that looks great! I'll coordinate with @jonbob to do a final review and testing to get this merged.

@jhkennedy jhkennedy requested a review from jonbob August 7, 2019 17:55
jonbob added a commit that referenced this pull request Aug 7, 2019
Add new ne30 grids with active MALI

This PR adds two new grids, ne30_oECv3_aisgis and ne30_oECv3wLI_aisgis, for
configurations with an active (but not dynamic) MALI component using a
combined Antarctic and Greenland ice sheet model.  Without the active MALI
component, ELM cannot compute the required variables needed analyze the
(snow) surface mass balance. This will allow SMB analysis for the low-res
watercycle and cryosphere campaigns, which is critically important for:
* new snow model physics being developed
* updating the ELM-->MALI downscaling
* prepping/validation for v3 runs with dynamic ice sheet model runs.

[BFB]
@jonbob
Copy link
Contributor

jonbob commented Aug 7, 2019

merged to next

@jonbob jonbob merged commit ae468ac into master Aug 8, 2019
jonbob added a commit that referenced this pull request Aug 8, 2019
Add new ne30 grids with active MALI

This PR adds two new grids, ne30_oECv3_aisgis and ne30_oECv3wLI_aisgis, for
configurations with an active (but not dynamic) MALI component using a
combined Antarctic and Greenland ice sheet model.  Without the active MALI
component, ELM cannot compute the required variables needed analyze the
(snow) surface mass balance. This will allow SMB analysis for the low-res
watercycle and cryosphere campaigns, which is critically important for:
* new snow model physics being developed
* updating the ELM-->MALI downscaling
* prepping/validation for v3 runs with dynamic ice sheet model runs.

[BFB]
@jonbob
Copy link
Contributor

jonbob commented Aug 8, 2019

merged to master and NML DIFF blessed

@jonbob jonbob deleted the jhkennedy/ne30-glc branch August 8, 2019 16:40
rljacob pushed a commit that referenced this pull request Apr 12, 2021
Add new ne30 grids with active MALI

This PR adds two new grids, ne30_oECv3_aisgis and ne30_oECv3wLI_aisgis, for
configurations with an active (but not dynamic) MALI component using a
combined Antarctic and Greenland ice sheet model.  Without the active MALI
component, ELM cannot compute the required variables needed analyze the
(snow) surface mass balance. This will allow SMB analysis for the low-res
watercycle and cryosphere campaigns, which is critically important for:
* new snow model physics being developed
* updating the ELM-->MALI downscaling
* prepping/validation for v3 runs with dynamic ice sheet model runs.

[BFB]
rljacob pushed a commit that referenced this pull request Apr 21, 2021
Add new ne30 grids with active MALI

This PR adds two new grids, ne30_oECv3_aisgis and ne30_oECv3wLI_aisgis, for
configurations with an active (but not dynamic) MALI component using a
combined Antarctic and Greenland ice sheet model.  Without the active MALI
component, ELM cannot compute the required variables needed analyze the
(snow) surface mass balance. This will allow SMB analysis for the low-res
watercycle and cryosphere campaigns, which is critically important for:
* new snow model physics being developed
* updating the ELM-->MALI downscaling
* prepping/validation for v3 runs with dynamic ice sheet model runs.

[BFB]
jgfouca pushed a commit that referenced this pull request Nov 7, 2024
…s-change

EAMxx: refactor machine_specs.py file
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.

7 participants