-
Notifications
You must be signed in to change notification settings - Fork 420
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
Merge operation accepts only single input #345
Comments
ref #325 |
It looks like the problem is that we're ignoring |
@adamcrume |
This wouldn't be quite that straightforward. The code for the ops is generated by |
Does the below build_instance method close this? https://tensorflow.github.io/rust/tensorflow/ops/struct.Merge.html |
The Merge operation (https://www.tensorflow.org/api_docs/cc/class/tensorflow/ops/merge) should accept a list of inputs, and output the first received tensor.
However, the rust binding (https://tensorflow.github.io/rust/tensorflow/ops/fn.merge.html) currently accepts only a single input (despite the input variable being named "inputs"). Digging through the source code, I found that
Merge.build_impl
internally also callsnd.add_input
instead ofnd.add_inputs
(https://tensorflow.github.io/rust/src/tensorflow/ops/ops_impl.rs.html#57909)Is there a way to pass multiple inputs using this api?
The text was updated successfully, but these errors were encountered: