-
Notifications
You must be signed in to change notification settings - Fork 102
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
Don't take Tensor by value #38
Comments
zou3519
pushed a commit
that referenced
this issue
May 26, 2021
zou3519
pushed a commit
that referenced
this issue
May 26, 2021
zou3519
pushed a commit
that referenced
this issue
May 26, 2021
zou3519
pushed a commit
to zou3519/pytorch
that referenced
this issue
Jul 20, 2022
mikekgfb
pushed a commit
to mikekgfb/pytorch
that referenced
this issue
Jul 21, 2022
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Better to take it by reference, e.g., here
https://github.com/zou3519/functorch/blob/68df44f7cdb72864d5e43674357bca7a57b81220/functorch/csrc/PythonKey.cpp#L28
The text was updated successfully, but these errors were encountered: