Documentation mistake in SVM's page (Bug #3605)

Added by Ahmad Sanny almost 12 years ago. Updated over 10 years ago.

Status:Done Start date:2014-03-12
Priority:Normal Due date:
Assignee:Ahmad Sanny % Done:

0%
Category:documentation
Target version:-
Affected version:2.4.9 (latest release) Operating System:Any
Difficulty: HW Platform:Any
Pull request:https://github.com/Itseez/opencv/pull/2753

Description

The documentation on SVM http://docs.opencv.org/doc/tutorials/ml/introduction_to_svm/introduction_to_svm.html has a simple mistake:
In the explanation segment the code fro step 1 was written as:
Mat trainingDataMat(3, 2, CV_32FC1, trainingData);
Mat labelsMat (3, 1, CV_32FC1, labels);

while, based in the code block above it, it should be
Mat trainingDataMat(4, 2, CV_32FC1, trainingData);
Mat labelsMat (4, 1, CV_32FC1, labels);

Associated revisions

Revision bcfcc779
Added by Vadim Pisarevsky about 10 years ago

Merge pull request #3605 from jet47:cuda-photo-refactoring

History

#1
Updated by Ivan Korolev almost 11 years ago

Hi Ahmad,
thanks for the bug report!
Now OpenCV team doesn't have a technical writer, so if you could make a pull request on GitHub (http://www.code.opencv.org/projects/opencv/wiki/How_to_contribute), your help will be much appreciated!

  • Category set to documentation
  • Status changed from New to Open

#2
Updated by Daniil Osokin almost 11 years ago

Hi, Ahmad! Please, could you send a bugfix for this typo? It will help a lot. Also, we've started printing contributors for each release (http://code.opencv.org/projects/opencv/wiki/ChangeLog), so don't miss your chance to be involved in OpenCV's "Hall of fame" =)

  • Assignee set to Ahmad Sanny

#3
Updated by Steven Puttemans almost 11 years ago

  • Pull request set to https://github.com/Itseez/opencv/pull/2716
  • Affected version changed from 2.4.8 (latest release) to 2.4.9 (latest release)

#4
Updated by Vladislav Vinogradov almost 11 years ago

  • Pull request changed from https://github.com/Itseez/opencv/pull/2716 to https://github.com/Itseez/opencv/pull/2753

#5
Updated by Daniil Osokin over 10 years ago

  • Status changed from Open to Done

Also available in: Atom PDF