search

scverse / scvi-tools

Deep probabilistic analysis of single-cell and spatial omics data
http://scvi-tools.org/
BSD 3-Clause "New" or "Revised" License
1.25k stars 355 forks source link

Problems of torchmetrics #3031

Closed HelloWorldLTY closed 3 weeks ago

HelloWorldLTY commented 3 weeks ago

Report

Hi, I notice that the default setting will lead to a problem caused by torchmetrics when importing pertpy:

https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/11648

To fix it, we should use: torchmetrics==0.11.4

I can create a pull request if necessary.

Version information

No response

Zethson commented 3 weeks ago

Dear @HelloWorldLTY,

This might be an issue that you should report to the scvi-tools developers. Torchmetrics is a transitive dependency that we get from them. I'll transfer this to the scvi-tools repository.

canergen commented 3 weeks ago

Can you expand on what the issue is with torchmetrics>0.11.4? I think in our tests it works without the version restriction. Generally, we try to avoid version pinning and fix the underlying issue that is incompatible. Torch and torchmetrics need to have corresponding versions - conda/pip should handle this. If you use custom install commands, can you share these?

HelloWorldLTY commented 3 weeks ago

Hi, I notice that this issue can be resolved by upgrading the version of python to 3.10/3.11. The problem is mentioned here:

https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/11648