Skip to content

doc: Add MPI-4.0 partial compliance features #11498

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

Merged
merged 3 commits into from
Mar 28, 2023
Merged

Conversation

wckzhang
Copy link
Contributor

No description provided.

@jjhursey
Copy link
Member

bot:ibm:retest

@jsquyres
Copy link
Member

@wzamazon Can you double check your list against https://github.com/open-mpi/ompi/projects/2?

@wckzhang
Copy link
Contributor Author

Aye, I'll cross compare

@wckzhang
Copy link
Contributor Author

also @jsquyres wzamazon is Wei, I'm wckzhang

@jsquyres
Copy link
Member

also @jsquyres wzamazon is Wei, I'm wckzhang

☹️ My bad. Sorry about that!

@wckzhang
Copy link
Contributor Author

@jsquyres updated

@jsquyres
Copy link
Member

jsquyres commented Mar 28, 2023

@wckzhang I made a few suggestions in a trailing commit on this PR. If you accept them, squash that commit into your commit.

In making those suggestions, I found some other errors regarding the MPI standard major/minor version rendering, so I pre-pended some commits on this PR to fix that, too. No need to squash those -- just wanted to make you aware of why they're here.

jsquyres and others added 3 commits March 28, 2023 12:19
RST apparently does not render |foo|.|bar| properly.  Specifically:
the "foo" macro will render properly, but the "bar" macro will not.

As such, split the MPI standard version into three macros so that we
don't have to render MPI's major and minor versions in the pattern
that RST will not render correctly.

Also add a cross reference from MPI_Get_version.3 to the release notes
where we talk about Open MPI's MPI conformance.

Signed-off-by: Jeff Squyres <[email protected]>
@wckzhang
Copy link
Contributor Author

Squashed it, thanks Jeff

@jsquyres jsquyres merged commit 5725bdd into open-mpi:main Mar 28, 2023
@jsquyres
Copy link
Member

Thanks @wckzhang. Can you cherry pick to v5.0.x?

@wckzhang
Copy link
Contributor Author

done - #11539

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants