mlverse / torch

R Interface to Torch
https://torch.mlverse.org
Other
483 stars 66 forks source link

Question about NEWS #1130

Closed sebffischer closed 4 months ago

sebffischer commented 5 months ago

The news mentions, that tensors are now moved to cpu in `as.array(), but there is still this line: https://github.com/mlverse/torch/blob/92a8a05a3d4963f1d740939de087ad35a822ccf9/R/tensor.R#L415

Is this intentional?

dfalbel commented 5 months ago

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