kdexd / virtex

[CVPR 2021] VirTex: Learning Visual Representations from Textual Annotations
http://kdexd.xyz/virtex
MIT License
556 stars 61 forks source link

Update Requirements: `numpy` missing #29

Closed GeorgeBatch closed 2 years ago

GeorgeBatch commented 2 years ago

Dear authors,

Thank you very much for providing the code and instructions for using this library! I tried to use the instructions from http://kdexd.xyz/virtex/virtex/usage/setup_dependencies.html:

cd virtex
conda create -n virtex python=3.6
conda activate virtex
pip install -r requirements.txt

I run into a problem that numpy was not found when I ran pip install -r requirements.txt. I fixed it by running conda install numpy before that.

Maybe you could add it to the requirements.txt so others can use the same version you used.

Many thanks, George Batchkala

kdexd commented 2 years ago

Hi @GeorgeBatch – I cannot reproduce this issue, numpy already exists in requirements and I could install it with pip install -r requirements.txt. I will close this issue, you seem to have found some fix to unblock your work! Feel free to re-open it if you face any issues related to this later!