Skip to content
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

Fix mpirun.1.rst typos #12937

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jsquyres
Copy link
Member

Thanks to @satishskamath for raising the issue.

Refs #12936

@rhc54 Can you confirm the PRRTE / PRTE fixes on this PR? (restricted to the 1st commit)

Copy link
Contributor

@rhc54 rhc54 left a comment

Choose a reason for hiding this comment

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

Few minor corrections. Do you also cover PMIx params in here somewhere? I didn't see them mentioned.

docs/man-openmpi/man1/mpirun.1.rst Outdated Show resolved Hide resolved
docs/man-openmpi/man1/mpirun.1.rst Outdated Show resolved Hide resolved
docs/man-openmpi/man1/mpirun.1.rst Outdated Show resolved Hide resolved
Fix some "PRRTE" typos (where they should be "PRTE"), and add some
clarification language.

Signed-off-by: Jeff Squyres <[email protected]>
Put MPI_COMM_WORLD in fixed-width font.

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

Thanks @rhc54 -- fixed.

Describe these options and recomend that users use them when passing
PMIx and PRRTE MCA parameters, respectively.

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

Hey @rhc54 -- I added blurbs about --pmixmca and --prtemca. Do those look ok?

@rhc54
Copy link
Contributor

rhc54 commented Nov 27, 2024

Hey @rhc54 -- I added blurbs about --pmixmca and --prtemca. Do those look ok?

Definitely!! Thanks!

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.

2 participants