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

How to implement CenterPoint + Ours(w/o virtual) #31

Closed
yangyt37 opened this issue Mar 23, 2023 · 3 comments
Closed

How to implement CenterPoint + Ours(w/o virtual) #31

yangyt37 opened this issue Mar 23, 2023 · 3 comments

Comments

@yangyt37
Copy link

yangyt37 commented Mar 23, 2023

May I ask how to implement CenterPoint + Ours(w/o virtual)? Is it just to delete virtual_points2 in points = np.concatenate([points, virtual_points1, virtual_points2], axis=0).astype(np.float32) in MVP-main/CenterPoint-vir/det3d/datasets/pipelines/loading.py read_file function?

@yangyt37
Copy link
Author

I have done the following two experiments: one is to delete semantic information (reduce the dimension to five dimensions) and preserve virtual point coordinates (preserve virt_point2); Another method is to preserve semantic information (preserve virt_points1) and delete virtual points (delete virt_point2). The result of the second method is much better than that of the first method. It seems that semantic information can improve the performance of the CenterPoint more than more virtual point.

@tianweiy
Copy link
Owner

  1. yes

  2. I think the first approach won't work (and will degrade the performance). Basically, we can't just remove the semantic information as the model won't be able to distinguish virtual and real points in that stage, which hurts the localization accuracy.

It's true that semantic information improves the detection the most, but our focus here is that adding virtual points (as well as their semantic information and confidence score) can help more with the localization at long range (table 4)

@yangyt37
Copy link
Author

Thank you very much!

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