Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tensorboard logger should support remote directories #2161

Closed
f4hy opened this issue Jun 12, 2020 · 2 comments · Fixed by #2164
Closed

tensorboard logger should support remote directories #2161

f4hy opened this issue Jun 12, 2020 · 2 comments · Fixed by #2164
Labels
feature Is an improvement or enhancement help wanted Open to be worked on

Comments

@f4hy
Copy link
Contributor

f4hy commented Jun 12, 2020

🚀 Feature

Tensorboard allows you to write to gc, s3, hdfs, etc by specifying paths with the right prefix e.g. logDir='hdfs://path/to/logs/

However the lightning logger breaks this. see tensorboard.py#L99

Motivation

Training often occurs on remote clusters which don't persist the local disk at the time the job ends. The local disk is also not accessible from outside tools so tensorboard can not access the results while training is in progress.

Pitch

Replace all directory operations with some remote-aware tool. tensorboard itself provides a gfile compatible handle . There are other options as well. tensorboard itself supports these things natively so maybe we can just get around doing any local file operations and leverage tensorboard lib to write remotely.

Alternatives

Some other option would be to write locally but add hooks to sync the logs to a remote storage.

@f4hy f4hy added feature Is an improvement or enhancement help wanted Open to be worked on labels Jun 12, 2020
@github-actions
Copy link
Contributor

Hi! thanks for your contribution!, great first issue!

@f4hy
Copy link
Contributor Author

f4hy commented Jun 16, 2020

Whats the right way to discuss solutions here. I have one proposed solution in a draft PR, but figure its best to discuss with the maintainers first about how best we can solve this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Is an improvement or enhancement help wanted Open to be worked on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant