-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Enable opt-in autodetection of distributed configuration for mpi4py, attempt 2 #22235
base: main
Are you sure you want to change the base?
Conversation
…tion of MPI, as long as mpi4py is available
Remove unneeded commented code.
Further clean up unneeded comments.
Clean up documentation slightly.
Sorry JAX team - I made an inadvertent force push and don't want to leave that PR as the only option. I'm going to suggest you reject #20174 and if you have the ability to click |
It's hard for us to squash as part of our merge, unfortunately, which is why we ask that you do it. (For us to do it, I think we have to push to your branch, which usually annoys people.) |
Squashing commits from #20174