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

SMS_Ly3_P1x1.1x1_smallvilleIA.ICLM45CNCROP too slow on Cetus #1418

Closed
bishtgautam opened this issue Apr 17, 2017 · 2 comments
Closed

SMS_Ly3_P1x1.1x1_smallvilleIA.ICLM45CNCROP too slow on Cetus #1418

bishtgautam opened this issue Apr 17, 2017 · 2 comments
Assignees
Labels

Comments

@bishtgautam
Copy link
Contributor

@amametjanov reported that the test is taking longer than 1 hour to
run on Cetus.

FYI, @bbye.

@bishtgautam bishtgautam self-assigned this Apr 17, 2017
@bishtgautam
Copy link
Contributor Author

@bbye: What are the crop-related output variables that will be non-zero in the second year of simulation? Also, will reducing the simulation length from 3 to 2 years still test the crop model?

@bbye
Copy link
Contributor

bbye commented Apr 18, 2017

@bishtgautam Yes, a two year simulation should be sufficient. All the standard variables for carbon (i.e. LEAFC, FROOTC, LIVESTEMC) would be non-zero. Crop specific ones could include CRPYLD, or better yet GRAINC or GRAINN.

bishtgautam pushed a commit that referenced this issue Apr 18, 2017
The 3-years long test for the crop model is reduced to 2-years long
to allow the test to finish within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
bishtgautam pushed a commit that referenced this issue Apr 18, 2017
The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
bishtgautam pushed a commit that referenced this issue Apr 19, 2017
The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jqyin added a commit that referenced this issue Apr 21, 2017
Shortens the simulation length of a crop model test

The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jqyin added a commit that referenced this issue May 2, 2017
Shortens the simulation length of a crop model test

The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jgfouca pushed a commit that referenced this issue Jun 2, 2017
The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jgfouca pushed a commit that referenced this issue Jun 2, 2017
Shortens the simulation length of a crop model test

The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jgfouca pushed a commit that referenced this issue Jul 14, 2017
Adds support for data model to read multiple time slices at once
A new namelist variable ("readmode") is added that determines how
a temporal stream dataset is read. At present, "readmode" can be set to:

single : Default method of reading a single time slice at a time
full_file : Reads all time slices in a given netcdf file at once
Test suite:
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes 

User interface changes?: new xml variable, readmode

Code review:jedwards
jgfouca pushed a commit that referenced this issue Feb 27, 2018
The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jgfouca pushed a commit that referenced this issue Feb 27, 2018
Shortens the simulation length of a crop model test

The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jgfouca pushed a commit that referenced this issue Mar 14, 2018
The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
jgfouca pushed a commit that referenced this issue Mar 14, 2018
Shortens the simulation length of a crop model test

The 3-years long test for the crop model is reduced to 2-years long,
so the test finishes within 1:00:00 wallclock time on Cetus.
The test needs to be longer than 1-year because crops are planted
in the second year.

Fixes #1418
[BFB]
mark-petersen pushed a commit to mark-petersen/E3SM that referenced this issue Jan 19, 2021
Update needed for ACME

Framework features brought in:

* f444d0f Merge PR E3SM-Project#1418 'matthewhoffman/framework/output_record_reference_time' into develop
* 263e14f Merge PR E3SM-Project#1428 'mark-petersen/framework/couple_fixes' into develop
* bcce31d Merge PR E3SM-Project#1424 'amametjanov:az/tools/cp-prebuilt-tools' into develop
* 98cfeea Merge PR# 1349 'akturner/framework/forcing_cleanup' into develop
* 9359319 Merge PR E3SM-Project#1347 'akturner/framework/forcing_restart_timestamp' into develop
* e9ce203 Merge PR E3SM-Project#1348 'akturner/framework/forcing_at_init' into develop
* 4974284 Merge PR E3SM-Project#1368 'akturner/framework/improved_messages_in_driver' into develop
* 86d50c5 Merge PR E3SM-Project#1417 'akturner/framework/forcing_multiple_blocks' into develop
* 9116da3 Merge branch 'framework/validation-of-streams-using-interval_in-interval_out' into develop
* e466b46 Merge branch 'framework/interval_in-interval_out-support-for-streams' into develop
* 30dc955 Merge branch 'az/framework/mpas_dmpar-race-fix' into develop
* b632938 Merge branch 'framework/i8_interval_division' into develop
* 6dac06c Merge branch 'framework/log_write_IBM_error' into develop
* 960a648 Merge branch 'framework/cleanup-logging-stream-manager' into develop
* 504c282 Merge branch 'framework/make-streams-with-direction-none-inactive' into develop
* 5903748 Merge branch 'framework/correctly_remove_blk_fields' into develop
* 3565965 Merge branch 'framework/iostreams-real4dfield-bug' into develop
* 8b60591 Merge branch 'framework/missing-deallocate-nEdgesOnCellField-bootstrapping' into develop
* 70b953b Merge branch 'master' into develop
rljacob pushed a commit that referenced this issue May 6, 2021
Adds support for data model to read multiple time slices at once
A new namelist variable ("readmode") is added that determines how
a temporal stream dataset is read. At present, "readmode" can be set to:

single : Default method of reading a single time slice at a time
full_file : Reads all time slices in a given netcdf file at once
Test suite:
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes 

User interface changes?: new xml variable, readmode

Code review:jedwards
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants