Closed nirgoren closed 6 months ago
Hi @nirgoren ,
You're an old version of torchdistill. It is already resolved in torchdistill v1.0.0
Hi @nirgoren ,
You're an old version of torchdistill. It is already resolved in torchdistill v1.0.0
I see. Note that pip automatically downloaded v0.3.3 for me due to having torch v2.2.1 installed and v1.0.0 of torchdistill requiring a version of torch that is <=2.1.0
. It would be nice to have compatibility with the latest version of torch, if that doesn't cause any issues.
You could still use torchdistill v1.0.0 with the latest torch if you force-install torchdistill though I've not tested it.
From torchdistill v1.0.0, I started specifying min/max versions of key requirements to avoid the same mistakes as the old torchdistill had e.g., torchdistill v0.3.3.
Next release of torchdistill supports the latest torch and is around the corner, probably next week. Stay tuned!
Bug description When trying to import
torchdistill.core.forward_hook
with an up-to-date version of pytorch, you getModuleNotFoundError: No module named 'torch._six'
error.To Reproduce
import torchdistill.core.forward_hook
Expected behavior The import should succeed without errors.
Environment:
Additional context Related to this change in pytorch: https://github.com/pytorch/pytorch/pull/94709 Apparently
string_classes
is no longer needed andstr
can be used instead.