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.

Sign up for GitHub

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

Jump to bottom

[GNA] Set input scale factors for imported model (#7139) #7140

Merged
merged 4 commits into from
Aug 19, 2021
Merged

[GNA] Set input scale factors for imported model (#7139) #7140

merged 4 commits into from
Aug 19, 2021

Conversation

mryzhov
Copy link
Contributor

@mryzhov mryzhov commented Aug 18, 2021 •
edited
Loading

Merge #7139

Details:

  • Those changes allow GNA customers to correct the input scale factor for the old models (previously exported) using speech sample, because they don't have the latest version IR.
  • If scale factors for inputs were defined in the plugin configuration we should change them for the imported model
  • Ignoring default scale factors (1.0f), they should be replaced my imported values
  • Modified tests to cover all cases.

Tickets:

  • 62964

mryzhov added 2 commits August 18, 2021 14:17
@mryzhov
[GNA] Set custom scale factor for imported model
0911e6c 
# Conflicts:
#	inference-engine/samples/speech_sample/main.cpp
@mryzhov
Added tests
facb029 
# Conflicts:
#	inference-engine/tests/functional/plugin/gna/shared_tests_instances/import_export_tests/import_reshape_permute_conv.cpp
@mryzhov mryzhov added the category: GNA OpenVINO GNA plugin label Aug 18, 2021
@mryzhov mryzhov added this to the 2021.4.1 milestone Aug 18, 2021
@mryzhov mryzhov requested a review from a team August 18, 2021 11:28
@mryzhov mryzhov self-assigned this Aug 18, 2021
@mryzhov mryzhov requested review from a team August 18, 2021 11:28
@openvino-pushbot openvino-pushbot added category: inference OpenVINO Runtime library - Inference category: IE Tests OpenVINO Test: plugins and common labels Aug 18, 2021
@mryzhov mryzhov merged commit 4714b8e into openvinotoolkit:releases/2021/4 Aug 19, 2021
@mryzhov mryzhov deleted the merge/7139 branch December 8, 2021 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elilobanova elilobanova elilobanova approved these changes

Assignees

@mryzhov mryzhov

Labels
category: GNA OpenVINO GNA plugin category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference
Projects
None yet
Milestone
2021.4.1
Development

Successfully merging this pull request may close these issues.
3 participants
@mryzhov @elilobanova @openvino-pushbot