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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace tensorboard by CSVLogger #19

Open
Borda opened this issue Jun 7, 2021 · 9 comments
Open

replace tensorboard by CSVLogger #19

Borda opened this issue Jun 7, 2021 · 9 comments
Labels
enhancement New feature or request Example Example / Demo / Tutorial good first issue Good for newcomers help wanted Extra attention is needed

Comments

@Borda
Copy link
Member

Borda commented Jun 7, 2021

馃殌 Feature

Replace inline tensorboard (TB) with a simple chart from standard CSLoggers

Motivation

The inline tensorboard is a cool interactive tool, but:

  • it requires saved logos that are missing in the notebook
  • TB inside notebooks is a heavy JS object (~30MB)

Pitch

A reader would also see the progress immediately, maybe add plotting to the csvlogger?

Alternatives

Some other simple logger?

Additional context

@Borda Borda added enhancement New feature or request help wanted Extra attention is needed Example Example / Demo / Tutorial good first issue Good for newcomers labels Jun 7, 2021
@Borda Borda added this to To do in Improve Notebooks Aug 1, 2021
@InCogNiTo124
Copy link

I'd like to work on this :)

@Borda
Copy link
Member Author

Borda commented Aug 17, 2021

Cool, let's do all in one PR 馃惏

@InCogNiTo124
Copy link

@Borda im not sure how do I add plotting to the csv logger?

@Borda
Copy link
Member Author

Borda commented Aug 22, 2021

@Borda im not sure how do I add plotting to the csv logger?

see the visualization over here: https://github.com/Borda/kaggle_plant-pathology/blob/main/notebooks/Plant-Pathology-with-Lightning.ipynb

@kaushikb11
Copy link
Contributor

@Borda Why not add both the loggers?

@Borda
Copy link
Member Author

Borda commented Aug 26, 2021

@Borda Why not add both the loggers?

good point, we can :]

@InCogNiTo124
Copy link

@Borda Why not add both the loggers?

good point, we can :]

Doesn't that kill the point:

  • TB inside notebooks is a heavy JS object (~30MB)

@Borda
Copy link
Member Author

Borda commented Aug 26, 2021

yes, the suggestion is use both loggers but show the chart only from the CSV logger
but if a user wants he would have also logs in tensor board and show/open it separately

@InCogNiTo124
Copy link

Hey, I'm just letting you know that I won't be able to finish this up. Someone else should take it. sorry!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Example Example / Demo / Tutorial good first issue Good for newcomers help wanted Extra attention is needed
Projects
Development

No branches or pull requests

3 participants