Add nuScenes Dataset Support for 2D Object Detection#397
Add nuScenes Dataset Support for 2D Object Detection#397tejasstanley wants to merge 13 commits intoJdeRobot:masterfrom
Conversation
…date nuscenes-devkit in pyproject.toml
…onMetrics into nuScenes_detection
|
Hi @tejasstanley . Thanks for your contribution! It will take me a couple of days to review it, but at first sight, the docstrings don't follow the Sphinx style (reference: https://sphinx-rtd-tutorial.readthedocs.io/en/latest/docstrings.html). Can you update that? Also, can you share links so that I can download the dataset myself? |
|
Hi @dpascualhe, thanks for the feedback, I will update the docstrings. |
|
Hi @tejasstanley , great work! It is working for me. Regarding the notebook, can you update it to contain only relative paths so that your home directory is not in it (if you could also fix it for COCO, that would be great!). Also, why are some classes dropped in the dataset? Beyond, and more importantly, for the particular case of 2D object detection, after going through nuScenes, maybe we should pivot towards nuImages (https://www.nuscenes.org/nuimages), so that we don't have problems with projections, occlusions, etc. What do you think? Would you be interested in checking if integrating nuImages for object detection instead of nuScenes makes more sense? |
|
Thanks for the feedback @dpascualhe. Yes I will update the relative paths for the datasets. Regarding the class drops, there were many classes like traffic barriers and etc, which were hampering the visualization. If needed the set can be kept empty also; all classes would be restored. |
I think supporting nuImages for image object detection makes more sense, sorry for not identifying this earlier, but structure-wise your dataset class can be reused I hope! |
|
Hi @dpascualhe, apologies for the delay. You were right, nuImages seems like a much better fit for both segmentation and BBOX detection. |
|
Awesome! Looks good. I'll download the dataset and test it thoroughly to give you a proper review asap. Thanks! |
|
Still in progress my review, but can you rebase latest changes in master? They had a fix so that you don't have to add that small change in torch_segmentation. Also, can you look for models pretrained with nuImages? It would be nice to have models to properly test it. Regarding nuScenes, I'd get rid of it yes. nuImages is the way to go 😄 |
There was a problem hiding this comment.
It's looking good! I'd get rid of the nuscenes-related changes, and build a new tutorial specific for nuimages (detection and segmentation, while keeping the previous tutorials on image detection and segmentation unchanged. The implementation of the dataset itself is looking fine beyond the minor comments I raised. Make sure to update with the latest changes in master and resolve any conflicts. Upon solving these issues, we can merge.
Good job! 😄
|
Hi @dpascualhe, |

PR: Add nuScenes Dataset Support for 2D Object Detection
Summary
This PR adds support for the nuScenes dataset for 2D object detection by introducing the
NuScenesDetectionDatasetclass. It parses 3D bounding boxes from nuScenes into 2D camera views, keeps canonical class labels, and works with the existing tutorials (tutorial_image_detection.ipynb).Changes Included
NuScenesDetectionDatasetinheriting fromImageDetectionDataset.nuscenes-devkitinpyproject.toml.Notes / Pending Tasks
Testing
References