-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Can't set TRAJECTORY_BUILDER_3D.use_intensities = true #1955
Comments
This doesn't seem to be documented anywhere else, but in the config files I found the following: -- When setting use_intensities to true, the intensity_cost_function_options_0
-- parameter in ceres_scan_matcher has to be set up as well or otherwise
-- CeresScanMatcher will CHECK-fail. Will try it out and post back here later. |
No luck. Adding TRAJECTORY_BUILDER_3D.use_intensities = true
TRAJECTORY_BUILDER_3D.ceres_scan_matcher.intensity_cost_function_options_0.weight = 0.5
TRAJECTORY_BUILDER_3D.ceres_scan_matcher.intensity_cost_function_options_0.huber_scale = 0.3
TRAJECTORY_BUILDER_3D.ceres_scan_matcher.intensity_cost_function_options_0.intensity_threshold = 40 to my config file still results in
(this is a smaller bag file so the length of |
Looks like |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm trying to improve my SLAM results by including intensities:
but this quickly leads to
(I can run SLAM when the flag is set to false, but the loop closure is not ideal.)
However, I can tell that my bag file contains a velodyne lidar
"intensity"
field:and I can also use the
"intensity"
field in the.h5
file we end up saving to disk further down the processing chain.Is this feature not supported, or might I have misconfigured something?
The text was updated successfully, but these errors were encountered: