Revised progress bar ETA about textual HOT 3 CLOSED

While making a start on this I ran into some confusing results and ended up uncovering #4096; so I think this will result in a revamp of how ProgressBar does the calculations even in the most simplistic of situations.

Some form of "please reset this ProgressBar for reuse" method might also be a good idea.

from textual.

WiP on this has addressed #4096, but also uncovered a further issue where the ETAStatus appeared to be updating when it wasn't necessary; this has now been identified and fixed too.

An updated method of calculating the ETA is now in place and seems to produce reasonable results; for constant-time progress the timer looks spot on; and for accelerating, decelerating and reasonably random progress the results look good too, and generally more/less optimistic/pessimistic where appropriate.

Work on this is having further knock-on effects on how the documentation screen shots are generated, and also their use in snapshot tests; this will likely need some more work on how the docs for ProgressBar are handled, and might require removal of that code from snapshot tests. If the latter happens we'll need to do a bit of work on producing unit tests for the ETA calculation (which might require isolating the ETA code somewhat).

A question has also been raised about the newer approach taken to slide a window over the updates to perform the calculation.

+1d

from textual.

Don't forget to star the repository!

Follow @textualizeio for Textual updates.

from textual.