Skip to content
Snippets Groups Projects

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 :thumbsup: votes from core developers, no :thumbsdown: 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

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading