[Bug] too much dice rolls (245) doesn't show stats for nerds graph

selfcustody / krux

Open-source signing device firmware for Bitcoin

https://selfcustody.github.io/krux/

Other

179 stars 36 forks source link

[Bug] too much dice rolls (245) doesn't show stats for nerds graph #348

Closed tadeubas closed 6 months ago

tadeubas commented 6 months ago

Summary

Most of dices 1 and some 20

Device

[ ] Amigo
[ ] M5stickV
[X] Dock
[ ] Bit
[ ] Yahboom

Version

[ ] Official release (selfcustody/krux):
[X] Beta release (odudex/krux_binaries):

Describe the bug

When generating a new mnemonic rolling d20 dices, when most of dices gives 1 and few gives 20, with 245 rolls the stats for nerds doesn't show rolls distribution graph correctly

odudex commented 6 months ago

Fixed with 29b94686f7e6c786f0aea632c5950a3c6674e012 Please let me know otherwise.