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

fixed crash related to loading model with fq and sigmoid #6644

Merged
merged 2 commits into from
Jul 20, 2021
Merged

fixed crash related to loading model with fq and sigmoid #6644

merged 2 commits into from
Jul 20, 2021

Conversation

dmitriikhurtin
Copy link
Contributor

@dmitriikhurtin dmitriikhurtin commented Jul 14, 2021 •
edited
Loading

Details:

  • Fixed crash related with fusion of FQ with Sigmoid.
  • Fixed a bug with array indexes in GNAPlugin::UpdateInputScaleFromNetwork() function that led to a segmentation error.

Tickets:
CVS-58805

@dmitriikhurtin dmitriikhurtin requested review from elilobanova and a team July 14, 2021 13:30
@openvino-pushbot openvino-pushbot added category: GNA OpenVINO GNA plugin category: IE Tests OpenVINO Test: plugins and common labels Jul 14, 2021
@dmitriikhurtin
renamed multiple_input.* to multiple_input_fq.*; removed two unnecess…
69673f6 
…ary FQ layers from smoke_fq_fusion_with_sigmoid test; moved FQ params to test params
@mryzhov mryzhov merged commit e8fa1aa into openvinotoolkit:master Jul 20, 2021
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
@dmitriikhurtin
fixed crash related to loading model with fq and sigmoid (openvinotoo…
47dc594 
…lkit#6644)

* fixed crash related with loading model with fq and sigmoid

* renamed multiple_input.* to multiple_input_fq.*; removed two unnecessary FQ layers from smoke_fq_fusion_with_sigmoid test; moved FQ params to test params
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Aug 30, 2021
@dmitriikhurtin @andrei-cv
fixed crash related to loading model with fq and sigmoid (openvinotoo…
3bc74cb 
…lkit#6644)

* fixed crash related with loading model with fq and sigmoid

* renamed multiple_input.* to multiple_input_fq.*; removed two unnecessary FQ layers from smoke_fq_fusion_with_sigmoid test; moved FQ params to test params
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
@dmitriikhurtin @akuporos
fixed crash related to loading model with fq and sigmoid (openvinotoo…
82e980b 
…lkit#6644)

* fixed crash related with loading model with fq and sigmoid

* renamed multiple_input.* to multiple_input_fq.*; removed two unnecessary FQ layers from smoke_fq_fusion_with_sigmoid test; moved FQ params to test params
tadamowicz pushed a commit to tadamowicz/openvino that referenced this pull request Aug 30, 2023
@dmitriikhurtin @dood-apo
fixed crash related to loading model with fq and sigmoid (openvinotoo…
00d7554 
…lkit#6644)

* fixed crash related with loading model with fq and sigmoid

* renamed multiple_input.* to multiple_input_fq.*; removed two unnecessary FQ layers from smoke_fq_fusion_with_sigmoid test; moved FQ params to test params
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

@mryzhov mryzhov mryzhov approved these changes

Assignees
No one assigned
Labels
category: GNA OpenVINO GNA plugin category: IE Tests OpenVINO Test: plugins and common
Projects
None yet
Milestone
No milestone
Development

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