Skip to content

added BackgroundSubtractorCNT #994

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

Merged
merged 3 commits into from
Mar 10, 2017

Conversation

lhelontra
Copy link
Contributor

New background subtraction solution, very fast compared with mog2 or Knn. Good solution for low spec hardware. For more details, see: https://www.theimpossiblecode.com/blog/fastest-background-subtraction-opencv/

@sovrasov
Copy link
Contributor

sovrasov commented Mar 3, 2017

👍

Copy link
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.

There is link for generated documentation with this patch.


//! @}

/** @brief Background subtraction based on counting.
Copy link
Member

Choose a reason for hiding this comment

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

Please put code below before this line above:

//! @}

It is required for proper documentation generation.


/** @brief Implementation of background subtraction based on counting.
* About as fast as MOG2 on a high end system (benchmarked on )
* More than twice faster than MOG2 on cheap hardware (benchmarked on Raspberry Pi3).
Copy link
Member

Choose a reason for hiding this comment

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

This part of comment is a "duplication" of comment from .hpp file. Probably it is better to fix it or eliminate duplication.

BTW, this comment is not a part of auto-generated documentation.

* @param useHistory determines if we're giving a pixel credit for being stable for a long time
* @param maxStability maximum allowed credit for a pixel in history
* @param isParallel determines if we're parallelizing the algorithm
*/
Copy link
Member

Choose a reason for hiding this comment

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

This documentation comment should be moved into .hpp file for createBackgroundSubtractorCNT function.

@lhelontra
Copy link
Contributor Author

@alalek Thanks for your suggestions.

@alalek alalek merged commit de34d70 into opencv:master Mar 10, 2017
@sagi-z
Copy link

sagi-z commented Jul 24, 2017

Hi guys,
I'm the author of this code.
It is great that you're inserting my implementation here.

Just one request - please restore this line to the class documentation comment (deleted in the merge):
Algorithm by Sagi Zeevi

(see here : https://github.com/sagi-z/BackgroundSubtractorCNT/blob/master/bgsubcnt.h)

Thanks!

@alalek
Copy link
Member

alalek commented Jul 24, 2017

@sagi-z Patch: #1290 (docs preview)

@sagi-z
Copy link

sagi-z commented Jul 24, 2017

@alalek It looks okay now. Thanks for the quick response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants