Skip to content

Conversation

@boegel
Copy link
Contributor

@boegel boegel commented Sep 19, 2021

Type of changes

  • Bug fix
  • New feature
  • Documentation / docstrings
  • Tests
  • Other: examples

Checklist

  • I've run the latest black with default args on new code.
  • I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
  • I've added tests for new code.
    • Is this required for examples too? If so, happy to look into it...
  • I accept that @willmcgugan may be pedantic in the code review.

Description

Additional example for progress bars, more dynamic in nature than the existing live_progress.py; see also discussion in #1500 .

Screenshot while running:

image

Screenshot when done:

image

Copy link
Member

@willmcgugan willmcgugan left a comment

Choose a reason for hiding this comment

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

Looks good! Couple of small requests.

Could you run this against the latest black?


import time

from rich.console import RenderGroup
Copy link
Member

Choose a reason for hiding this comment

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

RenderGroup was deprecate, the new name is just Group

)
# group of progress bars;
# some are always visible, others will disappear when progress is complete
group = RenderGroup(
Copy link
Member

Choose a reason for hiding this comment

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

Should be Group here. Could you also give group a more descriptive name?

@willmcgugan
Copy link
Member

Nice work!

Could you maybe put the app progress in a Panel to separate it visually? And speed it up a fraction...

@willmcgugan
Copy link
Member

Any chance you can have another look at this?

@boegel boegel force-pushed the dynamic_progress_example branch from 014adfb to 28a33cb Compare October 16, 2021 11:52
@boegel
Copy link
Contributor Author

boegel commented Oct 16, 2021

Any chance you can have another look at this?

Sorry for not getting back to this sooner.

Requested changes done, and rebased on latest master to fix merge conflict on CHANGELOG.md.

@boegel
Copy link
Contributor Author

boegel commented Nov 20, 2021

@willmcgugan Anything in particular that's blocking this PR? Happy to make additional changes if needed.

@willmcgugan willmcgugan merged commit 666d0cf into Textualize:master Nov 20, 2021
@willmcgugan
Copy link
Member

Sorry, forgot about this one. Looks good. Thanks!

@boegel boegel deleted the dynamic_progress_example branch November 20, 2021 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants