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] Remove custom shape inference factories #27924

Merged
merged 22 commits into from
Jan 20, 2025
Merged

[cpu] Remove custom shape inference factories #27924

merged 22 commits into from
Jan 20, 2025

Conversation

praasz
Copy link
Contributor

@praasz praasz commented Dec 5, 2024

Details:

  • Remove custom shape inference factories CPU nodes.

Related PR

Tickets:

@praasz praasz added this to the 2025.0 milestone Dec 5, 2024
@praasz praasz requested review from a team as code owners December 5, 2024 08:33
@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Dec 5, 2024
praasz added 2 commits December 5, 2024 13:34
@praasz
Update FakeQunatize shape inference to return dedicated port mask
1499658 
Signed-off-by: Raasz, Pawel <[email protected]>
@praasz
Fallback shape inference use full port mask
da7e5a8 
to inputs data as constant if available

Signed-off-by: Raasz, Pawel <[email protected]>
@praasz praasz force-pushed the shape-infer/remove-cpu-custom-shape-infer-factories branch from 14ee211 to 75dfef5 Compare December 9, 2024 10:38
@praasz praasz requested a review from maxnick December 16, 2024 10:54
@mlukasze
Copy link
Contributor

@maxnick could you take a look again, please?

@praasz praasz added the no_stale Do not mark as stale label Dec 20, 2024
@mlukasze
Copy link
Contributor

mlukasze commented Jan 8, 2025

@maxnick could you check it again, please?

@mlukasze mlukasze removed this from the 2025.0 milestone Jan 10, 2025
@maxnick maxnick added this to the 2025.1 milestone Jan 14, 2025
@maxnick
Copy link
Contributor

maxnick commented Jan 16, 2025

@mlukasze , now can we merge this PR?

@mlukasze mlukasze added this pull request to the merge queue Jan 17, 2025
@mlukasze
Copy link
Contributor

yes! :)

@mlukasze mlukasze removed this pull request from the merge queue due to a manual request Jan 17, 2025
@mlukasze mlukasze enabled auto-merge January 17, 2025 06:28
@mlukasze mlukasze disabled auto-merge January 17, 2025 08:05
@mlukasze mlukasze enabled auto-merge January 17, 2025 08:05
@mlukasze
Copy link
Contributor

please, resolve conflicts

@praasz praasz requested a review from a team as a code owner January 17, 2025 18:44
@github-actions github-actions bot added the category: TF FE OpenVINO TensorFlow FrontEnd label Jan 17, 2025
@praasz praasz removed the no_stale Do not mark as stale label Jan 20, 2025
praasz
praasz commented Jan 20, 2025
View reviewed changes
src/frontends/tensorflow/src/frontend.cpp Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This part was changed by clang-format.

@mlukasze mlukasze added this pull request to the merge queue Jan 20, 2025
Merged via the queue into openvinotoolkit:master with commit 0fce5f3 Jan 20, 2025
184 checks passed
@praasz praasz deleted the shape-infer/remove-cpu-custom-shape-infer-factories branch January 21, 2025 05:41
MirceaDan99 pushed a commit to MirceaDan99/openvino that referenced this pull request Jan 22, 2025
@praasz @mlukasze
@maxnick @MirceaDan99
[cpu] Remove custom shape inference factories (openvinotoolkit#27924)
87235ee 
### Details:
 - Remove custom shape inference factories CPU nodes.

### Related PR
- openvinotoolkit#27770 

### Tickets:
 -  CVS-118704

---------

Signed-off-by: Raasz, Pawel <[email protected]>
Co-authored-by: Michal Lukaszewski <[email protected]>
Co-authored-by: Maksim Kutakov <[email protected]>
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

Labels
category: CPU OpenVINO CPU plugin category: TF FE OpenVINO TensorFlow FrontEnd
Projects
None yet
Milestone
2025.1
Development

Successfully merging this pull request may close these issues.
3 participants
@praasz @mlukasze @maxnick