Skip to content

Commit 1f6d6f0

Browse files
gdemarcqalalek
authored andcommitted
Merge pull request #1736 from gdemarcq:staticSaliencyFineGrained_corrections
* Correction for computeBinaryMap crash: - the function staticSaliencyFineGrained::computeSaliency() outputs a normalized floating point image - Add an assert in StaticSaliency::computeBinaryMap in order to check image depth (must be CV_32F) * - Change assert in cv::Saliency::StaticSaliency::computeBinaryMap() - Change normalization factor in cv::Saliency::StaticSaliencyFineGrained::computeSaliencyImpl()
1 parent 1070e70 commit 1f6d6f0

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

modules/saliency/src/staticSaliency.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ namespace saliency
5353
bool StaticSaliency::computeBinaryMap( InputArray _saliencyMap, OutputArray _binaryMap )
5454
{
5555
Mat saliencyMap = _saliencyMap.getMat();
56+
CV_CheckTypeEQ(saliencyMap.type(), CV_32FC1, "");
5657
Mat labels = Mat::zeros( saliencyMap.rows * saliencyMap.cols, 1, 1 );
5758
Mat samples = Mat_<float>( saliencyMap.rows * saliencyMap.cols, 1 );
5859
Mat centers;

modules/saliency/src/staticSaliencyFineGrained.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ bool StaticSaliencyFineGrained::computeSaliencyImpl(InputArray image, OutputArra
6666
{
6767
Mat dst(Size(image.getMat().cols, image.getMat().rows), CV_8UC1);
6868
calcIntensityChannel(image.getMat(), dst);
69-
dst.copyTo(saliencyMap);
69+
dst.convertTo(saliencyMap, CV_32F, 1.0f/255.0f); // values are in range [0; 1]
7070

7171
#ifdef SALIENCY_DEBUG
7272
// visualize saliency map

0 commit comments

Comments
 (0)