Skip to content

Remove hard tensorboardX requirement#1571

Merged
jeffra merged 3 commits intomasterfrom
remove-tbx
Nov 18, 2021
Merged

Remove hard tensorboardX requirement#1571
jeffra merged 3 commits intomasterfrom
remove-tbx

Conversation

@jeffra
Copy link
Copy Markdown
Collaborator

@jeffra jeffra commented Nov 17, 2021

PyTorch 1.2+ has support and same API as tensorboardX, this PR switches to torch.utils.tensorboard.

Fixes #1570

Refresh on an old unmerged PR #330

@jeffra jeffra marked this pull request as draft November 17, 2021 23:09
@jeffra jeffra changed the title Remove hard tensorboardX requirement [WIP] Remove hard tensorboardX requirement Nov 17, 2021
@jeffra jeffra marked this pull request as ready for review November 17, 2021 23:17
@jeffra jeffra changed the title [WIP] Remove hard tensorboardX requirement Remove hard tensorboardX requirement Nov 17, 2021
@jeffra jeffra enabled auto-merge (squash) November 18, 2021 01:15
@jeffra jeffra merged commit a90497e into master Nov 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

possible relaxation on the tensorboardX hardwired version requirement

2 participants