Skip to content

Refactor ml2#3032

Merged
opencv-pushbot merged 16 commits intoopencv:masterfrom
vpisarev:refactor_ml2
Aug 4, 2014
Merged

Refactor ml2#3032
opencv-pushbot merged 16 commits intoopencv:masterfrom
vpisarev:refactor_ml2

Conversation

@vpisarev
Copy link
Copy Markdown
Contributor

@vpisarev vpisarev commented Aug 2, 2014

completely revised opencv_ml code. It's much simpler and more in-line with our current coding methodologies.

There are some drawbacks, however:

  1. some of parallelization was removed and not restored yet
  2. gradient boosting trees are not included yet
  3. extremely random trees are not included yet
  4. decision trees lost the support for missing measurements (and surrogate splits as well); pruning has not been tested yet and is currently disabled. On the other hand, those lost features will hardly affect 2 big computer vision algorithms based on decision trees: random trees & boosting.
  5. traincascade - our major and one of the most important apps using ml, has not been updated to use the new code. Instead it uses an extracted subset of the old code. But it's probably for the good right now.

@vpisarev
Copy link
Copy Markdown
Contributor Author

vpisarev commented Aug 4, 2014

@mdim, please, review

@vpisarev vpisarev assigned vpisarev and unassigned mdim Aug 4, 2014
@vpisarev
Copy link
Copy Markdown
Contributor Author

vpisarev commented Aug 4, 2014

@mdim said, she can review this only in a few days from now; let's put it in now and then correct problems if any

@vpisarev
Copy link
Copy Markdown
Contributor Author

vpisarev commented Aug 4, 2014

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants