patch 4 corner detection sample possible bug (Bugfix #2532)

Added by Recep Doga Siyli over 12 years ago. Updated about 12 years ago.

Status:Done Start date:2012-11-11
Priority:Normal Due date:
Assignee:Alexander Smorkalov % Done:

0%
Category:samples
Target version:2.4.4
Affected version:2.4.3 Operating System:
Difficulty: HW Platform:
Pull request:

Description

Hi, I am not an expert but when I tried running the sample provided at:
http://docs.opencv.org/2.4.3rc/doc/tutorials/features2d/trackingmotion/generic_corner_detector/generic_corner_detector.html

at lines

float lambda_1 = myHarris_dst.at<float>( j, i, 0 );
float lambda_2 = myHarris_dst.at<float>( j, i, 1 );

the VisualStudio threw an exception. Then I changed the code to

float lambda_1 = myHarris_dst.ptr<float>(j)[6*i+0];
float lambda_2 = myHarris_dst.ptr<float>(j)[6*i+1];

which worked just fine for my case.. I am not sure if other people are facing the same issue.

Associated revisions

Revision 627b4410
Added by Alexander Smorkalov about 12 years ago

Bugfix #2532 patch 4 corner detection sample possible bug solved.

Tutorial text was not consistent with tutorial source code in samples directory.
Inline source code was replaced on "includeliteral" directive with link to cpp file.

History

#1
Updated by Andrey Kamaev over 12 years ago

  • Target version set to 2.4.4

#2
Updated by Kirill Kornyakov over 12 years ago

  • Description changed from Hi, I am not an expert but when I tried running the sample provided at : http... to Hi, I am not an expert but when I tried running the sample provided at: http:... More
  • Tracker changed from Patch to Bugfix

#3
Updated by Vadim Pisarevsky about 12 years ago

the proposed fix seems to be correct

  • Target version deleted (2.4.4)

#4
Updated by Vadim Pisarevsky about 12 years ago

  • Assignee deleted (Kirill Kornyakov)

#5
Updated by Alexander Smorkalov about 12 years ago

  • Affected version set to 2.4.3
  • Assignee set to Alexander Smorkalov

#6
Updated by Andrey Kamaev about 12 years ago

Fix is pushed to 2.4

  • Status changed from Open to Done

#7
Updated by Kirill Kornyakov about 12 years ago

  • Target version set to 2.4.4

Also available in: Atom PDF