TobyPDE / libforest

C++ Random Forest Library

7 stars 4 forks source link

Segfault with Random Forest #28

Closed TobyPDE closed 9 years ago

TobyPDE commented 9 years ago

There is some kind of segmentation fault with the efficient entropy histograms in the default random forest implementation. We could reproduce it with MNIST and the rf example.

Bitbucket: https://bitbucket.org/geekstack/libforest/issue/13
Originally reported by: Tobias Pohlen
Originally created at: 2015-03-06T15:59:53.162

TobyPDE commented 9 years ago

The problem was because the class labels were negative. We can avoid this type of errors by using stronger assertions in the API. Thus, #21 resolves this issue.

Original comment by: Tobias Pohlen

TobyPDE commented 9 years ago

21 resolves the issue.

Original comment by: Tobias Pohlen