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.

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

add Pad12 support to TSDataMovementForward/Backward transformations #18314

Merged
merged 3 commits into from
Jul 5, 2023

Conversation

evkotov
Copy link
Contributor

@evkotov evkotov commented Jun 30, 2023

Details:

  • update transformations to support new v12::Pad
  • add unit test

Tickets:

  • 112279

@evkotov evkotov added the category: transformations OpenVINO Runtime library - Transformations label Jun 30, 2023
@evkotov evkotov added this to the 2023.1 milestone Jun 30, 2023
@evkotov evkotov self-assigned this Jun 30, 2023
@evkotov evkotov requested a review from a team as a code owner June 30, 2023 11:42
@evkotov evkotov requested a review from t-jankowski July 4, 2023 11:09
Comment on lines +117 to +120
return std::make_shared<ov::op::v1::Pad>(parent_nodes[0],
parent_nodes[1],
parent_nodes[2],
ov::op::PadMode::CONSTANT);
Copy link
Contributor

Choose a reason for hiding this comment

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

There are using namespace std; and using namespace ov; at the front of the file (applies elsewhere std:: or ov:: are used)

Copy link
Contributor

Choose a reason for hiding this comment

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

let's change this in the next PR

@itikhono itikhono enabled auto-merge (squash) July 5, 2023 06:53
@itikhono itikhono merged commit 6a0c6a1 into openvinotoolkit:master Jul 5, 2023
@evkotov evkotov deleted the pad12_ts_data_movement branch July 5, 2023 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: transformations OpenVINO Runtime library - Transformations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants