Fixed test errors, added support for C data types.
This commit is contained in:
parent
c5b4b99350
commit
f8b23cff18
@ -2325,13 +2325,16 @@ double cv::compareHist( InputArray _H1, InputArray _H2, int method )
|
|||||||
}
|
}
|
||||||
else if( method == CV_COMP_KL_DIV )
|
else if( method == CV_COMP_KL_DIV )
|
||||||
{
|
{
|
||||||
for( j = 0; j < len; j++ ){
|
for( j = 0; j < len; j++ )
|
||||||
|
{
|
||||||
double p = h1[j];
|
double p = h1[j];
|
||||||
double q = h2[j];
|
double q = h2[j];
|
||||||
if( p == 0.0 )
|
if( fabs(p) <= DBL_EPSILON ) {
|
||||||
continue;
|
continue;
|
||||||
if( q == 0.0 )
|
}
|
||||||
q += 1e-10;
|
if( fabs(q) <= DBL_EPSILON ) {
|
||||||
|
q = 1e-10;
|
||||||
|
}
|
||||||
result += p * cv::log( p / q );
|
result += p * cv::log( p / q );
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -2370,7 +2373,7 @@ double cv::compareHist( const SparseMat& H1, const SparseMat& H2, int method )
|
|||||||
CV_Assert( H1.size(i) == H2.size(i) );
|
CV_Assert( H1.size(i) == H2.size(i) );
|
||||||
|
|
||||||
const SparseMat *PH1 = &H1, *PH2 = &H2;
|
const SparseMat *PH1 = &H1, *PH2 = &H2;
|
||||||
if( PH1->nzcount() > PH2->nzcount() && method != CV_COMP_CHISQR && method != CV_COMP_CHISQR_ALT)
|
if( PH1->nzcount() > PH2->nzcount() && method != CV_COMP_CHISQR && method != CV_COMP_CHISQR_ALT && method != CV_COMP_KL_DIV )
|
||||||
std::swap(PH1, PH2);
|
std::swap(PH1, PH2);
|
||||||
|
|
||||||
SparseMatConstIterator it = PH1->begin();
|
SparseMatConstIterator it = PH1->begin();
|
||||||
@ -2450,6 +2453,18 @@ double cv::compareHist( const SparseMat& H1, const SparseMat& H2, int method )
|
|||||||
s1 = fabs(s1) > FLT_EPSILON ? 1./std::sqrt(s1) : 1.;
|
s1 = fabs(s1) > FLT_EPSILON ? 1./std::sqrt(s1) : 1.;
|
||||||
result = std::sqrt(std::max(1. - result*s1, 0.));
|
result = std::sqrt(std::max(1. - result*s1, 0.));
|
||||||
}
|
}
|
||||||
|
else if( method == CV_COMP_KL_DIV )
|
||||||
|
{
|
||||||
|
for( i = 0; i < N1; i++, ++it )
|
||||||
|
{
|
||||||
|
float v1 = it.value<float>();
|
||||||
|
const SparseMat::Node* node = it.node();
|
||||||
|
float v2 = PH2->value<float>(node->idx, (size_t*)&node->hashval);
|
||||||
|
if( !v2 )
|
||||||
|
v2 = 1e-10;
|
||||||
|
result += v1 * cv::log( v1 / v2 );
|
||||||
|
}
|
||||||
|
}
|
||||||
else
|
else
|
||||||
CV_Error( CV_StsBadArg, "Unknown comparison method" );
|
CV_Error( CV_StsBadArg, "Unknown comparison method" );
|
||||||
|
|
||||||
@ -2795,7 +2810,7 @@ cvCompareHist( const CvHistogram* hist1,
|
|||||||
CvSparseMatIterator iterator;
|
CvSparseMatIterator iterator;
|
||||||
CvSparseNode *node1, *node2;
|
CvSparseNode *node1, *node2;
|
||||||
|
|
||||||
if( mat1->heap->active_count > mat2->heap->active_count && method != CV_COMP_CHISQR && method != CV_COMP_CHISQR_ALT)
|
if( mat1->heap->active_count > mat2->heap->active_count && method != CV_COMP_CHISQR && method != CV_COMP_CHISQR_ALT && method != CV_COMP_KL_DIV )
|
||||||
{
|
{
|
||||||
CvSparseMat* t;
|
CvSparseMat* t;
|
||||||
CV_SWAP( mat1, mat2, t );
|
CV_SWAP( mat1, mat2, t );
|
||||||
@ -2897,6 +2912,13 @@ cvCompareHist( const CvHistogram* hist1,
|
|||||||
result = 1. - result*s1;
|
result = 1. - result*s1;
|
||||||
result = sqrt(MAX(result,0.));
|
result = sqrt(MAX(result,0.));
|
||||||
}
|
}
|
||||||
|
else if( method == CV_COMP_KL_DIV )
|
||||||
|
{
|
||||||
|
cv::SparseMat sH1, sH2;
|
||||||
|
((const CvSparseMat*)hist1->bins)->copyToSparseMat(sH1);
|
||||||
|
((const CvSparseMat*)hist2->bins)->copyToSparseMat(sH2);
|
||||||
|
result = cv::compareHist( sH1, sH2, CV_COMP_KL_DIV );
|
||||||
|
}
|
||||||
else
|
else
|
||||||
CV_Error( CV_StsBadArg, "Unknown comparison method" );
|
CV_Error( CV_StsBadArg, "Unknown comparison method" );
|
||||||
|
|
||||||
|
@ -1021,11 +1021,12 @@ int CV_CompareHistTest::validate_test_results( int /*test_case_idx*/ )
|
|||||||
sq0 += v0*v0;
|
sq0 += v0*v0;
|
||||||
sq1 += v1*v1;
|
sq1 += v1*v1;
|
||||||
result0[CV_COMP_BHATTACHARYYA] += sqrt(v0*v1);
|
result0[CV_COMP_BHATTACHARYYA] += sqrt(v0*v1);
|
||||||
if( fabs(v0) > DBL_EPSILON )
|
|
||||||
{
|
{
|
||||||
if( fabs(v1) < DBL_EPSILON )
|
if( fabs(v0) <= DBL_EPSILON )
|
||||||
v1 += 1e-10;
|
continue;
|
||||||
result0[CV_COMP_KL_DIV] += v0 * cv::log( v0 / v1 );
|
if( fabs(v1) <= DBL_EPSILON )
|
||||||
|
v1 = 1e-10;
|
||||||
|
result0[CV_COMP_KL_DIV] += v0 * std::log( v0 / v1 );
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -1052,10 +1053,11 @@ int CV_CompareHistTest::validate_test_results( int /*test_case_idx*/ )
|
|||||||
s0 += v0;
|
s0 += v0;
|
||||||
sq0 += v0*v0;
|
sq0 += v0*v0;
|
||||||
result0[CV_COMP_BHATTACHARYYA] += sqrt(v0*v1);
|
result0[CV_COMP_BHATTACHARYYA] += sqrt(v0*v1);
|
||||||
if( fabs(v0) > DBL_EPSILON )
|
|
||||||
{
|
{
|
||||||
if( fabs(v1) < DBL_EPSILON )
|
if (v0 <= DBL_EPSILON)
|
||||||
v1 += 1e-10;
|
continue;
|
||||||
|
if (!v1)
|
||||||
|
v1 = 1e-10;
|
||||||
result0[CV_COMP_KL_DIV] += v0 * cv::log( v0 / v1 );
|
result0[CV_COMP_KL_DIV] += v0 * cv::log( v0 / v1 );
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user