<@U0BLPSJAK>, something wrong with division :sligh...
# korgey
y9san9
09/05/2021, 5:32 PM@Deactivated User, something wrong with division 🙂
y9san9
09/05/2021, 5:42 PMokay the problem was because of big precision (this is a standard in cryptoworld)
y9san9
09/05/2021, 5:50 PMthis also causes problems when using
times (*)d
Deactivated User
09/05/2021, 11:05 PMcan you provide an
assertEqualsy
y9san9
09/06/2021, 8:50 AMOkay, I will do that asap
y9san9
09/06/2021, 4:38 PMready, added my comment on the issue https://github.com/korlibs/kbignum/issues/6
👍 1
y9san9
09/08/2021, 6:54 PMThis is not related to division, it's kind of overflow of something, I don't know, but I really need this fix :(
d
Deactivated User
09/08/2021, 7:00 PMwhat do you mean about it not being related to div? can you reproduce it with another operation?
Deactivated User
09/08/2021, 7:01 PMI was going to check sometime this weekend, not sure if I will have time before, but if you have more info or other operations that fail, please provide them so I can check them along the div when I have some time to check
y
y9san9
09/08/2021, 7:01 PMSorry, my miss! The problem was that .bi does not take radix in my case. But it's an another issue, please make it to throw an exceptions if there is an unknown symbols...
y9san9
09/08/2021, 7:02 PMand also the problem was that there is
0xd
Deactivated User
09/08/2021, 7:02 PMoh i see
Deactivated User
09/08/2021, 7:02 PMwell, we can support that I guess
y
y9san9
09/08/2021, 7:04 PMThanks to you a lot for your work! K
K 1
d
Deactivated User
09/08/2021, 7:17 PMRegarding to the checks and prefix support: https://github.com/korlibs/korge-next/commit/fd43fdff8bf05dfe638971262d0911fdc75b003d
Deactivated User
09/08/2021, 7:48 PMAnd also checked the division issue and fixed it here: https://github.com/korlibs/korge-next/commit/3068d8ef175ed879ea4d6849006e2c08b05edd40
y
y9san9
09/08/2021, 9:49 PMGood job!
y9san9
09/14/2021, 1:34 PM@Deactivated User when can you publish the new library version?
d
Deactivated User
09/14/2021, 2:11 PMI didn't have plans for any soon release. Since I usually publish releases for all the libraries at once when there is significative work.
But since it seems you need it, I have backported the changes to the kbignum repo, and triggered a build here:
• https://github.com/korlibs/kbignum/releases/tag/v2.4.1.1
• https://github.com/korlibs/kbignum/actions/runs/1233972729
y
y9san9
09/15/2021, 4:23 AMThanks a lot again 😁
👍 1