microsoft / DeepSpeed

DeepSpeed is a deep learning optimization library that makes distributed training and inference easy, efficient, and effective.
https://www.deepspeed.ai/
Apache License 2.0
35.38k stars 4.11k forks source link

TensorBoardX is too old in requirements.txt #789

Closed surak closed 1 month ago

surak commented 3 years ago

tensorboardX 1.8 is from mid 2019. Version 1.9 is from October, 2019, and there are 2.0. and 2.1 already.

It's difficult to install legacy versions of softwares on our supercomputers just because of one software.

aphedges commented 3 years ago

I'm not really sure why it is pinned. Looking through the Git history, it seems that tensorboardX==1.8 has been in requirements.txt since the file was added in 577295f.

I'm using a version of pip old enough to allow me to install conflicting dependencies, but this will be a problem once I update. There doesn't seem to be much direct usage of tensorboardX in the codebase, so I hope this will be an easy change for them (or for a user, if we need to do it ourselves).

aphedges commented 2 years ago

tensorboardX was removed as a direct dependency in a90497e, so this issue should be closed.