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

Evaluate method in test.py #10

Closed
vdvchen opened this issue May 22, 2019 · 1 comment
Closed

Evaluate method in test.py #10

vdvchen opened this issue May 22, 2019 · 1 comment

Comments

@vdvchen
Copy link

vdvchen commented May 22, 2019

Thank you very much for your great work ,but I have some problem when trying to evaluate the results on S3DIS with test.py you provided.
According to the original paper of PointNet(http://openaccess.thecvf.com/content_cvpr_2017/papers/Qi_PointNet_Deep_Learning_CVPR_2017_paper.pdf),when testing on the S3DIS dataset, all points in each block are tested.But in your test.py,I find you sampled 4096 points in each block to test on.Dose it mean your evaluate protocol is different from that in PointNet?
Looking forward to your reply.

@WXinlong
Copy link
Owner

@vdvchen Hi, we use the same evaluate protocal with PointNet. You can check the official PointNet code. I also asked the PointNet author the same question and decided to follow their setting for fair comparision. I post the reply I got below.

In a strict test setting, we should use all the original points (could be more than 4096 or less than 4096). For convenience, in our scripts we are testing upsample/downsample to a block to a fixed number of 4096 points. The results should be very similar though. I think it is okay to go with the 4096 points.

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