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

Missing sweeps virtual points #26

Open
zw615 opened this issue Oct 4, 2022 · 2 comments
Open

Missing sweeps virtual points #26

zw615 opened this issue Oct 4, 2022 · 2 comments

Comments

@zw615
Copy link

zw615 commented Oct 4, 2022

Hi, I just found there are about 2000 sweeps of virtual points missing.

Here are the number of files in sweeps/LIDAR_TOP/

ls sweeps/LIDAR_TOP/ | wc -l
297737

Here are the number of files in sweeps/LIDAR_TOP_VIRTUAL/

ls data/nuscenes/sweeps/LIDAR_TOP_VIRTUAL/ | wc -l
295489

I directly download and extract the data from the official link here. I am pretty sure I didn't mess with the data. I wonder is this normal?

Thanks!

@tianweiy
Copy link
Owner

tianweiy commented Oct 4, 2022

hi zeyu, I also noticed this. I think the reason is that some of the sweeps are not used at all in our model. Basically, in nuscenes, we have key frames at 2FPS, and non keyframes (sweeps) at 20FPS. Our model uses key frames + last 9 non key frames so some non key frames may not be used at all in the training / inference.

The training / testing should work out of the box if you use the centerpoint repo.

Let me know if you have further questions

@zw615
Copy link
Author

zw615 commented Oct 10, 2022

Thanks for your prompt reply!

As you may have noticed, I am not using the centerpoint repo. I have managed to migrate the virtual points related code to the mmdet3d repo and am using it. I find simply discarding the sweep when it is missing or repeating other sweeps is able to work.

Thanks!

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