-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Adds Ray Workflow: Multiple Run Support, Distributed Hyperparameter Tuning, and Consistent Setup Across Local/Cloud #1301
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
Adds Ray Workflow: Multiple Run Support, Distributed Hyperparameter Tuning, and Consistent Setup Across Local/Cloud #1301
Conversation
Co-authored-by: James Smith <[email protected]> Signed-off-by: garylvov <[email protected]>
Co-authored-by: James Smith <[email protected]> Signed-off-by: garylvov <[email protected]>
Signed-off-by: glvov-bdai <[email protected]>
Signed-off-by: glvov-bdai <[email protected]>
Signed-off-by: glvov-bdai <[email protected]>
Signed-off-by: glvov-bdai <[email protected]>
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.
LGTM, thanks for adding!
Hi, have you tried using runtime environments to specify different pip packages, it always throw me an error saying failed to setup runtime environment because some packages could not be found. but if I login to the node and executes python, then import the package, no error occurred Does this have something to do with the PYTHONPATH variable as in |
Description
This PR adds Ray support, which enables a lot of really cool stuff by leveraging the existing Hydra support, including but not limited to:
I know this PR seems huge, but most of the code diff is config files / argparser stuff / documentation / comments
My main project at BDAI is changing from RL to LfD effective November 1st, so I'm posting this PR as early as possible so I have as much time as possible to address comments.
It would be much appreciated if the NVIDIA folks can work with me to get this reviewed ASAP. I realize that this is a pretty big PR; but I also think that it adds a lot of cool functionality, and the merging process will go much smoother if I am able to devote time to this while at work. Thanks! ;)
Fixes # (issue#1190), (issue#1213)
Type of change
Screenshots
Checklist
pre-commit
checks with./isaaclab.sh --format
config/extension.toml
fileCONTRIBUTORS.md
or my name already exists there