-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Not only TensorType for Intput-Output params #102
Conversation
This is breaking changes for |
0e6e712
to
2657440
Compare
Well, I think it can be related to: #30 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great start for supporting sequences & maps! I'd like to merge this into the v2 branch since it is a breaking API change, could you port these changes over to there? I think it would require creating a new PR.
Yes, I will do it. |
2657440
to
fedf929
Compare
@decahedron1 I rebased from v2 branch) |
fedf929
to
7d89f9c
Compare
Hello! When I tried to create
Session
I got error:I made small research and find java bindings for onnxruntime: https://github.com/microsoft/onnxruntime/blob/6832b688a2848f64d4d43ac0084c6609aa0203bc/java/src/main/native/OrtJniUtil.c.
I made something close to the implementation.