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

In KITTI test, the generated point cloud is not base on input #142

Open
romantic-black opened this issue Dec 26, 2023 · 3 comments
Open

Comments

@romantic-black
Copy link

image
image

@kaali-billi
Copy link

The problem is the data manipulation during training and evaluation. The transformer works on normalized input and GT, but the normalizing factor and the centroid assumed are not the same as the GT scaling factor and the GT centroid. That is why there is error on reconstruction position and side. They have not mentioned this anywhere in the publication or the code. Took me 4 months to finally understand what was going wrong. I have started a new training cycle with changes to the input data to equalize the scaling and centroid.

pc_norm_gt
iss_correc

@kaali-billi
Copy link

Extract the centroid(cen_par) and scaling factor(m_par) and use these to normalize the GT on every instance using pc_norm() and pc_norm_gt(). @yuxumin I believe you need to address this issue otherwise your publication is not completely correct.

@fabricecarles
Copy link

yes @kaali-billi the current implementation is not correct and I also make some changes as suggested in my issue #130. I found that results are far better in case of real world scenario (i.e when you make inference without any ground truth)

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

3 participants