-
Notifications
You must be signed in to change notification settings - Fork 262
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
Force install a py2 compatible version of pandas #51
base: master
Are you sure you want to change the base?
Conversation
kitti2bag depends on pykitti which depends on pandas which deprecated py2 support in 0.24 causing breakage in the docker build. This patch works around this issue by first installing a py2 compatible version of pandas before installing kitti2bag.
travis failing with same error on master. https://travis-ci.org/tomas789/kitti2bag/builds/610976165?utm_medium=notification&utm_source=github_status |
Confirmed this fix for the docker image build |
anybody willing to give it a merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable.
@amtagrwl thanks for the review. Somehow merge is still blocked? |
@LiyouZhou Yeah. Travis CI fails. Could you please confirm that the pull request builds? |
@amtagrwl verified today that this pr builds and master is still broken. |
kitti2bag depends on pykitti which depends on pandas
which deprecated py2 support in 0.24 causing breakage
in the docker build. This patch works around this issue
by first installing a py2 compatible version of pandas
before installing kitti2bag.