fix: improve usage of indicatif for tracking progress #101

letian · 2025-08-10T01:34:19Z

letian commented

2025-08-10 01:34:19 +00:00

The multiprogress wasn't used properly and leading to conflicting progress bars (within our own progress bars, as well as the log wrapper).

This PR introduce a layer on top of indicatif::MultiProgress to properly handle sections of progress bars, where we can dynamically add/update/remove progress bars from any sections.

We can see in the demo that new sections + progress bars are added on the fly and that extra logs (e.g. info logs) are appended on top of the progress bars.

Progress are also grouped together based on their parent score.

The multiprogress wasn't used properly and leading to conflicting progress bars (within our own progress bars, as well as the log wrapper). This PR introduce a layer on top of `indicatif::MultiProgress` to properly handle sections of progress bars, where we can dynamically add/update/remove progress bars from any sections. We can see in the demo that new sections + progress bars are added on the fly and that extra logs (e.g. info logs) are appended on top of the progress bars. Progress are also grouped together based on their parent score.

demo-1.gif

583 KiB

letian added 2 commits 2025-08-10 01:34:23 +00:00

refactor: Interpret score with a provided method on Score

Run Check Script / check (pull_request) Has been cancelled

Details

2c8f45c44e

instrument interpret state changes (wip)

Run Check Script / check (pull_request) Has been cancelled

Details

9db475849c

letian force-pushed better-indicatif-progress from 26f72f0214 to 336e1cfefe