Skip to content

Conversation

@amametjanov
Copy link
Member

@amametjanov amametjanov commented Dec 15, 2025

Default to machine-specific XML-based pe-counts

[BFB]

@amametjanov amametjanov self-assigned this Dec 15, 2025
@amametjanov amametjanov added BFB PR leaves answers BFB Testing Anything related to unit/system tests labels Dec 15, 2025
mahf708
mahf708 previously approved these changes Dec 15, 2025
Copy link
Contributor

@mahf708 mahf708 left a comment

Choose a reason for hiding this comment

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

Looks good, tbc you should be able to run the statistical testing outside the cime testing infra, see docs https://docs.e3sm.org/E3SM/EAMxx/user/multi-instance-rcs/

@ndkeen
Copy link
Contributor

ndkeen commented Dec 15, 2025

Yea, I'm not a fan of P specification in tests, but we were just trying to get RCS working/tested.
What about instead, creating XML match for pelayout?
It may be that we need more than compset/resolution.

Default to machine-specific XML-based pe-counts
@amametjanov amametjanov force-pushed the azamat/tests/rm-rcs-pecounts branch from 42bfcd9 to 99e5bdc Compare December 15, 2025 19:57
@amametjanov amametjanov requested review from ndkeen and removed request for ndkeen December 15, 2025 19:58
@mahf708 mahf708 dismissed their stale review December 15, 2025 21:54

ndkeen should approve, not me

@amametjanov amametjanov assigned ndkeen and unassigned amametjanov Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BFB PR leaves answers BFB Testing Anything related to unit/system tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants