[02:37] *** lucasb left [03:33] *** MasterDuke left [06:19] *** domidumont joined [06:42] good *, #moarvm [07:03] Good Friday! (Fridays are a thing on IRC, aren't they?) [07:04] I believe so. But they seem to come and go. [07:12] *** patrickb joined [07:16] patrickb: so we go deeper down the rabbit hole... [07:16] good * (morning over here) :-) [07:17] The C library pow() (at least for glibc) is accurate for all powers of 10 [07:17] but attempting to build those powers of 10 with floating point multiplication by the algorithm used in the pow_i OP isn't. [07:17] sigh [07:17] and then I found [07:17] r: say 1e23 [07:17] rakudo-jvm 08b449e1a: OUTPUT: «Error while reading '/home/camelia/p6eval-token': Permission denied at /home/camelia/rakudo-j-inst/bin/eval-client.pl line 10.␤» [07:17] ..rakudo-moar 6fdc2b690: OUTPUT: «9.9999999999999992e+22␤» [07:17] Should I try that on mingw? [07:17] no, no need [07:18] because we (er, I) probably need to fix that one first [07:19] the problem seems to be (have not tested this with a debugger) that 1e23 (and one other) end up as binary that grissu3 can't format. So it falls back to sprintf [07:19] and we have sprintf with %17g [07:19] and that's, um, not correct here [07:19] but likely %15g isn't correct for other values. [07:20] the soution seems to be to move (it all) to Ryū [07:21] which has a suitably licensed implentation written in C [07:21] and then tackle the previous step. Worst case is 309 8-byte values in a lookup table. [07:22] But that's not for today. [07:22] I have no coffee. What could possibly go wrong? :-) [07:23] oh. That sounds like a fair share of pain. I recall how much effort and reading up zoffix once did to arrive at the grissu3 solution we currently have... [07:23] nwc10: Thank you for championing this. [07:25] It's getting worrying - I can look at code that has if (e == 0 and m ==0) { ... } and know what that means just as-is [07:25] ao [07:26] sorry keyboard switched configs [07:26] exponent =0 and mantissa = 0 -> it's a zero float? [07:28] patrickb: that would e too obvious :) [07:28] (typo pun :-) [07:29] ah I wish that was on purpose :) [07:35] yes. [08:11] *** zakharyas joined [08:53] *** Geth joined [09:51] o/ [10:02] \o [10:47] *** linkable6 left [10:49] *** linkable6 joined [11:23] *** zakharyas left [11:33] *** hankache joined [11:38] *** frost-lab joined [11:40] *** hankache left [11:51] *** frost-lab left [12:52] *** Kaeipi joined [12:54] *** Kaeipi left [12:55] *** Kaiepi left [13:35] *** sena_kun left [13:37] *** sena_kun joined [14:08] *** zakharyas joined [14:09] *** sena_kun left [14:11] *** sena_kun joined [15:00] *** cog left [15:00] *** cog joined [16:12] *** cog left [16:13] *** cog joined [16:25] *** domidumont left [18:32] *** MasterDuke joined [19:15] nine: any reason you didn't PR https://github.com/MoarVM/MoarVM/commit/74505ce54d ? [19:18] *** MasterDuke left [19:19] *** MasterDuke joined [19:34] *** zakharyas left [19:35] *** zakharyas joined [19:42] Somewhere between https://github.com/MoarVM/MoarVM/commit/ba124ad12875f3f4f85bc2433e92b7a5f0aaa47c and https://github.com/MoarVM/MoarVM/commit/9481289950b89dabb553e63d277038518c590b92 the App::Uni module was broken, we need to investigate and fix it. [19:45] it's definitely a moarvm-caused problem, not all the unicode-related commits in rakudo between that timespan? [19:47] hm, but there are some string ops commits in there... [19:48] MasterDuke, it bisects to the moarvm bump and I checked manually it's indeed the bump. [19:49] oh, but i just cloned it and the tests pass for me [19:51] hm, i wasn't at HEAD everywhere, let me try again... [19:54] yup, still pass [19:56] hm, that blin gist says it's version 1.0.2, but the git clone says 1.0.1... [19:56] oh, but its META6.json does say 1.0.2 [19:58] [Coke]: does it fail locally for you? [20:00] also installs fine with zef [20:08] [Coke]: also, the repo description still says "Perl 6 - command-line utility" and the readme says unicode 12, but we're now using 13.1 [20:15] [Coke]: Any interest in PRs to merge in some of the functionality from https://gist.github.com/japhb/7b8a68da1b4b6c201e5334c1269b951a ? [20:22] *** zakharyas left [20:54] *** patrickb left [20:55] MasterDuke: wait, I didn't? [20:57] MasterDuke: oh, I simply forgot after pushing the branch. But... there's more to come on that anyway. I found other places where we missed propagating the facts. Will commit and push those and then create the PR [20:57] cool