Skip to content

Tighten MPI rank checks in Ethier tests #3

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
Merged
yslan merged 1 commit into from
Dec 11, 2025
Merged

Tighten MPI rank checks in Ethier tests #3

yslan merged 1 commit into from
Dec 11, 2025

Conversation

@yslan
Copy link
Owner

@yslan yslan commented Dec 11, 2025

Summary

  • convert the parsed MPI rank count in Ethier tests to an integer
  • use a strict equality assertion for the recorded number of ranks

Testing

  • Not run (not requested)

Codex Task

@yslan yslan merged commit 9abba45 into fix_ci_io_test Dec 11, 2025
26 checks passed
@yslan yslan deleted the codex/review-pr-#872-for-nek5000 branch December 11, 2025 18:59
yslan added a commit that referenced this pull request Dec 12, 2025
@yslan
Fix CI (Nek5000#872)
9ceaf2a 
* Add FFLAGS=-mcmodel=medium for IO_Test

* Fix uninitialized prefix for IO_Test

* Update deprecated SafeConfigParser

Since python3.12, Use ConfigParser

* Add genmap for io_test_rs

* Add implicit into SIZE.legacy and makenek.inc

* verbose err for #rank chk

* git workflow: Fall back to ubuntu2204, add np check in Ethier

* Tighten MPI rank checks in Ethier's np tests (#3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

codex

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yslan