-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Fix - Port TorchRL Pendulum tutorial from pytorch.org/rl to pytorch.org/tutorials #2653
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
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/2653
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 6c025c6 with merge base a668406 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Quick question @svekars, Can you please let me know where the static images need to be placed?. |
You can put under |
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.
Some suggestions. Make sure that the formatting looks good in the PR preview. Also, please fix the spellcheck.
Co-authored-by: Svetlana Karslioglu <[email protected]>
Co-authored-by: Svetlana Karslioglu <[email protected]>
Co-authored-by: Svetlana Karslioglu <[email protected]>
Co-authored-by: Svetlana Karslioglu <[email protected]>
Co-authored-by: Svetlana Karslioglu <[email protected]>
Co-authored-by: Svetlana Karslioglu <[email protected]>
Co-authored-by: Svetlana Karslioglu <[email protected]>
please maker sure to fix the spellcheck |
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 doing this!
Still need to fix spellcheck and resolve the merge conflict. Otherwise, looks good. |
Fixes #2352
Description
Initial changes made to port https://github.com/pytorch/rl/blob/main/tutorials/sphinx-tutorials/pendulum.py
cc @vmoens @nairbv @sekyondaMeta @svekars @carljparker @NicolasHug @kit1980 @subramen