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

This exports pointcloud for odometry data and adapts to the latest pykitti #45

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

ulterzlw
Copy link
Collaborator

No description provided.

@valgur
Copy link
Collaborator

valgur commented Jun 25, 2019

Hi! Thanks for the fixes. There's an existing pull request incorporating similar changes, plus quite a few others, open here: #38. May I ask you to review it and comment if there's anything from this PR that could be included there instead?

@ulterzlw
Copy link
Collaborator Author

Hi! Thanks for the fixes. There's an existing pull request incorporating similar changes, plus quite a few others, open here: #38. May I ask you to review it and comment if there's anything from this PR that could be included there instead?

Hi! Thanks. Well, in my situation, I just need a script to generate a bag containing raw data or odometry data completely. The repository doesn't work for me. So I made some minor changes.
As for the PR #38, I had a quick look. Since i'm not familiar with cli, it made me struggled. And It would be great to include point cloud data. Because, usually, we may need to couple the vision and lidar to get a better result.

@ulterzlw ulterzlw force-pushed the master branch 2 times, most recently from ff9c922 to 85f755f Compare January 4, 2023 07:43
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