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] New flags to select GNA target generation #5429

Merged
merged 8 commits into from
May 7, 2021
Merged

[GNA] New flags to select GNA target generation #5429

merged 8 commits into from
May 7, 2021

Conversation

kbruniec
Copy link
Contributor

@kbruniec kbruniec commented Apr 27, 2021 •
edited
Loading

Details:

  • Implement mechanism for GNA2 and GNA3 to coexist in OpenVINO
  • Add unit tests
  • Select consistent device (eg. in GNA_SW_EXACT mode) based on EXECUTION TARGET

Tickets:

  • 53714

@kbruniec kbruniec requested a review from a team April 27, 2021 14:06
@openvino-pushbot openvino-pushbot added the category: GNA OpenVINO GNA plugin label Apr 27, 2021
@kbruniec kbruniec force-pushed the gna-new-parameters-for-compile-and-exec-mode branch from 48f949d to ee9c6b1 Compare April 27, 2021 15:38
@kbruniec kbruniec requested a review from a team April 27, 2021 15:38
@kbruniec kbruniec force-pushed the gna-new-parameters-for-compile-and-exec-mode branch from ee9c6b1 to 9dca402 Compare April 29, 2021 07:23
@kbruniec kbruniec requested review from a team, dorloff, sirzabek and p-rozen April 29, 2021 07:23
@kbruniec kbruniec force-pushed the gna-new-parameters-for-compile-and-exec-mode branch from 8501311 to c92dd30 Compare April 30, 2021 10:18
@kbruniec kbruniec force-pushed the gna-new-parameters-for-compile-and-exec-mode branch from c92dd30 to e6e6536 Compare May 5, 2021 14:06
sirzabek
sirzabek reviewed May 6, 2021
View reviewed changes
inference-engine/include/gna/gna_config.hpp Outdated Show resolved Hide resolved
inference-engine/include/gna/gna_config.hpp Outdated Show resolved Hide resolved
inference-engine/src/gna_plugin/gna_device.cpp Outdated Show resolved Hide resolved
inference-engine/src/gna_plugin/gna_device.cpp Outdated Show resolved Hide resolved
inference-engine/samples/speech_sample/speech_sample.hpp Outdated Show resolved Hide resolved
inference-engine/src/gna_plugin/gna_device.cpp Show resolved Hide resolved
@dorloff dorloff merged commit 7c07c61 into openvinotoolkit:master May 7, 2021
tadamowicz pushed a commit to tadamowicz/openvino that referenced this pull request Aug 30, 2023
@kbruniec @dood-apo
[GNA] New flags to select GNA target generation (openvinotoolkit#5429)
a16f9bd 
* New flags for GNA trget

* Add new flags to configuration

* Update utest

* Update speech_Sample

* Add unit tests

* Guard for wrong values

* Use gna execution target as consistent device

* Apply review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sirzabek sirzabek sirzabek approved these changes

@nkogteva nkogteva nkogteva approved these changes

@dorloff dorloff Awaiting requested review from dorloff

@p-rozen p-rozen Awaiting requested review from p-rozen

Assignees
No one assigned
Labels
category: GNA OpenVINO GNA plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kbruniec @nkogteva @sirzabek @openvino-pushbot @dorloff