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

Speedup tensordicts #124

Merged
merged 19 commits into from
May 13, 2022
Merged

Speedup tensordicts #124

merged 19 commits into from
May 13, 2022

Conversation

vmoens
Copy link
Contributor

@vmoens vmoens commented May 13, 2022

Refactors a few operations in tensordict.set, tensordict.set_ and attribute gathering to make tensordict more efficient.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 13, 2022
@vmoens vmoens added the performance Performance issue or suggestion for improvement label May 13, 2022
@vmoens vmoens merged commit 4be4246 into main May 13, 2022
@vmoens vmoens deleted the speedup_td branch May 13, 2022 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. performance Performance issue or suggestion for improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants