-
Notifications
You must be signed in to change notification settings - Fork 902
More docs updates #10762
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
More docs updates #10762
Conversation
248c62f
to
e4d9833
Compare
e4d9833
to
5827eef
Compare
Pushed a few text tweaks and spelling corrections this morning. @awlauria this is ready to review. See https://ompi--10762.org.readthedocs.build/en/10762/running-apps/tuning.html for the bulk of the new material in this PR. |
5827eef
to
6e27a78
Compare
@jjhursey Updated. The bulk of the changes are visible here: https://ompi--10762.org.readthedocs.build/en/10762/running-apps/tuning.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Trivial change: put the explanation text *before* the example. Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
This file was a placeholder and did not contain any useful content, anyway. Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
Moved a bunch of MCA-related content out of faq/general-tuning to a new file in running-apps/tuning.rst. Removed AMCA documentation because that functionality has morphed into the "--tuned" functionality. Indeed, the --am CLI parameter is silently converted to --tuned. Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
6e27a78
to
6acc2c2
Compare
Had to force push the same commits (rebased to change their hashes) to un-stick the AWS Jenkins on this PR. |
See individual commit messages for details.
NOTE: To see the new "tuning" docs page that contains material ported/updated/converted from the FAQ, see https://ompi--10762.org.readthedocs.build/en/10762/running-apps/tuning.html