Skip to content

Upgrade torchrl and tensordict requirements #3474

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

Merged
merged 1 commit into from
Jul 22, 2025

Conversation

vmoens
Copy link
Contributor

@vmoens vmoens commented Jul 18, 2025

@svekars I think we should upgrade torchrl and tensordict - 0.7 is quite old for both!

Copy link

pytorch-bot bot commented Jul 18, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/3474

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 825c32b with merge base 9a44439 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the cla signed label Jul 18, 2025
@vmoens
Copy link
Contributor Author

vmoens commented Jul 20, 2025

Looks like tests are passing! 🥳

@svekars svekars merged commit 0ce4e16 into pytorch:fix-dqn_with_rnn_tutorial Jul 22, 2025
22 checks passed
svekars added a commit that referenced this pull request Jul 23, 2025
* Fix DQN w RNN tutorial
* bump torchrl and tensordict req (#3474)

---------

Co-authored-by: Vincent Moens <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants