Skip to content

Conversation

D00E
Copy link

@D00E D00E commented Sep 21, 2025

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • 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 another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

Description

Is in support to a pull request regarding, issue opencv/opencv#26476.
Core paired pull request: opencv/opencv#27810
This adds function templates inline with the pure virtual overloaded apply function present for background subtraction. Supporting the build of the pull request:

@D00E D00E force-pushed the contrib-known-foreground-mask branch from 61003e2 to 093fc18 Compare September 22, 2025 17:10
@asmorkalov asmorkalov self-requested a review September 23, 2025 15:22
@asmorkalov asmorkalov self-assigned this Sep 23, 2025
@D00E
Copy link
Author

D00E commented Oct 1, 2025

@asmorkalov I'm sure you'll be busy, sorry to bother you, but let me know what you think if you do get a chance. From my understanding the paired pr relies on this and this wont build without the bg subtractor base class modifications made in the other pr.

Thanks, Dave

@asmorkalov
Copy link
Contributor

@D00E Thanks a lot for the contribution! OpenCV CI system looks for PRs with the same branch name and author and test them together. You need to close one of PRs in the pair, create branch with the same name and create PR with the new branch again.

@D00E D00E closed this Oct 3, 2025
@D00E D00E deleted the contrib-known-foreground-mask branch October 3, 2025 18:36
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.

2 participants