-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[ONNX] Extend ONNX Frontend with BitwiseOr-18 operator #21755
Conversation
Added bitwise_or.prototxt and bitwise_or_broadcast_condition.prototxt
build_jenkins |
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.
Thank you for your contribution, PR looks great!
I've left one comment regarding code style with some guides how to fix it.
I think issue is due to 2 whitespaces at the end of line |
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.
LGTM, thanks for quick fix, lets wait for CI to pass :)
build_jenkins |
Hi, wheels failed due to issue with uint64 test case OnnxBackendNodeModelTest.test_bitwise_or_ui64_bcast_3v1d_cpu Please, add it to skipped list as for bitwise_and. |
build_jenkins |
Hi! @mmikolajcz @gkrivor Is there a community forum like a discord/slack channel or a mailing list for OpenVINO? Thanks |
hey @rghvsh discord will be available after new year, we are preparing for xmas break and we might not be able to serve communication with you guys in a proper way. |
Details:
Tickets: