Skip to content

Switches from tensorboardX to torch.utils.tensorboard#330

Closed
ShadenSmith wants to merge 4 commits intodeepspeedai:masterfrom
ShadenSmith:drop-tensorboardx
Closed

Switches from tensorboardX to torch.utils.tensorboard#330
ShadenSmith wants to merge 4 commits intodeepspeedai:masterfrom
ShadenSmith:drop-tensorboardx

Conversation

@ShadenSmith
Copy link
Copy Markdown
Contributor

Tested manually, things seem to work as expected.

@ShadenSmith ShadenSmith added the dependencies Pull requests that update a dependency file label Aug 21, 2020
@ShadenSmith ShadenSmith requested review from jeffra and tjruwase August 21, 2020 08:09
@jeffra
Copy link
Copy Markdown
Collaborator

jeffra commented Aug 21, 2020

After we figure out the fix for this build issue, can we remove it from here too? https://github.com/microsoft/DeepSpeed/blob/master/Dockerfile#L50

@ShadenSmith ShadenSmith marked this pull request as draft September 6, 2020 15:16
@rocm-mici
Copy link
Copy Markdown

Can one of the admins verify this patch?

@jeffra jeffra closed this Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants