Skip to content
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

Add support for ReduceL1 and ReduceL2 to ONNX Importer #1838

Conversation

blesniewski
Copy link
Contributor

Description:

  • Using Reduce L1 and Reduce L2 ops in onnx importer
  • Fixed comment on Reduce L1 definition

@blesniewski blesniewski self-assigned this Aug 18, 2020
@openvino-pushbot openvino-pushbot added the category: Core OpenVINO Core (aka ngraph) label Aug 18, 2020
@blesniewski blesniewski force-pushed the use_new_reduce_ops_in_onnx_importer branch from c8979c1 to f51b3f8 Compare August 19, 2020 07:10
Copy link
Contributor

@postrational postrational left a comment

Choose a reason for hiding this comment

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

Looks good. Please verify why these 2 tests are still failing:

[gw11] [ 63%] PASSED tests/test_onnx/test_ops_reduction.py::test_reduce_l2[reduction_axes0] 
[gw13] [ 65%] PASSED tests/test_onnx/test_ops_reduction.py::test_reduce_l1[reduction_axes0] 
[gw11] [ 66%] PASSED tests/test_onnx/test_ops_reduction.py::test_reduce_l2[reduction_axes1] 
[gw13] [ 67%] PASSED tests/test_onnx/test_ops_reduction.py::test_reduce_l1[reduction_axes1] 
[gw11] [ 68%] PASSED tests/test_onnx/test_ops_reduction.py::test_reduce_l2[reduction_axes2] 
[gw13] [ 68%] PASSED tests/test_onnx/test_ops_reduction.py::test_reduce_l1[reduction_axes2] 
[gw13] [ 70%] XFAIL tests/test_onnx/test_ops_reduction.py::test_reduce_l1_default_axes 
[gw11] [ 71%] XFAIL tests/test_onnx/test_ops_reduction.py::test_reduce_l2_default_axes 

@blesniewski blesniewski force-pushed the use_new_reduce_ops_in_onnx_importer branch from f51b3f8 to 4c5797a Compare August 20, 2020 12:58
@blesniewski blesniewski force-pushed the use_new_reduce_ops_in_onnx_importer branch from 4c5797a to 7d5e4b7 Compare August 20, 2020 13:05
@blesniewski
Copy link
Contributor Author

@openvinotoolkit/openvino-maintainers - I think this is ready for merging, the two tests mentioned before were also enabled

@postrational postrational merged commit d368241 into openvinotoolkit:master Aug 27, 2020
@blesniewski blesniewski deleted the use_new_reduce_ops_in_onnx_importer branch August 28, 2020 10:08
RomanZm pushed a commit to RomanZm/openvino that referenced this pull request Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants