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 megan compatibility with fates #98

Closed

Conversation

mvertens
Copy link

Description of changes

Megan changes without the fire emissions commits.

Specific notes

Same notes as the previous now closed PR
#74

Contributors other than yourself, if any:

CTSM Issues Fixed (include github issue #):
#36

Are answers expected to change (and if so in what way)?
Yes, all the MEG_* fields will change, but nothing else.

Any User Interface Changes (namelist or namelist defaults changes)?
No

Does this create a need to change or add documentation? Did you do so?
Yes, but haven;'t changed yet pending further coupled model testing.

Testing performed, if any:
Ran the following test command:
./create_test --xml-category aux_clm_noresm --xml-machine betzy --baseline-root ~/baselines/ctsm_develop --generate noresm2_5_alpha08 --test-id ctsmpr1 --project nn9039k -o

    FAIL ERS_Ld761.ne30pg3_ne30pg3_mtn14.2000_DATM%GSWP3v1_CLM60%FATES-NOCOMP_SICE_SOCN_MOSART_SGLC_SWAV.betzy_intel.clm-FatesColdNoComp RUN time=2995
    FAIL SMS_D.1x1_brazil.2000_DATM%GSWP3v1_CLM60%FATES-SP_SICE_SOCN_SROF_SGLC_SWAV.betzy_intel.clm-FatesColdDryDepSatPhen RUN time=56
    FAIL SMS_D_Ld3.ne30pg3_ne30pg3_mtn14.2000_DATM%GSWP3v1_CLM60%FATES-SP_SICE_SOCN_SROF_SGLC_SWAV.betzy_intel.clm-FatesColdSatPhen_prescribed RUN time=62

All the other tests passed.

ekluzek and others added 30 commits September 18, 2024 14:28
Also makes it accessible from table of contents. Not actually changing file contents at this point.
* The experience of making a mesh from an input with 1d vs. 2d coordinates is identical, so only provide one example.
* In example, use as input file the 5x5_amazon surface dataset included with CTSM.
* Various wording and typo fixes.
…le and just use the local n_pfts variable that matches the name on the dataset
@mvertens
Copy link
Author

Closing since this is the wrong branch to merge to. Should be noresm not master.

@mvertens mvertens closed this Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants