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

[benchmark] Do not check nireq if sync and time is specified #28134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[benchmark] Do not check nireq if sync and time is specified #28134

wants to merge 1 commit into from
+8 −5

Conversation

valir
Copy link
Contributor

@valir valir commented Dec 18, 2024

Details:

  • This disables nireq check in case execution time is specified when using the sync API

Tickets:

@valir valir requested review from a team as code owners December 18, 2024 17:04
@valir valir requested review from ilya-lavrenov and removed request for a team December 18, 2024 17:04
@github-actions github-actions bot added category: samples OpenVINO Runtime Samples category: tools OpenVINO C++ / Python tools labels Dec 18, 2024
@valir valir force-pushed the valir/master/CVS-159638 branch from b828113 to 3856693 Compare December 18, 2024 17:08
@valir valir force-pushed the valir/master/CVS-159638 branch 3 times, most recently from 5647994 to d2db119 Compare December 19, 2024 10:30
Wovchena
Wovchena approved these changes Dec 19, 2024
View reviewed changes
Copy link
Contributor

@Wovchena Wovchena left a comment

Choose a reason for hiding this comment

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

I'd guess the builds still fail because of code style. Make them pass

@valir valir force-pushed the valir/master/CVS-159638 branch from 6979b35 to 595e4ff Compare December 20, 2024 11:03
@valir valir force-pushed the valir/master/CVS-159638 branch from 595e4ff to 22605a9 Compare December 20, 2024 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wovchena Wovchena Wovchena approved these changes

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

Assignees

@Wovchena Wovchena

Labels
category: samples OpenVINO Runtime Samples category: tools OpenVINO C++ / Python tools
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@valir @Wovchena