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

[CPU] zero-point fix #18435

Merged
merged 3 commits into from
Jul 19, 2023
Merged

[CPU] zero-point fix #18435

merged 3 commits into from
Jul 19, 2023

Conversation

eshoguli
Copy link
Contributor

@eshoguli eshoguli commented Jul 7, 2023 •
edited
Loading

Details:

  • CPU zero-point fuse fix

Tickets:

@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Jul 7, 2023
@github-actions github-actions bot added the category: IE Tests OpenVINO Test: plugins and common label Jul 14, 2023
@eshoguli eshoguli marked this pull request as ready for review July 14, 2023 18:40
@eshoguli eshoguli requested review from a team as code owners July 14, 2023 18:40
@eshoguli eshoguli force-pushed the es/cpu/conv_int_fix branch 3 times, most recently from 3537498 to 0271e6d Compare July 14, 2023 21:10
@dmitry-gorokhov dmitry-gorokhov added this to the 2023.1 milestone Jul 17, 2023
@eshoguli eshoguli force-pushed the es/cpu/conv_int_fix branch 2 times, most recently from 6467f04 to 03fda3f Compare July 18, 2023 08:43
maxnick
maxnick approved these changes Jul 18, 2023
View reviewed changes
Copy link
Contributor

@maxnick maxnick left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is one minor comment left.

src/plugins/intel_cpu/tests/functional/subgraph_tests/src/conv_with_zero_point_fuse.cpp Show resolved Hide resolved
@dmitry-gorokhov dmitry-gorokhov merged commit 38dec7b into openvinotoolkit:master Jul 19, 2023
24 checks passed
alvoron pushed a commit to alvoron/openvino that referenced this pull request Jul 24, 2023
@eshoguli @alvoron
[CPU] Fixed zero-point fusion transformation (openvinotoolkit#18435)
11aaf58 
Proper hanlding for subgraphs where ZP parent node has more than one output port
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxnick maxnick maxnick approved these changes

Assignees

@maxnick maxnick

@nshchego nshchego

Labels
category: CPU OpenVINO CPU plugin category: IE Tests OpenVINO Test: plugins and common
Projects
None yet
Milestone
2023.1
Development

Successfully merging this pull request may close these issues.
4 participants
@eshoguli @maxnick @dmitry-gorokhov @nshchego