Skip to content

Auto-select PECOUNT from active ocean points at case init#233

Open
manishvenu wants to merge 3 commits into
mainfrom
add_pecount_arg
Open

Auto-select PECOUNT from active ocean points at case init#233
manishvenu wants to merge 3 commits into
mainfrom
add_pecount_arg

Conversation

@manishvenu

@manishvenu manishvenu commented Jun 3, 2026

Copy link
Copy Markdown
Member

Details:

  • This PR must be merged first: Add PECOUNT as optional cvar for create_newcase pe layout ESMCI/visualCaseGen#41
  • Adds _compute_pecount() to Case, which counts active ocean cells from ocn_topo.tmask and selects a CIME pecount tier (XS/S/M/L)
  • Uses pts/core benchmarks derived from tx2_3v3: 170 pts/core without MARBL, 60 pts/core with MARBL, in increments of full Derecho nodes (128 cores)
  • Sets cvars["PECOUNT"] in _configure_launch() so create_newcase receives the appropriate --pecount flag

Closes #162

Adds _compute_pecount() to Case, which counts active ocean cells from
ocn_topo.tmask and maps them to a CIME pecount tier (XS/S/M/L) using
pts/core benchmarks from tx2_3v3 (170 pts/core without MARBL, 60
pts/core with MARBL). Sets cvars["PECOUNT"] in _configure_launch() so
create_newcase receives the appropriate --pecount flag.

Also bumps the visualCaseGen submodule to the add_pecount_arg branch
which adds PECOUNT cvar support to visualCaseGen.
@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

📄 Preview your docs here:
👉 https://CROCODILE-CESM.github.io/CrocoDash/pr-233/index.html

@manishvenu manishvenu requested a review from alperaltuntas June 3, 2026 20:14
@codecov

codecov Bot commented Jun 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

NTASKS layout is just wrong.

1 participant