We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Before the update to PyTorch 1.13 in b2b8eb9, the PyTorch bindings included the following functions
@Namespace("at") public static native @ByVal Tensor cat(@ByVal TensorArrayRef tensors, @Cast("int64_t") long dim/*=0*/); @Namespace("at") public static native @ByVal Tensor cat(@ByVal TensorArrayRef tensors);
as well as a few variants like cat_out, all coming from ATen/ops/cat.h, which has the following diff:
cat_out
ATen/ops/cat.h
-TORCH_API inline at::Tensor cat(at::TensorList tensors, int64_t dim=0) { +inline at::Tensor cat(const at::ITensorListRef & tensors, int64_t dim=0) { return at::_ops::cat::call(tensors, dim); }
So the type changed from TensorList to ITensorListRef which seems to be skipped in the mappings:
TensorList
ITensorListRef
javacpp-presets/pytorch/src/main/java/org/bytedeco/pytorch/presets/torch.java
Line 2369 in c0d15db
Could this be the reason why cat is not mapped anymore?
cat
The text was updated successfully, but these errors were encountered:
* Map at::ITensorListRef as used by at::cat() in presets for PyT…
at::ITensorListRef
at::cat()
10ddc85
…orch (issue #1293)
Good catch! Done in commit 10ddc85. Thanks
Sorry, something went wrong.
Thanks for adding! Could you perhaps re-trigger the CI? The windows build timed out and I think that prevented publishing the snapshots.
Works like a charm with the new snapshot. :)
val t = torch.Tensor(Seq(1,2,3)) // t: Tensor[Int32] = dtype=int32, shape=[3], device=CPU // [1, 2, 3] torch.cat(Seq(t,t)) // res0: Tensor[Int32] = dtype=int32, shape=[6], device=CPU // [1, 2, 3, 1, 2, 3]
So I think we can close this.
No branches or pull requests
Before the update to PyTorch 1.13 in b2b8eb9, the PyTorch bindings included the following functions
as well as a few variants like
cat_out
, all coming fromATen/ops/cat.h
, which has the following diff:So the type changed from
TensorList
toITensorListRef
which seems to be skipped in the mappings:javacpp-presets/pytorch/src/main/java/org/bytedeco/pytorch/presets/torch.java
Line 2369 in c0d15db
Could this be the reason why
cat
is not mapped anymore?The text was updated successfully, but these errors were encountered: