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.

Sign up for GitHub

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

Jump to bottom

CLM setup fails for A_BGCEXP_BCRC_RDCN_2000 compset #1831

Closed
maltrud opened this issue Oct 11, 2017 · 7 comments
Closed

CLM setup fails for A_BGCEXP_BCRC_RDCN_2000 compset #1831

maltrud opened this issue Oct 11, 2017 · 7 comments
Assignees
@bishtgautam
Labels
compset Coupled Model Land

Comments

@maltrud
Copy link
Contributor

maltrud commented Oct 11, 2017

it appears that changes to CLMBuildNamelist.pm cause failure when setting up fully coupled BGC. the failure didn't occur in the development branch. here is the error message:

ERROR : CLM build-namelist::CLMBuildNamelist::setup_cmdl_methane() : use_lch4 is inconsistent with the commandline setting of -methane
ERROR clm.buildnml: /global/u1/m/maltrud/acme_repo/master_has_BGC_updates_Oct05_2017/components/clm/bld/build-namelist -infile /global/u1/m/maltrud/acme_repo/master_has_BGC_updates_Oct05_2017/A_BGCEXP_BCRC_RDCN_2000_ne30_oEC60to30v3_edison01/Buildconf/clmconf/namelist -csmdata /project/projectdirs/acme/inputdata -inputdata /global/u1/m/maltrud/acme_repo/master_has_BGC_updates_Oct05_2017/A_BGCEXP_BCRC_RDCN_2000_ne30_oEC60to30v3_edison01/Buildconf/clm.input_data_list -ignore_ic_year -namelist " &clm_inparm start_ymd=00010101

the error can easily be averted by adding use_lch4 = .true. in user_nl_clm, but the hope is that it would be set automatically (as it apparently was in the development branch) when using create_newcase.
@bishtgautam
Copy link
Contributor

bishtgautam commented Oct 11, 2017 •
edited
Loading

@maltrud, What is the full ./create_newcase command to reproduce this error?

@maltrud
Copy link
Contributor Author

maltrud commented Oct 11, 2017

@bishtgautam , here's what i did on edison:

./create_newcase -case ../../A_BGCEXP_BCRC_RDCN_2000_ne30_oEC60to30v3_edison01 -compset A_BGCEXP_BCRC_RDCN_2000 -res ne30np4_oEC60to30v3 -mach edison

@bishtgautam
Copy link
Contributor

I can reproduce the error. I will now work on a fix for it.

@bishtgautam
Copy link
Contributor

@maltrud I updated to the latest master (9c31301) and I don't get any error.

@maltrud
Copy link
Contributor Author

maltrud commented Oct 11, 2017

thanks, @bishtgautam . i figured that since my version was only a few days old, this would still be a problem, but it isn't. i'll make sure to always update in the future before submitting an issue.

jgfouca pushed a commit that referenced this issue Oct 11, 2017
@jedwards4b
Merge pull request #1831 from ESMCI/jgfouca/add_valid_lists_for_options
c4efac7 
Add valid_lists for some of the *_OPTION entries that lacked them
Test suite: scripts_regression_tests
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #1318

User interface changes?: N

Update gh-pages html (Y/N)?:N

Code review: jedwards
@bishtgautam
Copy link
Contributor

@maltrud: Can this issue be closed?

@bishtgautam
Copy link
Contributor

The compset doesn't exist in the master, so won't fix it.

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

@bishtgautam bishtgautam

Labels
compset Coupled Model Land
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@maltrud @bishtgautam