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

Force install a py2 compatible version of pandas #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LiyouZhou
Copy link
Collaborator

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.

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
Copy link
Collaborator Author

@mjshiggins
Copy link

Confirmed this fix for the docker image build

@LiyouZhou
Copy link
Collaborator Author

anybody willing to give it a merge?

Copy link

@amtagrwl amtagrwl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable.

@LiyouZhou
Copy link
Collaborator Author

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

@amtagrwl
Copy link

amtagrwl commented Jul 9, 2020

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

@LiyouZhou
Copy link
Collaborator Author

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

ErikUckert added a commit to ErikUckert/kitti2bag that referenced this pull request Nov 8, 2021
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.

3 participants