FCGF icon indicating copy to clipboard operation
FCGF copied to clipboard

I find an ERROR of the ground truth pose processing!

Open chenghao-shi opened this issue 3 years ago • 2 comments

Hi, Thank you for sharing this wonderful work! A lot of the current registration works follow your evaluation metrics. However, I recently find an error in "data_loaders.py". The line 467 "M2 = M @ reg.transformation" should be "M2 = reg.transformation @ M". I have done several tests and am pretty sure of this. Could you check this and tell if it is right.Thanks!

chenghao-shi avatar May 27 '22 04:05 chenghao-shi

in open3d, PointCloud.transform(transformation) means transformation @ PointCloud

gitouni avatar Sep 06 '22 02:09 gitouni

Definitely, so M2 should be calculated as transformation @ M.

施成浩 @.***

 

------------------ 原始邮件 ------------------ 发件人: "chrischoy/FCGF" @.>; 发送时间: 2022年9月6日(星期二) 上午10:25 @.>; @.@.>; 主题: Re: [chrischoy/FCGF] I find an ERROR of the ground truth pose processing! (Issue #65)

in open3d, PointCloud.transform(transformation) means transformation @ PointCloud

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

chenghao-shi avatar Oct 11 '22 07:10 chenghao-shi