Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add EXCL_STRIDE to XML PE configs #4884

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
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add EXCL_STRIDE to XML PE configs #4884

wants to merge 2 commits into from

Conversation

amametjanov
Copy link
Member

This enables selection of custom EXCL_STRIDE values from
config_pesall.xml PE files to populate $case-dir/env_mach_pes.xml: e.g.

  <grid ...>
    <mach name="pm-gpu">
      <pes compset=...>
        <ntasks>
          ...
        </ntasks>
        <pstrid>
          <pstrid_atm>16</pstrid_atm>
        </pstrid>
        <excl_stride>
          <excl_stride_atm>16</excl_stride_atm>
        </excl_stride>
      </pes>
    </mach>
  </grid>

Checklist

  • My code follows the style guidlines of this proejct (black formatting)
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • I have added tests that excerise my feature/fix and existing tests continue to pass
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding additions and changes to the documentation

@github-actions
Copy link
Contributor

You can preview documentation at https://esmci.github.io/cime/branch/azamat/pes/add-xstrid-to-xml/html/index.html

jedwards4b
jedwards4b reviewed Oct 21, 2025
View reviewed changes
Copy link
Contributor

@jedwards4b jedwards4b left a comment

Choose a reason for hiding this comment

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

I'm having trouble understanding what new functionality this provides? Can't we already specify this with the existing xml variables?

@amametjanov
Copy link
Member Author

Previously, I couldn't set excl_stride for a matching grid+compset+mach as in this config_pesall.xml file when matching a _PS modifier in

./cime/scripts/create_test SMS_PS.ne256pg2_r025_RRSwISC6to18E3r5.WCYCLXX2010.pm-gpu_gnugpu

I tried black formatting, but GH-CI black hook is still failing. Not sure how to get it passing:

> black --version
black, 25.9.0 (compiled: yes)
Python (CPython) 3.11.7

> black CIME/case/case.py
All done! ✨ 🍰 ✨
1 file left unchanged.

> black CIME/XML/pes.py
All done! ✨ 🍰 ✨
1 file left unchanged.

@amametjanov amametjanov mentioned this pull request Oct 22, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jedwards4b jedwards4b jedwards4b left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@amametjanov @jedwards4b