Skip to content

Fix empty progress bar#9144

Merged
jacobtomlinson merged 2 commits intodask:mainfrom
jacobtomlinson:fix/empty-progress
Nov 17, 2025
Merged

Fix empty progress bar#9144
jacobtomlinson merged 2 commits intodask:mainfrom
jacobtomlinson:fix/empty-progress

Conversation

@jacobtomlinson
Copy link
Member

@jacobtomlinson jacobtomlinson commented Nov 14, 2025

Closes #9143

  • Calls MultiProgressWidget.make_widget(tasks) even if progress is given an empty iterable of tasks
  • If there are no tasks display no tasks given instead of the completion time
image
@github-actions
Copy link
Contributor

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

    27 files  ±0      27 suites  ±0   9h 43m 57s ⏱️ - 2m 31s
 4 112 tests ±0   4 007 ✅ +1    104 💤 ±0  1 ❌  - 1 
51 516 runs  ±0  49 331 ✅ +1  2 184 💤 ±0  1 ❌  - 1 

For more details on these failures, see this check.

Results for commit 236fb27. ± Comparison against base commit c7d9c55.

Copy link
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

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

Thanks @jacobtomlinson. LGTM overall. Might be worth adding a small test for the empty case but up to you

@jacobtomlinson jacobtomlinson merged commit 23d78c8 into dask:main Nov 17, 2025
58 of 60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants