SPML icon indicating copy to clipboard operation
SPML copied to clipboard

Please add a requirements.txt with pinned versions (pip freeze)

Open yepremyana opened this issue 2 years ago • 5 comments

Required Python Packages pytorch >= 1.6 numpy scipy tqdm easydict == 1.9 PyYAML PIL opencv pydensecrf

yepremyana avatar Aug 09 '21 23:08 yepremyana

Thanks for the suggestion! The requirements.txt is uploaded correspondingly.

twke18 avatar Aug 14 '21 15:08 twke18

I think for this project it would be best to do pip freeze > requirements.txt to capture the current version and achieve repeatable installs. This will be especially useful months/years down the line if someone comes across this repo wanting to reproduce the results.

yepremyana avatar Aug 16 '21 18:08 yepremyana

Great suggestion! Since pip freeze > requirements.txt lists many packages which are not used for the project, we instead create another file our_requirements.txt for listing our complete python environment to avoid confusion.

twke18 avatar Sep 04 '21 17:09 twke18

thank you for adding this information. One thing I noticed is that on our_requirements the torch version is listed as torch==1.5.0+cu101, but in the readme and the requirement.txt pytorch >= 1.6? Also can you please confirm what version of python and cuda you are using, I am having trouble replicating your results.

yepremyana avatar Sep 09 '21 23:09 yepremyana

In our experience, we do not observe performance gap among different versions of PyTorch. Which experiments are you currently trying to reproduce? Do you have exactly the same number of batch_size and crop_size as mentioned in the paper (and in the bashscript)?

twke18 avatar Sep 10 '21 00:09 twke18