Hue-Saturation range in tutorial (Bug #2016)


Added by abid rahman almost 13 years ago. Updated almost 13 years ago.


Status:Done Start date:2012-06-04
Priority:Low Due date:
Assignee:Vadim Pisarevsky % Done:

0%

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

Description

Hi,

Below is the part of code from http://code.opencv.org/svn/opencv/trunk/opencv/samples/cpp/tutorial_code/Histograms_Matching/compareHist_Demo.cpp (Line 46-48)

    // hue varies from 0 to 256, saturation from 0 to 180
    float h_ranges[] = { 0, 256 };
    float s_ranges[] = { 0, 180 };

It is same in tutorials also.

Actually isn't it in the opposite way, ie, hue = [0,180] and saturation = [0,255]?


Associated revisions

Revision 517fc5e2
Added by Vadim Pisarevsky almost 13 years ago

fixed typo in the tutorial code (thanks to Abid Rahman; ticket #2016)

Revision 6a5a0fe8
Added by Vadim Pisarevsky almost 11 years ago

Merge pull request #2016 from pemmanuelviel:kmeansppSquareDist

History

Updated by Andrey Pavlenko almost 13 years ago

  • Category set to documentation

Updated by Vadim Pisarevsky almost 13 years ago

thanks! fixed in trunk, r2573

  • Status changed from Open to Done
  • Target version set to 2.4.2
  • Assignee set to Vadim Pisarevsky

Updated by Vadim Pisarevsky almost 13 years ago

oops, the correct revision is r8573

Also available in: Atom PDF