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

Update performance tests with v3 low res mesh #6271

Merged
merged 4 commits into from
Mar 20, 2024

Conversation

amametjanov
Copy link
Member

Update performance tests with v3 low res mesh:

  • run e3sm_prod_bench with v3 meshes in F- and B-cases
  • disable slow archiving of old tests on all ANL LCRC machines
  • add 3x32x2 PEs for ne4 I-cases on Chrysalis to match test-cases
  • shorten run-times of e3sm_prod RRM- and F-cases from Ld5 to Ld1
  • remove two redundant tests

[NML] -- for e3sm_prod_bench tests
[non-BFB] -- for 2 e3sm_prod tests due to test-name change

@amametjanov amametjanov added Machine Files non-BFB PR makes roundoff changes to answers. Testing Anything related to unit/system tests Chrysalis labels Feb 29, 2024
@amametjanov amametjanov requested a review from rljacob February 29, 2024 03:36
@amametjanov amametjanov self-assigned this Feb 29, 2024
Copy link

PR Preview Action v1.4.7
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6271/
on branch gh-pages at 2024-02-29 03:37 UTC

"SMS.ne30pg2_r05_IcoswISC30E3r5.F20TR.eam-wcprod_F20TR",
"SMS_Ld1.ne30pg2_r05_IcoswISC30E3r5.F20TR.eam-wcprod_F20TR",
Copy link
Member Author

Choose a reason for hiding this comment

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

Ld5 runs for 20-35 mins on Compy and Chrysalis.

@@ -301,7 +300,6 @@
"SMS_Ld2.ne30pg2_r05_IcoswISC30E3r5.BGCEXP_CNTL_CNPECACNT_1850.elm-bgcexp",
"SMS_Ld2.ne30pg2_r05_IcoswISC30E3r5.BGCEXP_CNTL_CNPRDCTC_1850.elm-bgcexp",
"SMS_D_Ld3.T62_oQU120.CMPASO-IAF",
"SMS_D_Ld1.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850",
Copy link
Member Author

Choose a reason for hiding this comment

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

Duplicate of SMS_D_Ld1.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850.allactive-wcprod.

Copy link
Member

Choose a reason for hiding this comment

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

Not exactly a duplicate since it doesn't have the testmod but doesn't add anything.

@@ -346,13 +344,12 @@
"e3sm_prod" : {
"inherit" : "e3sm_atm_prod",
"tests" : (
"SMS_Ld1.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850.allactive-wcprod_1850_r05",
Copy link
Member Author

Choose a reason for hiding this comment

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

Duplicate of SMS_Ld1.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850.allactive-wcprod_1850. Also removed test-mod dir allactive-wcprod_1850_r05, which is no longer used by anything else.

Comment on lines -355 to +352
"SMS_PS.northamericax4v1pg2_WC14to60E2r3.WCYCL1850.allactive-wcprodrrm_1850",
"SMS_Ld1_PS.northamericax4v1pg2_WC14to60E2r3.WCYCL1850.allactive-wcprodrrm_1850",
Copy link
Member Author

Choose a reason for hiding this comment

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

Ld5 runs for 25-50 mins at Compy and Chrysalis.

Comment on lines +526 to +529
<grid name="l%ne4np4.*">
<mach name="chrysalis">
<pes compset="any" pesize="any">
<comment>elm on chrysalis: any compset on ne4 grid, 3x32x2 NODESxMPIxOMP</comment>
Copy link
Member Author

Choose a reason for hiding this comment

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

Adding this to avoid over-decomposition run-time errors like

decompInit_lnd(): Number of clumps exceeds number of land grid cells 256 201

that can occur with otherwise default 4x32x2 PEs.

amametjanov added a commit that referenced this pull request Mar 20, 2024
Update performance tests with v3 low res mesh

Update performance tests with v3 low res mesh:
- run `e3sm_prod_bench` with v3 meshes in F- and B-cases
- disable slow archiving of old tests on all ANL LCRC machines
- add 3x32x2 PEs for ne4 I-cases on Chrysalis to match test-cases
- shorten run-times of `e3sm_prod` RRM- and F-cases from Ld5 to Ld1
- remove two redundant tests

[NML] -- for e3sm_prod_bench tests
[non-BFB] -- for 2 e3sm_prod tests due to test-name change
@amametjanov amametjanov merged commit b10c126 into master Mar 20, 2024
10 checks passed
@amametjanov amametjanov deleted the azamat/tests/v3res-update branch March 20, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chrysalis Machine Files non-BFB PR makes roundoff changes to answers. Testing Anything related to unit/system tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants