Skip to content
GitLab

Fix KNearestNeighborsMachineLearningModel

Merged Cédric Traizet requested to merge 2181_bug_knn into develop

Summary

closes #2181 (closed)

when using OpenCV >= 3 in OTB, the KNN machine learning model writes the number of neighbors that should be used in the training step, but this number is not read back when loading the model. As a consequence, the default number of neighbor (32) is always used.

Copyright

The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.

Check before merging:

  • All discussions are resolved
  • At least 2 👍 votes from core developers, no 👎 vote.
  • The feature branch is (reasonably) up-to-date with the base branch
  • Dashboard is green
  • Copyright owner has signed the ORFEO ToolBox Contributor License Agreement
  • Optionally, run git diff develop... -U0 --no-color | clang-format-diff.py -p1 -i on latest changes and commit
Edited by Cédric Traizet