-
-
Notifications
You must be signed in to change notification settings - Fork 56.4k
Skip InferList.TestStreamingInfer #25321
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
Conversation
asmorkalov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
@dkurt please let me know, when it's ready for merge. |
|
@asmorkalov, I have a concern that the test hangs not sporadically but because of memory issues. For example, after merge #25312 it executes fine. Let me move this PR to draft. |
|
These tests shouldn't run at all with the newer versions of OpenVINO, @dkurt which OpenVINO version do you refer too? |
|
@dmatveev, Our actions CI still depend on OpenVINO 2023.0. Locally tests won't run with 2023.1 |
@dkurt we can further lower down the check probably, to avoid the 2023.0 either. @TolyaTalamanov any ideas on this? Would it impact us much? Meanwhile, there's OpenVINO 2024.0 already available and 2024.1 is coming soon. Finally, the issue may be still irrelevant to the particular OpenVINO version or G-API's execution mechanics, let me prepare reproducer which will (sooner or later) hang without any of those components. |
|
@asmorkalov no problem at all! |
Pull Request Readiness Checklist
similar workaround as #25294
resolves #25320
See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request
Patch to opencv_extra has the same branch name.