You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi, thanks for your work on indicatif, which we've been happily using for years now.
Recently, an overflow bug became apparent to us and has already been fixed (thanks!). However, I seem to have stumbled across another one. Here is the link to our CI job as well as the offending line of code:
I'm almost certain the issue stems from using a progress bar somewhere with very little work to do, but I'm sure you agree that these potential issues should be handled, probably like the other overflow bug (i.e. saturating add). I'm happy to contribute code to fix the bug.
The text was updated successfully, but these errors were encountered:
Hi, thanks for your work on
indicatif
, which we've been happily using for years now.Recently, an overflow bug became apparent to us and has already been fixed (thanks!). However, I seem to have stumbled across another one. Here is the link to our CI job as well as the offending line of code:
https://github.com/MWATelescope/Birli/actions/runs/5828640212/job/15806705268#step:4:234
indicatif/src/multi.rs
Line 293 in 27c8399
I'm almost certain the issue stems from using a progress bar somewhere with very little work to do, but I'm sure you agree that these potential issues should be handled, probably like the other overflow bug (i.e. saturating add). I'm happy to contribute code to fix the bug.
The text was updated successfully, but these errors were encountered: