delve-team / delve

PyTorch model training and layer saturation monitor
https://delve-docs.readthedocs.io
MIT License
79 stars 13 forks source link

[JOSS review] Pinning Pytorch #54

Closed dionhaefner closed 2 years ago

dionhaefner commented 2 years ago

Is it really necessary to pin Pytorch to ==1.9.0? Seems quite restrictive to me, and makes the package harder to install (because if you e.g. do pip install delve and then pip install torchvision it gets overwritten again).

(This is a part of the ongoing review at https://github.com/openjournals/joss-reviews/issues/3992)

MLRichter commented 2 years ago

Resolved in PR https://github.com/delve-team/delve/pull/62, waiting for approval.