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

Allow int values as input to drive and sequence params, remove an unused cv2 dependency #47

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

Conversation

valgur
Copy link

@valgur valgur commented May 15, 2019

This makes initializing the classes a bit more convenient.
The cv2 dependency is quite heavy-weight and entirely unused right now.

By the way, would it be possible to create a new minor release of pykitti that incorporates the fix from #39, please? I'm using pykitti via kitti2bag and ROS1 is stuck with Python 2 for at least another year, unfortunately.

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.

1 participant