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

jaxdecomp proto #21

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

jaxdecomp proto #21

wants to merge 114 commits into from

Conversation

ASKabalan
Copy link
Collaborator

I am going to continue here the distributed PM

@ASKabalan ASKabalan marked this pull request as draft July 9, 2024 22:27
jaxpm/distributed.py Outdated Show resolved Hide resolved
scripts/distributed_pm.py Outdated Show resolved Hide resolved
@@ -0,0 +1,167 @@
#!/bin/bash
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This file should not stay in the repo in my opinion

@ASKabalan ASKabalan force-pushed the ASKabalan/jaxdecomp_proto branch from b39627f to b32014b Compare December 5, 2024 17:21
@EiffL
Copy link
Member

EiffL commented Dec 6, 2024

@all-contributors please add @ASKabalan for code and infrastructure, @aboucaud for review, @hsimonfroy for code

Copy link
Contributor

@EiffL

I've put up a pull request to add @ASKabalan! 🎉

@ASKabalan
Copy link
Collaborator Author

@all-contributors please add @aboucaud for review

Copy link
Contributor

@ASKabalan

I've put up a pull request to add @aboucaud! 🎉

@ASKabalan
Copy link
Collaborator Author

@EiffL I just noticed that the all contributer bot did not add Hugo and Alex
I added Alex

@ASKabalan
Copy link
Collaborator Author

Hello @EiffL

The test workflow is passing

Everything is good to go

This PR is getting too huge and I would like to merge it
And in a subsequent PR I will work on docstrings/Mypi and publishing to PyPi,

Can you please merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants