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.

Sign up for GitHub

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

Jump to bottom

Question about NEWS #1130

Closed

sebffischer opened this issue Jan 25, 2024 · 1 comment

Collaborator

sebffischer commented Jan 25, 2024 •

edited

Loading

The news mentions, that tensors are now moved to cpu in `as.array(), but there is still this line:

torch/R/tensor.R

Line 415 in 92a8a05

if (x$device$type == "cuda") {

Is this intentional?

Member

dfalbel commented Jan 25, 2024

Ohhh this not intentional at all. We should remove that so we don't raise the error any longer.

sebffischer mentioned this issue

fix(tensor): cuda to r conversion #1132

Merged

sebffischer closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment