search

huggingface / controlnet_aux

Apache License 2.0
400 stars 86 forks source link

`timm` required, but not listed in `setup.py` #13

Closed detkov closed 1 year ago

detkov commented 1 year ago

When importing MidasDetector, turns out that timm is needed here.

I suppose that if we don't even have a requirements.txt, it's better to list it in the setup.py.

Just pointing out, thanks for the repo!

patrickvonplaten commented 1 year ago

Ah yes good point! Let's maybe just add it to the requirements.txt file :-) Would you like to open a PR maybe?