wh200720041 / iscloam

Intensity Scan Context based full SLAM implementation for autonomous driving. ICRA 2020
Other
559 stars 131 forks source link

Loopclosure seems doesn't work #13

Closed kinggreat24 closed 3 years ago

kinggreat24 commented 3 years ago

Hi @wh200720041, thanks for sharing your excellent work! I have some questions after running your code as follows:

Wish to get reply from you, and thanks for work sincerely.

wh200720041 commented 3 years ago

Hi @kinggreat24 :

It takes a while for global optimization and loop closure detection. Hence the odometry and global optimization is running in a separate thread. Therefore, a certain delay is expected. You can wait for a while and I think path will be updated. Otherwise perhaps it is caused by GTSAM version issue, as you mentioned in another issue. I will take note of this and test a higher version of GTSAM to see if I can get the same problem.

For output of pose, it is simply a file writing function in cpp, there is a description file on how to generate and evaluate accuracy on KITTI website, you can read through that file. In the meanwhile, the evaluation code can also be downloaded from KITTI website.

During my test I didn't experience the lost of frames. Perhaps you could give more details, like which topic you are using to record the pose.

Regards Han

kinggreat24 commented 3 years ago

Thank you for your prompt reply, I have tried to use GTSAM-3.2.3 to check if it is due to version issues, but unfortunately I could not compile success with GTSAM-3.2.3 for some errors, such as

and so on. Can you give me some advice?

wh200720041 commented 3 years ago

Hi @kinggreat24

For the compiling of GTSAM, I think you can search on Google. Because I didn't experience this error when compile my own gtsam.

Han

kinggreat24 commented 3 years ago

Hi @wh200720041, I have replaced the gtsam with ceres, thanks for your help.

yjsx commented 3 years ago

Hi @kinggreat24 It seems that I meet the same question, but I have no idea how to solve it. Do you remember what did you do? Thanks.

kinggreat24 commented 3 years ago

I used Ceres instead of gtsam, you can try Ceres or g2o to optimize pose graph.

发自我的iPhone

------------------ Original ------------------ From: yjsx <notifications@github.com> Date: Sun,Dec 27,2020 5:58 PM To: wh200720041/iscloam <iscloam@noreply.github.com> Cc: kinggreat24 <450993749@qq.com>, Mention <mention@noreply.github.com> Subject: Re: [wh200720041/iscloam] Loopclosure seems doesn't work (#13)

Hi @kinggreat24 It seems that I meet the same question, but I have no idea how to solve it. Do you remember what did you do? Thanks.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

yjsx commented 3 years ago

ohoh, I see. Thank you! @kinggreat24

kinggreat24 commented 3 years ago

If you failed to solve the problem, I can show you my code.

发自我的iPhone

------------------ Original ------------------ From: yjsx <notifications@github.com> Date: Sun,Dec 27,2020 6:18 PM To: wh200720041/iscloam <iscloam@noreply.github.com> Cc: kinggreat24 <450993749@qq.com>, Mention <mention@noreply.github.com> Subject: Re: [wh200720041/iscloam] Loopclosure seems doesn't work (#13)

ohoh, I see. Thank you! @kinggreat24

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

yjsx commented 3 years ago

Thank you very much. Let me have a try. If I fail, I will ask you for help

If you failed to solve the problem, I can show you my code. 发自我的iPhone ------------------ Original ------------------ From: yjsx <notifications@github.com> Date: Sun,Dec 27,2020 6:18 PM To: wh200720041/iscloam <iscloam@noreply.github.com> Cc: kinggreat24 <450993749@qq.com>, Mention <mention@noreply.github.com> Subject: Re: [wh200720041/iscloam] Loopclosure seems doesn't work (#13) ohoh, I see. Thank you! @kinggreat24 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.