Adam Pocock
Adam Pocock
Can we get this integrated now the 1.11 release has happened? I can rebase it on master if necessary, then it'll be easier to work on the native binding improvements...
> > Can we get this integrated now the 1.11 release has happened? I can rebase it on master if necessary, then it'll be easier to work on the native...
Is it possible to get this branch merged in? I've fixed the merge conflict, and the JNI rewrite I've been doing started from this branch.
The linter seems to be treating my C code as C++ code, as it's complaining I'm using C style casts. Aside from that issue, is there a code formatter configuration...
I've started work on updating this branch with the new error handling code, but it's not ready for review yet.
This is ready for review now, I've finished updating it with the new JNI error handling.
This should be ready for review again.
@yuslepukhin would you mind reviewing this again?
> Copying or not, we want to make sure that the output tensors are deallocated when no longer needed. > > In reply to: [1054694679](https://github.com/microsoft/onnxruntime/pull/10653#issuecomment-1054694679) `OnnxSparseTensor` is `AutoCloseable`, and deallocates...
The Tensorflow Java API in this repo isn't maintained. The maintained version of TF-Java lives in https://github.com/tensorflow/java.