calcHist() should raise error for incompatible nimages and dims (Bug #3918)


Added by Baris Demiroz over 10 years ago. Updated over 9 years ago.


Status:Incomplete Start date:2014-09-25
Priority:Low Due date:
Assignee:Baris Demiroz % Done:

0%

Category:imgproc, video
Target version:-
Affected version:2.4.9 (latest release) Operating System:Any
Difficulty: HW Platform:Any
Pull request:

Description

When dims=1 is passed to calcHist() doesn't take nimages parameter into account, and processes as if nimages=1. However, this is not directly or intuitively visible from the documentation. Documentation should be more clear about this behavior and/or calcHist() should raise an error via CV_Assert() for incompatible parameters. The first impression of the function signature is that it is going to aggregate the histograms of all the given images.

PS. I found the whole calcHist() usage obscure, it is very hard to understand and use unlike most of the OpenCV.


History

Updated by Vladislav Vinogradov over 10 years ago

Hello Baris,

Could you add a sample code to reproduce the issue?

  • HW Platform changed from x64 to Any
  • Operating System changed from Windows to Any
  • Status changed from New to Incomplete
  • Category set to imgproc, video
  • Assignee set to Baris Demiroz

Updated by Maksim Shabunin over 9 years ago

Issue has been transferred to GitHub: https://github.com/Itseez/opencv/issues/4833

Also available in: Atom PDF