Closed chucker closed 2 months ago
Hmm. As the tests demonstrate, there's a bit of a problem here. In the tests, the snippet is just 27
chars. 3% of that is below one char, so the initial guess just shows… nothing.
I'm unsure if this means we should never have such a short snippet (probably), or if we need to add something that rounds up? (I think we effectively currently round down.)
Yeah, use max(5, length * progress)
as the number of revealed characters.
I've made some changes, but I'm not sure I've solved the underlying problem.
Oh, never mind. You already wrote "lgtm".
It previously started out at 10%, and proceeded in increments of 10% each. Now it starts out at 3% and increments by 5%.