-
Notifications
You must be signed in to change notification settings - Fork 889
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
Documentation: run-time changes v4.1.x --> v5.0.x #10480
Comments
Moved all the to-do items to the following github issues: * open-mpi#10479 * open-mpi#10480 * open-mpi#10481 Signed-off-by: Jeff Squyres <[email protected]>
Moved all the to-do items to the following github issues: * open-mpi#10479 * open-mpi#10480 * open-mpi#10481 Signed-off-by: Jeff Squyres <[email protected]> (cherry picked from commit e54cfa7)
Working on this |
Does the "don't use single dash" command-line options apply to using '-n' in the examples in the man page? i.e. should I replace them with '--n' (or '--np') ? I've replaced '-H' with '--host'. |
Is there a definition for all of these keywords anywhere? "* |
No - the
I doubt it - you might look to see if there is a "terminology" section as that would be the most likely place to find it. |
On review, I see that the mpirun man page (https://docs.open-mpi.org/en/main/man-openmpi/man1/mpirun.1.html ) covers exit status behavior in section 19.1.2.6.19 and return value in section 19.1.2.8. These seem like they should be merged into one section (probably into 19.1.2.8, since it already has the more significant heading in the manpage. Thoughts? |
There's too many diverse actions in this meta-issue and it's going to create a terrible PR (eventually). I'm going to break them into separate issues so that they can be tackled in smaller bites. I'll apply the same tags (to start) and assign the one(s) I've been working on to myself. |
Closing this, in favor of the smaller, more focused that track individual items. |
Originally taken from the "to-do" section in the Open MPI docs:
mpirun
options, but we don’t want people to do that.srun
.The text was updated successfully, but these errors were encountered: