search

cvlab-epfl / LIFT

Code release for the ECCV 2016 paper
485 stars 168 forks source link

Requirements file is not specified enough #41

Closed breakingflower closed 4 years ago

breakingflower commented 5 years ago

Tried to get the code to work but it seems the requirements are not backwards compatible. Can you specify package versions in the requirements file?

I've tried the following: 

conda create -n lift anaconda 
pip install -r requirements 
cd c-code/build 
cmake ..
make 
cd .. 
cd .. 
./run.sh

errors: first they were related to opencv, then flock, then hd5py, then ....

Can you please provide a more specified list such as numpy=1.12

Thanks

breakingflower commented 5 years ago

I have made a docker version of the tf-lift implementation of the same authors, which you can use by pulling the image from 

docker pull fremmen/tf-lift
kmyi commented 4 years ago

Thanks. Unfortunately, as you probably noticed, this code base is very old and we also don't know which exact versions any more. Which is why we also implemented the tensorflow port. They perform a bit differently, but should be comparable. Thank you again for the docker port.