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
[GSoC 2017] DeepGaze1 + Discriminant Saliency + BackgroundContrast #1252
base: 4.x
Are you sure you want to change the base?
Conversation
…tyle and comment will be handled later\nFinish module design\nWorking on realizing training function and fixing saliency map computing bugs\n
…ency. Path is hardcoded, need to be fix
7f270b3
to
c6756c0
Compare
caadbb4
to
1ee9867
Compare
…ze1 sample work without calculating AUC
@@ -165,6 +165,7 @@ namespace cv { | |||
// Generate mask | |||
Mat mask = regions == r; | |||
|
|||
|
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.
Please revert this useless change.
// of this distribution and at http://opencv.org/license.html. | ||
|
||
#include <opencv2/imgproc.hpp> | ||
#include <opencv2/highgui.hpp> |
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.
Please remove useless includes like highgui and imgcodecs. We shouldn't rely on modules which are not listed in dependencies list (see CMake).
{ | ||
fgWeightDig.at<double>(i, i) = fg.at<double>(i, 0); | ||
} | ||
//temp = (smoothDeri - smoothWeight + bgWeightDig + fgWeightDig); |
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.
Here and everywhere please remove the dead commented code.
@@ -40,6 +40,9 @@ | |||
//M*/ | |||
|
|||
#include "precomp.hpp" | |||
#include <iostream> |
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.
Changes in this file are seems to be useless, so please revert them.
This method uses seed superpixel algorithm to partition the image and calculate the probability of belonging to background with CIE-Lab color contrast. | ||
This method also provides an optimization framework may help foreground based saliency method perform better. | ||
*/ | ||
class CV_EXPORTS_W BackgroundContrast : public StaticSaliency |
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.
I think it's better to rename this class to BackgroundContrastSaliency
. The current name is misleading.
@@ -0,0 +1,149 @@ | |||
name: "CaffeNet" |
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.
Why did you put the modified prototxt to the unrelated module?
Project: Improve Background Subtraction with Aggregated Saliency
Mentor: @lenlen