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

Progress bar for neural network training #2216

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

HenKlei
Copy link
Member

@HenKlei HenKlei commented Nov 2, 2023

This PR adds a progress bar using rich for the neural network training.

@HenKlei HenKlei added the pr:new-feature Introduces a new feature label Nov 2, 2023
@HenKlei HenKlei added this to the 2023.2 milestone Nov 2, 2023
@HenKlei HenKlei requested a review from sdrave November 2, 2023 08:10
@HenKlei HenKlei self-assigned this Nov 2, 2023
@HenKlei HenKlei linked an issue Nov 2, 2023 that may be closed by this pull request
Copy link
Member

@sdrave sdrave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Before looking into details, we have to agree on

  1. Whether to use rich or another library for progress bars. That library needs to be added to pyMOR's core dependencies I guess.
  2. Whether to have an internal progress bar interface which wraps around the chosen library and that is used everywhere.

Further, have you checked what happens if the output is piped into a file, for instance via tee?

Edit: It also has to play nicely in a jupyter environment.

@sdrave
Copy link
Member

sdrave commented Nov 2, 2023

@HenKlei, maybe something to discuss at today's developer meeting?

@HenKlei
Copy link
Member Author

HenKlei commented Nov 2, 2023

@HenKlei, maybe something to discuss at today's developer meeting?

Yes, good idea. I will put it on the agenda.

@HenKlei
Copy link
Member Author

HenKlei commented Nov 2, 2023

The progress bar is not interactive anymore when piped through tee....

@sdrave sdrave removed this from the 2023.2 milestone Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:new-feature Introduces a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Progress bar?
2 participants