NiFangBaAGe / CoordFill

[AAAI 2023] CoordFill: Efficient High-Resolution Image Inpainting via Parameterized Coordinate Querying
BSD 3-Clause "New" or "Revised" License
84 stars 7 forks source link

`requirements.txt` wrong #11

Open 00sapo opened 9 months ago

00sapo commented 9 months ago

The project is not reproducible since Ipython 8 requires python >= 3.8, but opencv 3.* requires python <= 3.7.

Also, Pillow and rawpy have a misleading version number, that is not installable in Python 3.6 (I haven't tried with the other Python versions).

Further, the package skimage stops the installation saying to install scikit-image.

Last but not least, torch==1.8.1+cu101 and torchvision=0.9.1+cu101 don't exist in the standard repositories (pypi.org)

Have you used conda or some other non-standard distribution? If so, you should upload the environment.yml...

A requirements.txt that maybe works with standard Python 3.6, but for me can't compile pycocotools:

astropy==3.2.3
dominate==2.6.0
imageio==2.9.0
ipython
kornia==0.5.0
matplotlib==3.1.2
numpy==1.16.4
opencv_python==3.4.2.16
pandas==1.1.3
Pillow
pycocotools==2.0.4
PyYAML==6.0
rawpy
scikit_image==0.16.2
scipy==1.2.1
tensorboardX==2.6
thop==0.1.1.post2209072238
timm==0.3.2
torch==1.8.1
torchprofile==0.0.4
torchsummary==1.5.1
torchvision==0.9.1
tqdm==4.51.0
visdom==0.2.4
yacs==0.1.8
00sapo commented 9 months ago

Anyway, here is my fork, which is pip-installable (only for inference): https://github.com/00sapo/CoordFill

jackhu-bme commented 8 months ago

Any successful requirements.txt ? I'm quite confused about the dependency for this project. Just need to run this for comparison, but the compiling keep failing. If the dependency for https://github.com/00sapo/CoordFill can be provided, this is also ok. @00sapo

00sapo commented 8 months ago

Hi, my repository is installable via pip. Dependencies are listed in the project.toml. It's weird I have to answer for someone else's paper 🤔

jackhu-bme commented 8 months ago

Hi, my repository is installable via pip. Dependencies are listed in the project.toml. It's weird I have to answer for someone else's paper 🤔

Thanks! I guess I should consult you under your repo about its dependencies, I was not sure about it at that time. My appologies. Again, thanks so much for your help!