-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Dbow3 integration #784
Dbow3 integration #784
Conversation
A part of creating our working environment
Changes for compiling the src
change boolean to run viewer
Adding benchmark script
Adding socre and room to benchmark file (was missing)
Second attempt to integrate
Disclaimer: I don't intend to create spam. If the ORB_SLAM3 developers wish me to stop posting, I will cease immediately. Hello @yinonb4 , because ORB_SLAM3 is inactive, I am compiling a version with all the best community bugfixes/improvements at https://github.com/jeremysalwen/ORB_SLAM_COMMUNITY. I am curious about what benefits you found from switching DBoW2 with DBoW3. Does this improve accuracy or runtime? |
From DBOW3 repo:
It seems that this only improves runtime. |
@@ -37,4 +37,4 @@ echo "Configuring and building ORB_SLAM3 ..." | |||
mkdir build | |||
cd build | |||
cmake .. -DCMAKE_BUILD_TYPE=Release | |||
make -j4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why?
Replacing DBoW2 with DBoW3, with proper integration.
Code taken from:
https://github.com/rmsalinas/DBow3.git