-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
GNA MTL mixed precision #13166
GNA MTL mixed precision #13166
Conversation
56d0d04
to
133dd1e
Compare
f99084a
to
4d70291
Compare
0aa947d
to
ec30437
Compare
cc06ef9
to
4033a7d
Compare
4033a7d
to
1d66c5e
Compare
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.
please update at least tests and unitialized variables
src/tests_deprecated/unit/engines/gna/i16_quantisation_test.cpp
Outdated
Show resolved
Hide resolved
0962c15
to
3a0a1c7
Compare
src/plugins/intel_gna/src/transformations/pwl_approximation.hpp
Outdated
Show resolved
Hide resolved
3a0a1c7
to
d8a7ace
Compare
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
src/tests_deprecated/unit/engines/gna/graph_tools/graph_copy_tests.cpp
Outdated
Show resolved
Hide resolved
@@ -26,6 +26,7 @@ class GNAPluginForPrecisionTest : public GNAPluginNS::GNAPlugin { | |||
} | |||
void set_low_precision_input() { | |||
this->gnaFlags->input_low_precision = true; | |||
this->config.gnaFlags.input_low_precision = true; |
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.
we should clean it up in future (get rid of one of this two copies).
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.
I agree, but we need to sort it out in the plugin, not in the test.
.../intel_gna/tests/functional/shared_tests_instances/single_layer_tests/conv_low_precision.cpp
Outdated
Show resolved
Hide resolved
.../intel_gna/tests/functional/shared_tests_instances/single_layer_tests/conv_low_precision.cpp
Show resolved
Hide resolved
.../intel_gna/tests/functional/shared_tests_instances/single_layer_tests/conv_low_precision.cpp
Show resolved
Hide resolved
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.
Few minor changes needed, would be better to have few smaller PR-s.
2a63203
to
86f6b5a
Compare
* [GNA] Enable int8 weights for convolutions * [GNA] Implement convolution low precision tests for MTL Co-authored-by: Marcin Kacprzak <marcin.kacprzak@intel.com>
Details:
Tickets: