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

refactoring get_cpu_pinning() #22489

Merged
merged 16 commits into from
Feb 23, 2024
Merged

refactoring get_cpu_pinning() #22489

merged 16 commits into from
Feb 23, 2024

Conversation

wangleis
Copy link
Contributor

@wangleis wangleis commented Jan 29, 2024 •
edited
Loading

Details:

  • add ov::affinity deprecation in CPU plugin introduction
  • refactoring get_cpu_pinning()
  • Default CPU pinning on Windows is false on all platform

Tickets:

@wangleis wangleis requested a review from a team as a code owner January 29, 2024 07:04
@wangleis wangleis requested review from akopytko and removed request for a team January 29, 2024 07:04
@github-actions github-actions bot added the category: docs OpenVINO documentation label Jan 29, 2024
@wangleis wangleis changed the title add ov::affinity deprecation in CPU plugin introduction {DOC] add ov::affinity deprecation in CPU plugin introduction Jan 29, 2024
@wangleis wangleis changed the title {DOC] add ov::affinity deprecation in CPU plugin introduction [DOC] add ov::affinity deprecation in CPU plugin introduction Jan 29, 2024
@wangleis wangleis changed the title [DOC] add ov::affinity deprecation in CPU plugin introduction [DOC] add ov::affinity deprecation info in CPU plugin introduction Jan 29, 2024
@sunxiaoxia2022 sunxiaoxia2022 mentioned this pull request Jan 29, 2024
Merged
@ilya-lavrenov ilya-lavrenov added this to the 2024.0 milestone Jan 29, 2024
@wangleis wangleis requested review from a team as code owners February 2, 2024 08:53
@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Feb 2, 2024
@wangleis wangleis force-pushed the doc_affinity_deprecation branch from 25d3d65 to 3ec8c18 Compare February 4, 2024 06:13
@wangleis wangleis changed the title [DOC] add ov::affinity deprecation info in CPU plugin introduction refactoring get_cpu_pinning() Feb 4, 2024
@wangleis wangleis force-pushed the doc_affinity_deprecation branch from 5e48f82 to 8548eb5 Compare February 4, 2024 09:18
@wangleis
Copy link
Contributor Author

wangleis commented Feb 4, 2024

@dmitry-gorokhov @sunxiaoxia2022 @xufang-lisa Refactoring get_cpu_pinning() in this PR. Please review again.

@wangleis wangleis requested a review from a team as a code owner February 19, 2024 23:48
@github-actions github-actions bot added category: IE Tests OpenVINO Test: plugins and common category: CI OpenVINO public CI category: dependency_changes Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Feb 19, 2024
@wangleis wangleis force-pushed the doc_affinity_deprecation branch from 92e4914 to 8e71e58 Compare February 20, 2024 01:29
@github-actions github-actions bot removed category: IE Tests OpenVINO Test: plugins and common category: CI OpenVINO public CI category: dependency_changes Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Feb 20, 2024
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Feb 20, 2024
@wangleis wangleis requested a review from a team as a code owner February 21, 2024 05:58
@github-actions github-actions bot added the category: IE Tests OpenVINO Test: plugins and common label Feb 21, 2024
@wangleis wangleis added this pull request to the merge queue Feb 23, 2024
Merged via the queue into openvinotoolkit:master with commit 1686e03 Feb 23, 2024
104 checks passed
@wangleis wangleis deleted the doc_affinity_deprecation branch February 23, 2024 15:09
@dmitry-gorokhov
Copy link
Contributor

@wangleis This PR increases Win tests pass time more than 2 times. This is unacceptable. Please revert the PR

@wangleis wangleis mentioned this pull request Feb 28, 2024
Merged
github-merge-queue bot pushed a commit that referenced this pull request Feb 29, 2024
@wangleis
refactor cpu pinning (#23099)
9e759e4 
### Details:
- *Porting [PR
22489](#22489
 -- *add ov::affinity deprecation in CPU plugin introduction*
 -- *refactoring get_cpu_pinning()*
 -- *Default CPU pinning on Windows is false on all platform*
- *Enable CPU pinning in part of CPU functional test cases. Now Intel
CPU plugin func tests on Windows finish at 0h:27m:32.21s in this PR and
0h:30m:50.99s in [PR
23129](#23129

### Tickets:
 - *CVS-129030*
@wangleis wangleis mentioned this pull request Feb 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sunxiaoxia2022 sunxiaoxia2022 sunxiaoxia2022 approved these changes

@xufang-lisa xufang-lisa xufang-lisa approved these changes

@dmitry-gorokhov dmitry-gorokhov dmitry-gorokhov approved these changes

@akopytko akopytko Awaiting requested review from akopytko akopytko is a code owner automatically assigned from openvinotoolkit/openvino-docs-maintainers

@ilya-lavrenov ilya-lavrenov Awaiting requested review from ilya-lavrenov

@peterchen-intel peterchen-intel Awaiting requested review from peterchen-intel

@kblaszczak-intel kblaszczak-intel Awaiting requested review from kblaszczak-intel

Assignees

@dmitry-gorokhov dmitry-gorokhov

Labels
category: CI OpenVINO public CI category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation category: IE Tests OpenVINO Test: plugins and common github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
2024.0
Development

Successfully merging this pull request may close these issues.
7 participants
@wangleis @dmitry-gorokhov @sunxiaoxia2022 @xufang-lisa @ilya-lavrenov @akopytko @kblaszczak-intel