Skip to content

Wait/test any/some/all leaves out single request case #985

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

Open
wgropp opened this issue Mar 29, 2025 · 0 comments
Open

Wait/test any/some/all leaves out single request case #985

wgropp opened this issue Mar 29, 2025 · 0 comments
Labels
editor change Changes to be made by the document editor mpi-5.0 For inclusion in the MPI 5.0 standard scheduled no-no vote No-No vote is scheduled for the next meeting scheduled reading Reading is scheduled for the next meeting
Milestone

Comments

@wgropp
Copy link

wgropp commented Mar 29, 2025

Problem

p 519 uses MPI_WAIT{ANY|SOME|ALL} but to be consistent with the other use of this style, should be MPI_WAIT{|ANY|SOME|ALL} (which is used elsewhere)
Also true on p 631, lines 8, 37, 43. Also p 635, lines 41 and 45. As written, this says that MPI_WAIT and MPI_TEST are not included in this text, which is not the intent.
Note the first use of this construction on p 519 is explicitly for arrays of requests, in which case MPI_{WAIT|TEST}{ANY|SOME|ALL} (without the "|" before ANY) is correct.

Proposal

Add this missing "|" before the "ANY" where needed (as noted, one spot on p519 refers only to wait or test with an array of requests, and is correct without the "|" before ANY).

Changes to the Text

As described.

Impact on Implementations

None.

Impact on Users

Improves quality of the document

References and Pull Requests

See PR mpi-forum/mpi-standard/pull/1117

@wgropp wgropp added the mpi-5.0 For inclusion in the MPI 5.0 standard label Mar 29, 2025
@github-project-automation github-project-automation bot moved this to To Do in MPI 5.0 Mar 29, 2025
@wgropp wgropp added the editor change Changes to be made by the document editor label Mar 29, 2025
@wesbland wesbland added the scheduled reading Reading is scheduled for the next meeting label Apr 14, 2025
@wesbland wesbland moved this from To Do to In Progress in MPI 5.0 Apr 14, 2025
@wesbland wesbland added this to the June 2025 milestone Apr 14, 2025
@wesbland wesbland added the scheduled no-no vote No-No vote is scheduled for the next meeting label Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor change Changes to be made by the document editor mpi-5.0 For inclusion in the MPI 5.0 standard scheduled no-no vote No-No vote is scheduled for the next meeting scheduled reading Reading is scheduled for the next meeting
Projects
Status: In Progress
Development

No branches or pull requests

2 participants