[Bug] Time distribution intervals shift and have duplicates due to rounding

[GoldenWolf5]

[Bug] Time distribution intervals shift and have duplicates due to rounding

[pdtxie]

I'm not quite sure I follow this error, the time distribution will show the distribution of your times throughout the entire session, and by how it's currently written now, will dynamically readjust the column widths depending on your solves. When you do a solve, each bar interval won't be the same as before the solve as to produce a nice looking graph.

Could you explain in a bit more detail what you should expect when you had finished the solve in the video (the 1.320)?

[GoldenWolf5]

In the video before the solve, the increment of below 31 is 276. After the solve, of 1.320, which is under 31, the counter is still at 276. But instead, 31+ went down by 4, the 34+ went up by 4, the 37+ went down by 2, and so on. Shouldn’t 1 just be added to the <31, making it 277 and everything else stays the same?

…

On Fri, Mar 24, 2023 at 4:51 PM Tim Xie ***@***.***> wrote: I'm not quite sure I follow this error, the time distribution will show the distribution of your times throughout the entire session, and by how it's currently written now, will dynamically readjust the column widths depending on your solves. When you do a solve, each bar interval won't be the same as before the solve as to produce a nice looking graph. Could you explain in a bit more detail what you should expect when you had finished the solve in the video (the 1.320)? — Reply to this email directly, view it on GitHub <#152 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A6VQQFIH7NLZBW3RBX2FFOTW5YXRJANCNFSM6AAAAAAWHELC7Y> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[pdtxie]

yeah... I think what happened is it cascaded down, so the 1.3 got added to the first increment, but changed the bracket decimal (so <31 stays <31 but probably changed from like 31.4 to 31.2 as an example), which kicked out one solve (31.[2-3]x) keeping the number of <31 the same, but then causing a cascade shift for all the other intervals.

I know this is probably frustrating to hear - but this was initially implemented as just a small widget for a rough graph - a more detailed interactive chart is in the works

[pdtxie]

Mentioned in #207

[pdtxie]

Mentioned in #158