Hierarchical-Localization icon indicating copy to clipboard operation
Hierarchical-Localization copied to clipboard

Docker image build fails for distutils

Open paujar opened this issue 1 year ago • 3 comments

I got this from main branch, is this due some python dependencies that doesn't anymore work? image

paujar avatar Oct 15 '23 18:10 paujar

I don't use Docker so this hasn't been updated for a while. Since we now use the pre-built pycolmap available on pypi, we don't need to depend on the COLMAP Docker image - any other will do.

sarlinpe avatar Oct 15 '23 20:10 sarlinpe

FYI colmap has begun pushing new images (the previous image was 4 years old and based on colmap 3.6-dev.3). The new image does not have distutils.cmd installed. (distutils is actually deprecated now and was removed in python 3.12). This can be fixed by installing python3.8-distutils:

apt-get update
apt-get install -y python3.8-distutils

However, now colmap/colmap:latest actually has python 3.10 installed. As such, I think these lines could just be removed from the Dockerfile.

scott-vsi avatar Oct 22 '23 01:10 scott-vsi

When removing the lines, it is necessary to install python3-pip. Otherwise pip3 is not found.

Building the docker container without Lines 5-10 and with line 4 changed to RUN apt-get update -y && apt-get install -y python3-pip unzip wget worked for me.

I created pull request #330 to update the Dockerfile.

HelgeS avatar Dec 02 '23 09:12 HelgeS