Repository navigation
Progress bar is duplicated when used both as a context manager and as an iterable wrapper #301
Description
Description
When used both as context manager (e.g. with) and as iterable wrapper, exhausting the iterable and exiting the with scope causes the progress bar to duplicate.
Code
Consider the following example code:
from time import sleep
from progressbar import ProgressBar
from progressbar.widgets import AnimatedMarker, SimpleProgress, GranularBar, Timer
def simple_iterator():
for i in range(100):
sleep(0.02)
yield i
progress = ProgressBar(
max_value=100,
redirect_stdout=True,
redirect_stderr=True,
widgets=[AnimatedMarker("⡏⠟⠻⢹⣸⣴⣦⣇"), " ",
SimpleProgress(), " ",
GranularBar(markers="░▒▓█"), " ",
Timer()]
)
with progress:
for i in progress(simple_iterator()):
passASCIInema demo: https://asciinema.org/a/UEEc81s5GyrAR3QaqLDWGIMGX
Versions
- Python version:
3.12.6 (main, Sep 8 2024, 13:18:56) [GCC 14.2.1 20240805] - Python distribution/environment: CPython
- Operating System: Arch Linux, etc.
- Package version:
4.5.0
Activity
dreamscached commentedon Sep 22, 2024
I assume
finishis invoked twice for some reason, because leaving in either a context manager only or iterable wrapper only, it works as expected.dreamscached commentedon Oct 30, 2024
Bumping because it's still an issue and was not solved.
wolph commentedon Oct 30, 2024
Sorry about he inactivity, I've seen the issue but I've had very little time.
It's definitely a bug :)
Address issues #212, #295 and lock in #301
Address issues #212, #295 and lock in #301
wolph commentedon Jul 18, 2026
Fixed on
develop: using a bar as both a context manager and an iterable no longer finishes/draws it twice. Locked bytests/test_with.py::test_context_manager_and_iterable_no_duplicateand the reporter's-exact-widget-set varianttest_context_manager_and_iterable_reporter_widgets_no_duplicateadded in #326. Closing.Merge pull request #326 from wolph/harden-issue-tests-2