-
Notifications
You must be signed in to change notification settings - Fork 297
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
Support ver 7 #193
Support ver 7 #193
Conversation
# Conflicts: # doc/support_status.md # onnx_tf/opset_version.py
Huh, so if I understand correctly, since TF conforms with NP style broadcasting, we don't need to do anything special about broadcasting in version 7 right? |
On a separate note, can you comment why we should use cls.make_node instead of the helper.make_node from ONNX (i.e., here
|
Yes, we don't need |
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.
Make sense, thanks!
I am not sure to use
_limited_broadcast
and_np_broadcast
is a good idea.They changed broadcast mechanism to as same as numpy from since version 7.
Also fix the tile version, from 1 to 6. 1 is not implemented yet.