mlnbayes.cpp CvNormalBayesClassifier::train() covariance calculation (Feature #38)

Added by Markus D about 15 years ago. Updated about 15 years ago.

Status:Done Start date:
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:ml
Target version:-
Difficulty: Pull request:

Description

Hello,

please take a look at the train() method where the covariance calculation is done. In line 242 there is:

double s1val = sum1[j];

shouldn't there rather be:

double s1val = sum1[i];

?
I might be wrong, but the j variable is not used yet inside the loop. Patch attached.

OpenCV rocks! Keep up the good work!
Cheers, Markus

mlnbayes.patch - Patch for mlnbayes.cpp (585 Bytes) Markus D, 2009-12-14 07:51 pm

Associated revisions

Revision 83799d6b
Added by Andrey Kamaev over 12 years ago

Fix Mat indexing in TrackingMotion tutorial code (supersedes pull request #38)

Revision 97852e08
Added by Andrey Kamaev almost 12 years ago

Merge pull request #38 from jet47/clahe

sanity data for CLAHE

History

#1
Updated by Vadim Pisarevsky about 15 years ago

thanks! fixed in r2457

  • Status changed from Open to Done
  • (deleted custom field) set to fixed

Also available in: Atom PDF