Docs: compareHist has wrong distance definition (Bug #2073)

Added by Reto Grieder over 12 years ago. Updated over 12 years ago.

Status:Done Start date:2012-06-21
Priority:Normal Due date:
Assignee:Vadim Pisarevsky % Done:

0%
Category:documentation
Target version:2.4.2
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

Comparing the definition of CV_COMP_BHATTACHARYYA in cv::compareHist() and the Wikipedia article on the Bhattacharyya distance reveals discrepancies.

And looking at the Hellinger distance shows that the implementation (checked the sources) in compareHist() actually uses the Hellinger distance, which is also a true metric and therefore more useful because 0 means close and 1 means far.

Suggestion to correct the documentation and add CV_COMP_HELLINGER, but keep CV_COMP_BHATTACHARYYA for compatibility.

Associated revisions

Revision 87d17317
Added by Vadim Pisarevsky over 12 years ago

added synonym CV_COMP_HELLIGNER for CV_COMP_BHATTACHARYAA to reflect the real formula used (bug report #2073)

Revision 8a456c5a
Added by Andrey Pavlenko about 11 years ago

Merge pull request #2073 from KonstantinMatskevich:ocl_tapi_clahe

History

#1
Updated by Andrey Kamaev over 12 years ago

  • Assignee set to Vadim Pisarevsky

#2
Updated by Vadim Pisarevsky over 12 years ago

thanks! the synonym for CV_COMP_BHATTACHARYAA was added in r8728

  • Status changed from Open to Done
  • Target version set to 2.4.2

Also available in: Atom PDF