argoverse / argoverse-api

Official GitHub repository for Argoverse dataset
https://www.argoverse.org
Other
833 stars 239 forks source link

account for rounding in point cloud projection #262

Open tom-bu opened 2 years ago

tom-bu commented 2 years ago

The current code still considers a uv point that rounds up to the img width/height as valid. This causes errors when you try to run draw_ground_pts_in_image() in the ground_visualization.py.

johnwlambert commented 2 years ago

Thanks for the PR, @tom-bu. Would you mind adding a small unit test to your PR that fails without your change, and passes with your change?

tom-bu commented 2 years ago

Hi John, I have added a unit test. Let me know if it's what you're looking for. Thanks.