Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

FAQ: add MPI best practices #1546

Merged
merged 10 commits into from
Apr 17, 2024
Merged

FAQ: add MPI best practices #1546

merged 10 commits into from
Apr 17, 2024

Conversation

j-ogas
Copy link
Contributor

@j-ogas j-ogas commented Jan 26, 2023

Addresses: #379.

Dependent on #1449.

@j-ogas j-ogas requested review from reidpr and removed request for reidpr January 26, 2023 19:29
@j-ogas j-ogas requested a review from reidpr February 13, 2023 18:07
Copy link
Collaborator

@reidpr reidpr left a comment

Choose a reason for hiding this comment

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

Jordan, you asked if the tone is good. This is fantastic, more please!

@j-ogas j-ogas marked this pull request as ready for review April 1, 2024 19:05
@j-ogas j-ogas requested a review from reidpr April 1, 2024 19:06
@reidpr reidpr linked an issue Apr 15, 2024 that may be closed by this pull request
Copy link
Collaborator

@reidpr reidpr left a comment

Choose a reason for hiding this comment

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

Good work, nice and concise.

See my tidy and LMK if you like it.

doc/best_practices.rst Outdated Show resolved Hide resolved
doc/best_practices.rst Outdated Show resolved Hide resolved
doc/best_practices.rst Outdated Show resolved Hide resolved
doc/best_practices.rst Outdated Show resolved Hide resolved
@reidpr reidpr self-requested a review April 16, 2024 22:55
@reidpr reidpr merged commit 7386c35 into master Apr 17, 2024
6 checks passed
@reidpr reidpr deleted the faq-mpi branch April 17, 2024 20:46
kchilleri pushed a commit that referenced this pull request May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

write FAQ on MPI best practices
2 participants