Skip to content

Support Eltwise3D on OCV backend #14833

Merged
alalek merged 3 commits intoopencv:3.4from
l-bat:ocv_eltwise3d
Jun 22, 2019
Merged

Support Eltwise3D on OCV backend #14833
alalek merged 3 commits intoopencv:3.4from
l-bat:ocv_eltwise3d

Conversation

@l-bat
Copy link
Copy Markdown
Contributor

@l-bat l-bat commented Jun 19, 2019

Merge with extra: opencv/opencv_extra#621

force_builders=Custom,Custom Win,Custom Mac
build_image:Custom=ubuntu-openvino-2019r1:16.04
build_image:Custom Win=openvino-2019r1
build_image:Custom Mac=openvino-2019r1

test_modules:Custom=dnn,python2,python3,java
test_modules:Custom Win=dnn,python2,python3,java
test_modules:Custom Mac=dnn,python2,python3,java

buildworker:Custom=linux-1
test_opencl:Custom=ON
test_bigdata:Custom=1
test_filter:Custom=*

@dkurt
Copy link
Copy Markdown
Member

dkurt commented Jun 20, 2019

@l-bat, Please check IE tests: https://pullrequest.opencv.org/buildbot/builders/precommit_custom_linux/builds/2246

[ RUN      ] Test_ONNX_layers.Eltwise3D/2, where GetParam() = DLIE/OCL
unknown file: Failure
C++ exception with description "OpenCV(3.4.6-dev) /build/precommit_custom_linux/3.4/opencv/modules/dnn/src/op_inf_engine.cpp:450: error: (-215:Assertion failed) Failed to initialize Inference Engine backend: Invalid data dimensions in function 'initPlugin'
" thrown in the test body.
[  FAILED  ] Test_ONNX_layers.Eltwise3D/2, where GetParam() = DLIE/OCL (56 ms)

Looks like we can disable Eltwise layer for IE on GPU due IE supports 3D CNNs only on CPU.

Copy link
Copy Markdown
Member

@dkurt dkurt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Great! Thank you.

@dkurt dkurt self-assigned this Jun 22, 2019
@alalek alalek merged commit 1629443 into opencv:3.4 Jun 22, 2019
@alalek alalek mentioned this pull request Jun 26, 2019
arnaudbrejeon pushed a commit to arnaudbrejeon/opencv that referenced this pull request Jan 23, 2020
* Support Eltwise3d

* Refactoring

* Fix test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants