pytorch_cluster icon indicating copy to clipboard operation
pytorch_cluster copied to clipboard

setup.py doesnt specify torch as dependency leading to downstream failures

Open KukumavMozolo opened this issue 3 years ago • 3 comments

Hi there,

is there any chance that you will add torch as a requirement to setup.py or adopt PEP 517? I am running into problems installing this package while trying to install it using pip-tools as I described here: https://github.com/jazzband/pip-tools/issues/1662#issuecomment-1202839911

KukumavMozolo avatar Aug 05 '22 08:08 KukumavMozolo

Likely no, since this will result in a lot of issues regarding compilation - if we require torch in setup.py, it will install PyTorch with CUDA 10.2 which might not match your local CUDA version. In addition, you will need to have PyTorch installed in the first place to install this package.

rusty1s avatar Aug 05 '22 13:08 rusty1s

Thank you for your reply, I am not really knowledgeable on that matter but this https://github.com/huggingface/transformers project also depends on torch and I do not recall running into similar problems installing it via pip-tools. Maybe there is a solution there?

KukumavMozolo avatar Aug 05 '22 13:08 KukumavMozolo

Well, that‘s because transformers is Python only - it does not need to compile CUDA kernels.

rusty1s avatar Aug 05 '22 13:08 rusty1s

This issue had no activity for 6 months. It will be closed in 2 weeks unless there is some new activity. Is this issue already resolved?

github-actions[bot] avatar Feb 02 '23 01:02 github-actions[bot]