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

Clean dependencies #1

Merged
merged 10 commits into from
Oct 8, 2024
Merged

Clean dependencies #1

merged 10 commits into from
Oct 8, 2024

Conversation

takuma-sony
Copy link
Collaborator

For OSS release, this PR cleans dependencies as well as minor changes:

  • Minimized dependencies.
  • Move Dockerfile to top-level and refactored codes.
  • Remove unncessary files, nvidia_icd.json and mjkey.txt.
  • Add Apache 2.0 license
  • Allow users to execute codes without humanoid_bench
  • Add dev.requirements.txt
  • Add setup.py
  • Update README.md

@takuma-sony takuma-sony requested a review from joonleesky October 7, 2024 00:37
```
RUN pip install -U "jax[cuda12]==0.4.25" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html
pip install -U "jax[cuda12]==0.4.25" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html
Copy link
Contributor

Choose a reason for hiding this comment

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

To allow multiple JAX runs in a single gpu.

export XLA_PYTHON_CLIENT_PREALLOCATE=false 

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done.

@@ -9,239 +9,25 @@ dependencies:
- pip=21.0
- python=3.9.0
- pip:
- absl-py==2.0.0
Copy link
Contributor

Choose a reason for hiding this comment

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

can you change the name of conda file to either scale_rl or simba?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done.

@joonleesky
Copy link
Contributor

Everything seems to be working fine on my desktop, but the removed dependency on our lab's server isn't functioning properly. However, it's okay since our lab server's configuration is a bit unusual, and we're planning to upgrade to CUDA 12 starting in November.

@takuma-sony
Copy link
Collaborator Author

Thanks for the check. I'll go ahead merge this.

@takuma-sony takuma-sony merged commit 4df53a0 into master Oct 8, 2024
1 check failed
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.

2 participants