Skip to content

Issue #73: Fix for export_kitti.py to account for ego pose differences in LIDAR and camera - #75

Merged
ternaus merged 3 commits into
lyft:masterfrom
kyleyklee:kylelee82/bug-73
Dec 3, 2019
Merged

ternaus merged 3 commits into
lyft:masterfrom
kyleyklee:kylelee82/bug-73

Conversation

@kyleyklee

@kyleyklee kyleyklee commented Nov 16, 2019 •

Copy link
Copy Markdown
Contributor

BACKGROUND

This PR fixes the KITTI conversion issue related to #73 for camera and LIDAR ego pose differences in the Lyft dataset, which directly affect 2D image bounding boxes in the post-converted KITTI labels.

Example (7dbbd3cba5220b616cd74d1ca0c4ad0b43ae4feb08dab84dcbbc5d7dfe9bb000):

[Without ego pose correction]
Figure_1

[With ego pose correction]
Figure_1 fixed

Testing: export_kitti.py was run on the Lyft dataset and visually checked for box alignment.

@ternaus

ternaus commented Nov 26, 2019

Copy link
Copy Markdown
Contributor

@kylelee82 what is the state of the PR? Is it ready to be merged?

@kyleyklee

Copy link
Copy Markdown
Contributor Author

@ternaus the changes are complete and should fix the issue indicated (test was done by running on the dataset). If there are any concerns with removing the assert I'd be happy to get that included somehow.

@ternaus
ternaus merged commit 8f6f764 into lyft:master Dec 3, 2019
@kyleyklee
kyleyklee deleted the kylelee82/bug-73 branch December 3, 2019 05:33
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

Successfully merging this pull request may close these issues.

2 participants