Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Random Forest Laser Detector Parameters #99

Open
FullMetalNicky opened this issue Jul 5, 2021 · 0 comments
Open

Random Forest Laser Detector Parameters #99

FullMetalNicky opened this issue Jul 5, 2021 · 0 comments

Comments

@FullMetalNicky
Copy link

FullMetalNicky commented Jul 5, 2021

I am running the noetic branch with ROS Noetic on Ubuntu 20.04.
I noticed that when I train and test in the same program run, I get good results for the random forest laser detector, but when I load a trained model I get garbage.

My model was trained with:
rf_num_trees 15
rf_max_depth 10
But when I load it, I see that m_randomForest->getTermCriteria().maxCount is 50, instead of 15.

I noticed that in the function RandomForestDetector::trainOnFeatures the ROS parameters are considered. But if I load a detector model from a file and then attempt to classify - those parameters are not considered.
I am now getting the ROS params in the constructor, which seems to fix this issue.

Can you look into it, and tell me if I am doing something incorrectly?

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

No branches or pull requests

1 participant