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

Remove ngraph::helper #22704

Merged
merged 9 commits into from
Feb 23, 2024
Merged

Remove ngraph::helper #22704

merged 9 commits into from
Feb 23, 2024

Conversation

vurusovs
Copy link
Contributor

@vurusovs vurusovs commented Feb 7, 2024 •
edited
Loading

Details:

  • Port required functions to common_test_utils/test_helpers.*pp
  • Delete ov_models target

Tickets:

@vurusovs vurusovs requested review from a team as code owners February 7, 2024 12:22
@github-actions github-actions bot added category: IE Tests OpenVINO Test: plugins and common category: GPU OpenVINO GPU plugin category: CPU OpenVINO CPU plugin category: transformations OpenVINO Runtime library - Transformations category: AUTO OpenVINO AUTO device selection plugin labels Feb 7, 2024
@vurusovs vurusovs force-pushed the vu/ngraph_helpers branch 2 times, most recently from 7e74f1c to 787f5de Compare February 8, 2024 07:03
@vurusovs
Copy link
Contributor Author

vurusovs commented Feb 8, 2024 •
edited
Loading

Blocked by openvinotoolkit/openvino_contrib#870

@vurusovs vurusovs force-pushed the vu/ngraph_helpers branch 2 times, most recently from ed05996 to c494e07 Compare February 20, 2024 10:52
@github-actions github-actions bot removed the category: transformations OpenVINO Runtime library - Transformations label Feb 20, 2024
@vurusovs vurusovs added this to the 2024.1 milestone Feb 20, 2024
@ilya-lavrenov ilya-lavrenov removed their assignment Feb 20, 2024
@p-durandin
Copy link
Contributor

@vurusovs please create PR for NPU to remove helpers from
../npu/tests/functional/behavior/elf_config.cpp

@vurusovs vurusovs mentioned this pull request Feb 21, 2024
Merged
@vurusovs vurusovs requested review from a team as code owners February 21, 2024 13:38
@github-actions github-actions bot added category: HETERO OpenVINO HETERO plugin category: build OpenVINO cmake script / infra category: AUTO BATCH OpenVINO Auto Batch plugin labels Feb 21, 2024
@github-actions github-actions bot added the category: transformations OpenVINO Runtime library - Transformations label Feb 21, 2024
@p-durandin p-durandin enabled auto-merge February 22, 2024 14:20
@p-durandin p-durandin added this pull request to the merge queue Feb 23, 2024
Merged via the queue into openvinotoolkit:master with commit d58efea Feb 23, 2024
104 checks passed
ilya-lavrenov added a commit that referenced this pull request Feb 23, 2024
@ilya-lavrenov
Revert "Remove ngraph::helper (#22704)"
a42bb80 
This reverts commit d58efea.
@ilya-lavrenov ilya-lavrenov mentioned this pull request Feb 23, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@praasz praasz praasz approved these changes

@p-durandin p-durandin p-durandin approved these changes

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

Assignees

@olpipi olpipi

Labels
category: AUTO BATCH OpenVINO Auto Batch plugin category: AUTO OpenVINO AUTO device selection plugin category: build OpenVINO cmake script / infra category: CPU OpenVINO CPU plugin category: GPU OpenVINO GPU plugin category: HETERO OpenVINO HETERO plugin category: IE Tests OpenVINO Test: plugins and common category: transformations OpenVINO Runtime library - Transformations
Projects
None yet
Milestone
2024.1
Development

Successfully merging this pull request may close these issues.
5 participants
@vurusovs @p-durandin @ilya-lavrenov @praasz @olpipi