PRBonn / semantic-kitti-api

SemanticKITTI API for visualizing dataset, processing data, and evaluating results.
http://semantic-kitti.org
MIT License
783 stars 187 forks source link

Problem in the Dockerfile #85

Closed pyun-ram closed 2 years ago

pyun-ram commented 3 years ago

Hi,

Thanks for sharing the awesome development toolkit. It really helps a lot. When I was building the docker image from the provided Dockerfile. I am facing a problem in line-410. It raises a python syntax error when installing via pip. It seems that the pip version is too high and the f-print is not supported by python 3.7.5 according to the docker file.

This problem can be fixed by the following modification in the Dockerfile:

...
RUN updatedb

# Fix pip upgrading problem
RUN apt remove -y --purge python3-pip
RUN curl -O https://bootstrap.pypa.io/pip/3.4/get-pip.py
RUN python3 get-pip.py
RUN python3 -m pip install --upgrade "pip < 19.2"

# Install any python pre-reqs from requirements.txt
# RUN pip3 install -U pip
RUN pip install  scipy==0.19.1 \
...

These four lines modification might be not elegant, but it helps me solve this problem I am pretty sure you guys have a better solution. :)

jbehley commented 3 years ago

thanks for the fix. currently, I have no idea why this turns up now. at some point, I will have a deeper look, but currently, I have no time to investigate this further.