tomas789 / kitti2bag

Convert KITTI dataset to ROS bag file the easy way!
MIT License
702 stars 256 forks source link

Force install a py2 compatible version of pandas #51

Open LiyouZhou opened 4 years ago

LiyouZhou commented 4 years ago

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.

LiyouZhou commented 4 years ago

travis failing with same error on master. https://travis-ci.org/tomas789/kitti2bag/builds/610976165?utm_medium=notification&utm_source=github_status

mjshiggins commented 4 years ago

Confirmed this fix for the docker image build

LiyouZhou commented 3 years ago

anybody willing to give it a merge?

LiyouZhou commented 3 years ago

@amtagrwl thanks for the review. Somehow merge is still blocked?

amtagrwl commented 3 years ago

@LiyouZhou Yeah. Travis CI fails. Could you please confirm that the pull request builds?

LiyouZhou commented 3 years ago

@amtagrwl verified today that this pr builds and master is still broken.