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

JAX Integration with CleanRL #218

Closed
4 of 5 tasks
Tracked by #115
vwxyzjn opened this issue Jun 27, 2022 · 0 comments · Fixed by #225
Closed
4 of 5 tasks
Tracked by #115

JAX Integration with CleanRL #218

vwxyzjn opened this issue Jun 27, 2022 · 0 comments · Fixed by #225

Comments

@vwxyzjn
Copy link
Owner

vwxyzjn commented Jun 27, 2022

Problem Description

Given the incredible performance of the DDPG + JAX prototype (#187), it's worth prototyping JAX with other algorithms as well! This issue tracks the overall progress of integrating JAX with CleanRL.

Useful resources

Common gotchas and errors:

Useful pattern when extending

In CleanRL a filediff is incredibly helpful. For example, if I want to learn how TD3 is different from DDPG, I could do

  • open VS code and select ddpg_continuous_action.py and td3_continuous_action.py
  • right-click and left-click "compare selected"
  • the following file diff window shows up

image

Contribution process

There is a contribution checklist to help streamline the contribution process. For each new contribution, we'd need to add documentation, tests, run benchmark experiments, etc. See #186 as an example.

Tracked issues

@vwxyzjn vwxyzjn mentioned this issue Jun 27, 2022
@vwxyzjn vwxyzjn pinned this issue Jun 27, 2022
@joaogui1 joaogui1 mentioned this issue Jun 29, 2022
18 tasks
@vwxyzjn vwxyzjn mentioned this issue Jul 6, 2022
19 tasks
@vwxyzjn vwxyzjn reopened this Aug 24, 2022
@vwxyzjn vwxyzjn closed this as completed Nov 5, 2022
@vwxyzjn vwxyzjn unpinned this issue Mar 29, 2023
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 a pull request may close this issue.

1 participant