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

NOANTHRO qualifier NOT working in ctsm5.2.005 for CLM51 physics (only for CLM50 or CLM60) #2561

Closed
ekluzek opened this issue May 28, 2024 · 4 comments · Fixed by #2808
Closed
Assignees
Labels
bug something is working incorrectly science Enhancement to or bug impacting science usability Improve or clarify user-facing options

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented May 28, 2024

Brief summary of bug

This is from Jiang in the Paleo group:

Please note that run 091 is using CLM51%BGC-CROP in its compset. Based on that, my LGM simulation is using CLM51%SP-NOANTHRO.

General bug information

CTSM version you are using: ctsm5.2.005 (in cesm2_3_beta17)

Does this bug cause significantly incorrect results in the model's science? Yes

Configurations affected: CLM51%SP-NOANTHRO

Details of bug

Important output or errors that show the problem

See here
/glade/campaign/cesm/cesmdata/cseg/runs/cesm2_0/b.e23_alpha17f.BLT1850.ne30_t232.091
/glade/u/home/jiangzhu/cesm/cases/b.e23.BLT1850.ne30pg3_t232.cesm2_3_b17.21ka.01

@ekluzek ekluzek added priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations bug something is working incorrectly tag: bug - impacts science next this should get some attention in the next week or two. Normally each Thursday SE meeting. labels May 28, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented May 28, 2024

The problem is here:

https://github.com/ESCOMP/CTSM/blob/master/cime_config/config_component.xml#L205-L206

where NOANTHRO is only defined for CLM50 and CLM60.

What we should setup is a way for options that are NOT defined to trigger an error and abort.

@ekluzek
Copy link
Collaborator Author

ekluzek commented May 28, 2024

The same kind of problem shows up for DV and VIC for the setting of: CLM_BLDNML_OPTS, and for CLM_USER_MODS for CMIP6 and WACCMCMIP6 and FATES-SP.

For LND_TUNING_MODE invalid settings set an invalid value and hence die with an error. We should do something similar for these.

There's also some restrictions to configuration setup for description at the top that would catch certain issues -- but NOT all of them.

@ekluzek ekluzek changed the title NOANTHRO qualifier NOT working in ctsm5.2.005 NOANTHRO qualifier NOT working in ctsm5.2.005 for CLM51 physics (only for CLM50 or CLM60) May 28, 2024
@wwieder wwieder added this to the ctsm6.0.0 (code freeze) milestone Jun 20, 2024
@wwieder wwieder removed priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations next this should get some attention in the next week or two. Normally each Thursday SE meeting. labels Jun 20, 2024
@samsrabin samsrabin added science Enhancement to or bug impacting science and removed bug - impacts science labels Aug 8, 2024
@ekluzek ekluzek added the usability Improve or clarify user-facing options label Aug 29, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Sep 27, 2024

This will be resolved in terms of NOANTHRO with #2379

@wwieder
Copy link
Contributor

wwieder commented Oct 24, 2024

will be resolved in #2808

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is working incorrectly science Enhancement to or bug impacting science usability Improve or clarify user-facing options
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants