slothfulxtx / TransLoc3D

TransLoc3D : Point Cloud based Large-scale Place Recognition using Adaptive Receptive Fields (CVM2022)
MIT License
16 stars 5 forks source link

This package depends on nvidia-ml-py3 which is quite old, could it use nvidia-ml-py instead? #4

Closed mattip closed 2 years ago

mattip commented 2 years ago

As the title states, there is a dependency on the obsolete nvidia-ml-py3 in the requirements.txt. But this makes the package incompatible with gpustat, since it requires nvidia-ml-py (without the 3) instead. This trips up ray, here is the relevant discussion https://github.com/wookayin/gpustat/pull/107#issuecomment-1189837063

slothfulxtx commented 2 years ago

I suppose it could work, because no packages except for gpustat require nvidia-ml-py3.

Reguards, Tian-Xing Xu