search

princeton-vl / lietorch

BSD 3-Clause "New" or "Revised" License
661 stars 46 forks source link

Torch Version Requirement #8

Closed Iven-Wu closed 2 years ago

Iven-Wu commented 2 years ago

In README, it says torch>=1.7 is fine. However, in the source code, the lietorch requires torch.linalg.pinv, which seems only exists in torch 1.9+.

zachteed commented 2 years ago

Hi, thanks for catching this issue. pinv is in torch>=1.8. So everything should work in for 1.8 and 1.9 now. I've update the README