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

Submap for PGO? #6

Open
YangSiri opened this issue May 9, 2022 · 8 comments
Open

Submap for PGO? #6

YangSiri opened this issue May 9, 2022 · 8 comments

Comments

@YangSiri
Copy link

YangSiri commented May 9, 2022

Hi, @yanliang-wang

Did u use submap for loop closure or just single frame?

@yanliang-wang
Copy link
Owner

Hi, it is the submap form.

If a loop closure is found, the registration result between their submap will be added to the pose graph.

@YangSiri
Copy link
Author

YangSiri commented May 9, 2022

Thanks, great work~

@YangSiri
Copy link
Author

Hi, @yanliang-wang

There are still some questions that i want to ask you for help.

  1. Did u notice the missing blocks in the scan? In my experiments with VLP16 and your gif as well, i saw some blocks of missing points in some rings of one scan. During the odometry, the positions of the missing block vary. Is this phenomenon caused by motion undistortion(certainly not by occlusion)? But still, i think it is weird in fastLIO.
    2022-05-10 10-23-28屏幕截图
    2022-05-10 10-22-18屏幕截图

  2. When i check the raw time field in scan produced by velodyne_driver, it is negative. Do u know the reason for this?

@YangSiri
Copy link
Author

Hi, @yanliang-wang

I noticed you construct scancontext just using keframes instead of submap accumulated by kfs?

scManager.makeAndSaveScancontextAndKeys(*thisKeyFrameDS);

@yanliang-wang
Copy link
Owner

Hi, @yanliang-wang

I noticed you construct scancontext just using keframes instead of submap accumulated by kfs?

scManager.makeAndSaveScancontextAndKeys(*thisKeyFrameDS);

Yes, this part is from FAST_LIO_SLAM. I didn't change it. But it doesn't matter if you only use the radius-based loop closure.

@YangSiri
Copy link
Author

You were saying you also use frames to detect loop but match submaps to close the loop instead?

@yanliang-wang
Copy link
Owner

You were saying you also use frames to detect loop but match submaps to close the loop instead?

Yes, is there any problem?

@YangSiri
Copy link
Author

Hi, it is the submap form.

If a loop closure is found, the registration result between their submap will be added to the pose graph.

Of course no, I misunderstood. I thought you use submap for loop detection as well. Thanks~

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

2 participants