Skip to content

VideoCapture add open/read timeout params to FFMPEG backend#20549

Merged
alalek merged 3 commits intomasterfrom
unknown repository
Aug 13, 2021
Merged

VideoCapture add open/read timeout params to FFMPEG backend#20549
alalek merged 3 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 12, 2021

A little feature that was merged recently to 3.4

port of #20541

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch

@ghost ghost marked this pull request as draft August 13, 2021 00:09
use_opencl = params.get<int>(CAP_PROP_HW_ACCELERATION_USE_OPENCL);
}
#if USE_AV_INTERRUPT_CALLBACK
if (params.has(CAP_PROP_OPEN_TIMEOUT))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

_MSEC is missing here

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done

@ghost ghost marked this pull request as ready for review August 13, 2021 12:26
@ghost ghost marked this pull request as draft August 13, 2021 12:29
@ghost ghost marked this pull request as ready for review August 13, 2021 18:14
@ghost
Copy link
Copy Markdown
Author

ghost commented Aug 13, 2021

should be fixed now

@ghost
Copy link
Copy Markdown
Author

ghost commented Aug 13, 2021

Checks passed

@ghost ghost requested a review from alalek August 13, 2021 19:33
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@alalek alalek merged commit 917cd13 into opencv:master Aug 13, 2021
@alalek alalek mentioned this pull request Oct 15, 2021
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
…set_get

* VideoCapture add open/read timeout params to FFMPEG backend

* Fix wrong enum name

* Fix wrong enum name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant