-
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
[LPT] ConvolutionBackpropData support #5313
Merged
eshoguli
merged 13 commits into
openvinotoolkit:master
from
vzinovie:vz/lpt/convolution_backprop_data
May 17, 2021
Merged
[LPT] ConvolutionBackpropData support #5313
eshoguli
merged 13 commits into
openvinotoolkit:master
from
vzinovie:vz/lpt/convolution_backprop_data
May 17, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
openvino-pushbot
added
category: CPU
OpenVINO CPU plugin
category: IE Tests
OpenVINO Test: plugins and common
category: Core
OpenVINO Core (aka ngraph)
labels
Apr 19, 2021
vzinovie
force-pushed
the
vz/lpt/convolution_backprop_data
branch
from
April 20, 2021 11:22
da25146
to
01f80cf
Compare
eshoguli
reviewed
Apr 21, 2021
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.
Partially was done. Tests are not reviewed - WIP.
inference-engine/src/low_precision_transformations/include/low_precision/network_helper.hpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/convolution.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/convolution_backprop_data.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/convolution_backprop_data.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/convolution_backprop_data.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/network_helper.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/weightable_layer_transformation.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/transformations/include/ngraph_ops/deconvolution_ie.hpp
Show resolved
Hide resolved
inference-engine/src/transformations/src/ngraph_ops/deconvolution_ie.cpp
Show resolved
Hide resolved
vzinovie
force-pushed
the
vz/lpt/convolution_backprop_data
branch
from
April 21, 2021 16:15
2e0c7f0
to
333eb89
Compare
vzinovie
added a commit
to vzinovie/openvino
that referenced
this pull request
Apr 21, 2021
vzinovie
added a commit
to vzinovie/openvino
that referenced
this pull request
Apr 21, 2021
vzinovie
force-pushed
the
vz/lpt/convolution_backprop_data
branch
from
April 21, 2021 17:45
333eb89
to
b8a2efc
Compare
vzinovie
added a commit
to vzinovie/openvino
that referenced
this pull request
Apr 21, 2021
eshoguli
reviewed
Apr 22, 2021
.../functional/inference_engine/lp_transformations/convolution_backprop_data_transformation.cpp
Show resolved
Hide resolved
...d_tests_instances/low_precision_transformations/convolution_backprop_data_transformation.cpp
Show resolved
Hide resolved
inference-engine/tests/functional/shared_test_classes/src/base/layer_test_utils.cpp
Outdated
Show resolved
Hide resolved
vzinovie
force-pushed
the
vz/lpt/convolution_backprop_data
branch
from
April 29, 2021 13:50
17e2096
to
916c8ec
Compare
eshoguli
pushed a commit
that referenced
this pull request
May 4, 2021
* [LPT] Checks to not transform layers with incorrect zero points * [LPT] Fold not transformed weights * [LPT] Minor fixes; review from #5313
vzinovie
force-pushed
the
vz/lpt/convolution_backprop_data
branch
from
May 4, 2021 17:49
324e3fd
to
4e29161
Compare
v-Golubev
reviewed
May 6, 2021
inference-engine/src/low_precision_transformations/src/convolution_backprop_data.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/convolution_backprop_data.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/convolution_backprop_data.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/low_precision_transformations/src/convolution_backprop_data.cpp
Outdated
Show resolved
Hide resolved
.../functional/inference_engine/lp_transformations/convolution_backprop_data_transformation.cpp
Show resolved
Hide resolved
inference-engine/tests/functional/shared_test_classes/src/base/layer_test_utils.cpp
Outdated
Show resolved
Hide resolved
v-Golubev
approved these changes
May 6, 2021
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.
Changes look good for me. Thanks!
vzinovie
force-pushed
the
vz/lpt/convolution_backprop_data
branch
2 times, most recently
from
May 11, 2021 13:08
1ea24a4
to
44fc0c3
Compare
vzinovie
force-pushed
the
vz/lpt/convolution_backprop_data
branch
from
May 17, 2021 08:53
44fc0c3
to
182b1c3
Compare
yekruglov
pushed a commit
to yekruglov/openvino
that referenced
this pull request
Jun 7, 2021
* [LPT] ConvolutionBackpropData support * minor fixes * [Transformations] Legacy subtract precision keep * [LPT] ConvolutionBackpropData tests improvements * [LPT] ConvolutionBackpropData weights folding when can't be transformed * [LPT] CanBeTransformed unification and convolution weights folding * [LPT] GPU INT8 optimizations condition flag * [LPT] Concat precision predict improvement * [LPT] Turn off asymmetric quantization for Deconvolution on GPU * [LPT] Improvements from review * [LPT] Check if layer after concat isQuantized and require per-tensor quantize * [LPT] Improvement for Deconv->FQ pattern * [LPT] Commented failing tests
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)
category: CPU
OpenVINO CPU plugin
category: IE Tests
OpenVINO Test: plugins and common
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
issue: 45623
validation: