00:56
jnap joined
|
|||
timotimo | i forgot to change the temporary thing in bigint_div, i only did it in bigint_div_num >_> | 01:45 | |
dalek | arVM: 2e5deb8 | (Timo Paulssen)++ | src/ (2 files): try to get integer division with negative a behave. |
01:55 | |
arVM: 8b259f7 | (Timo Paulssen)++ | src/core/interp.c: make sure to abs arguments to gcd_i |
|||
arVM: 9cf4c8b | (Timo Paulssen)++ | src/math/bigintops.c: use temporary mp_ints for subtracting 1 in bigint_div* |
|||
arVM: 23c14d7 | (Timo Paulssen)++ | src/ (2 files): Merge branch 'negative_integer_division' eab5033 | (Timo Paulssen)++ | src/ (2 files): replace completely wrong appreach with better approach |
|||
02:03
benabik joined
|
|||
diakopter | timotimo++ | 02:19 | |
anyone with a rakudo push bit? | 02:20 | ||
timotimo | no, it's not timotimo++ | 02:21 | |
i did it the wrong way around :) | |||
diakopter | timotimo: can you patch something in rakudo for me | ||
timotimo | gotta run for a light rail now | ||
diakopter | replace all the occurences of perl6-m$(BAT) with $(M_RUNNER) in the template for moar's Makefile | 02:22 | |
er | 02:24 | ||
well, that, and the $(BAT) definition needs added back in at the top, but it seems someone removed it | |||
when on windows. | |||
gah | 02:25 | ||
[Coke] | sounds complicated. | ||
timotimo | it would seem to me that if div rounds towards zero but should floor instead, i need to add 1 to the right hand side to force a floor if the lhs is lt 0 | 02:29 | |
div referring to C div now | |||
but doing that did not seem to help | 02:32 | ||
diakopter | timotimo: you mean subtract 1? | 02:33 | |
timotimo | yes i did mean that | 02:34 | |
since i want the value to be bigger absolutely | |||
what do i do if a is positive and b negative, though? | 02:35 | ||
that case is not handled by the rat code | |||
diakopter | wait | ||
timotimo | because there the denominator is forced to be positive | ||
diakopter | no you wouldn't subtract 1, you would subtract b..? | ||
timotimo | i would subtract b?? | ||
diakopter | er | ||
you would subtract 1 from the result if a % b != 0 | 02:36 | ||
timotimo | i don't think so | 02:37 | |
i should be able to maybe i should subtract one from the *result* if that condition holds | |||
not backspaces enough | 02:38 | ||
i think you are right though. but i should do it completely differently | 02:39 | ||
i should divide the absolute values and negate if necessary | |||
that should give the correct result and not need to calculate the modulus | 02:40 | ||
i thinkbthat is what i will do | 02:42 | ||
if you see random b where words should be separated you know I'm on my phone keyboard | 02:44 | ||
03:01
jnap joined
|
|||
timotimo | hm. there is no implementation of integer division in libtommath that behaves like java does | 03:05 | |
hehe, the latest change to libtommath is merging a pull request made by moritz to fix mod on negative integers :D | 03:09 | ||
i'm actually not sure about MVM_bigint_div_num | 03:20 | ||
the / operator there does floating point division, doesn't it? | |||
03:21
FROGGS[mobile] joined
|
|||
timotimo | because there's mp_get_double on both sides of the / | 03:21 | |
ah, of course, the result is a num. | |||
so no magic needed | |||
timotimo should probably have tested before pushing | 03:22 | ||
yup. | |||
of course i made several mistakes | 03:23 | ||
i shouldn't be up coding at this hour. | |||
03:23
FROGGS[mobile] joined
|
|||
dalek | arVM: 71fab40 | (Timo Paulssen)++ | src/ (2 files): replace completely wrong appreach with better approach |
03:23 | |
timotimo | nobody saw that. | ||
now things are back to 20 failing tests :| | 03:31 | ||
you know what. forget about all of this. | 03:36 | ||
03:36
FROGGS[mobile] joined
|
|||
timotimo | this never happened. until tomorrow. when i'll retry and not be as sleep deprived. | 03:36 | |
04:02
jnap joined
|
|||
diakopter | timotimo: :) it's okay; we've all been there | 04:31 | |
diakopter especially, apparently | 05:15 | ||
timotimo | diakopter: you were right about the remiander check | 05:23 | |
i should have listened to you immediately. | |||
dalek | arVM: 17effc2 | (Timo Paulssen)++ | src/ (2 files): fix div_i and div_I to floor rather than round towards 0 |
05:24 | |
arVM: f2ce6f8 | (Timo Paulssen)++ | src/core/interp.c: gcd needs to abs both arguments before calculating |
|||
timotimo | this makes rounders.t pass completely | 05:25 | |
maybe now i can fall asleep | |||
diakopter | timotimo: well, I'm quite certain that's not anywhere near the "right" way to do it, even if it works | ||
timotimo | are you sure? | ||
diakopter | for those tests | ||
yea, I'm sure there's a branchless version | |||
timotimo | at least for the bigint stuff we get the remainder for free | ||
well, since it's - 1 in the true case and - 0 in the not-true case, yeah | |||
trivial branchless version | |||
er ... except for the outer branch | 05:26 | ||
but that can be done with a multiplication. | |||
diakopter | confounded again! ;) | ||
timotimo | the compiler (or the cpu, even) might be able to do this. | ||
diakopter | yeah | ||
timotimo | we're in the "make it work" stage right now anyway | ||
diakopter | the sufficiently smart compiler knows and sees all | ||
well, enough, anyway | 05:27 | ||
timotimo | it certainly does | ||
i wonder how the rest of rakudo copes with the div difference between jvm/moarvm and parrot | |||
diakopter | extremely sparse test coverage? | ||
timotimo | that must be it, yeah | ||
timotimo opens a quest on questhub | 05:28 | ||
questhub.io/realm/perl/quest/52d0d7...7c32000061 ← | 05:30 | ||
This type cannot unbox to a native string in method Str at src/gen/m-CORE.setting:1020 | 05:40 | ||
in method abs2rel at src/gen/m-CORE.setting:12607 | |||
in block frame_name_17 at t/spec/S32-io/io-spec-win.t:266 | |||
in block frame_name_1 at t/spec/S32-io/io-spec-win.t:257 | |||
i may look into this tomorrow :) | |||
hmm: not ok 823 - Rat.Rat yields correct Rat | 05:41 | ||
# got: '301281685344656669 1250' | |||
# expected: '301281685344656640 1250' | |||
somehow a num got used instead of a big integer or something? | 05:42 | ||
that difference is surprisingly big | |||
Files=767, Tests=29473, 912 wallclock secs ( 8.16 usr 1.09 sys + 3192.27 cusr 69.95 csys = 3271.47 CPU) | 05:43 | ||
the last entry in perl6_pass_rates.csv has 26962; is that Tests= number the number of succeeded tests or run tests? | 05:44 | ||
anyway. bed. | 05:45 | ||
jnthn | Whoa...timotimo++ hacked all night... | 08:06 | |
timotimo: yay, rounders.t now passes fully for me too | 08:16 | ||
08:19
ssutch joined
08:23
eternaleye joined
|
|||
FROGGS[mobile] | morning o/ | 08:46 | |
08:46
Woodi joined
11:09
cognominal joined
|
|||
jnthn | Epic win! The lounge at Frankfurt has a drink-all-you-want weissbier tap | 11:10 | |
FROGGS | \o/ | 11:15 | |
jnthn: what are you in frankfurt for? | |||
jnthn | FROGGS: Oh, just connecting here | 11:17 | |
FROGGS | good :o) | ||
jnthn | It's miserable weather. :) | 11:18 | |
FROGGS | it is very nice in berlin though | ||
11:21
FROGGS[mobile] joined
|
|||
masak | I can see Weissbier being very drink-all-you-want-able. | 11:33 | |
11:34
FROGGS[mobile] joined
|
|||
JimmyZ | E:\OpenSource\MoarVM\install\bin\moar --libpath=src\vm\moar\stage0 src\v | 11:34 | |
m\moar\stage0\nqp.moarvm --bootstrap --setting=NULL --no-regex-lib --target=mbc | |||
--output=gen\moar\stage1\nqpmo.moarvm gen\moar\stage1\nqpmo.nqp | |||
MAST::Local index out of range | |||
jnthn | JimmyZ: You'll almsot certainly have an out of date something or other... | 11:35 | |
OK, got a patch here that puts the serialization blob into the bytecode file as is rather than base-64-ing it. | 11:36 | ||
Shaves a whole megabyte off the build output of NQP | |||
nwc10 | cool | 11:39 | |
when does your plane leave? | |||
jnthn | Boarding starts in 20 ins | 11:40 | |
*mins | |||
nwc10 | Boarding started here. (Into the car. It takes ages these days :-/) | 11:41 | |
masak | O(@family) | 11:44 | |
jnthn | make test in NQP used to be 23s on my laptop. Now it's 21s. | ||
r: say 100 * 21 / 23 | |||
camelia | rakudo-parrot 263aa0, rakudo-jvm 263aa0: OUTPUT«91.304348» | ||
masak | \o/ | 11:45 | |
jnthn | Shaved about 5MB off the memory used by nqp-m -e "while 1 { }" too | 11:48 | |
dalek | arVM: cf39b8f | jonathan++ | src/ (5 files): Enable storing serialized data in bytecode section This avoids the whole base-64 of an MVMString and so forth, speeding both startup and compilation, cutting memory footprint and decreasing the disk size of NQP and Rakudo compilation output. |
11:53 | |
jnthn | flight & | 11:56 | |
FROGGS | yay!! | 12:43 | |
jnthn++ | |||
12:44
cognominal joined
|
|||
tadzik | woot | 12:50 | |
timotimo | yay! | 12:56 | |
a nice little gift before becoming tuitless for ... what was it, 6 days? | |||
gist.github.com/timo/4b8885415dbef0928f02 \o/ | 12:59 | ||
diakopter | I don't understand why I can't pull or --rebase pull the latest commits (since timotimo's merge) without making merge conflicts | 13:00 | |
timotimo | it could very well be the case that you pulled one of my commits that i then hastily force-pushed over | 13:01 | |
diakopter | seems so | ||
how to fix | |||
some kind of force pull? | 13:02 | ||
timotimo | git fetch | ||
git reset --hard origin/master | |||
nwc10 | who becomes tuitless? :-( | 13:04 | |
diakopter | yeah had to do that | ||
timotimo | jnthn said so | ||
"22:52 < jnthn> btw, I probably won't have many tuits at all during the next 5-6 days." | |||
diakopter: sorry :( | |||
nwc10 | aha | 13:05 | |
timotimo | but i think you were the only unfortunate soul who got to download these completely botched commits | ||
diakopter | I think he will have some p6 tuits ;) | ||
timotimo | mayhaps | 13:07 | |
diakopter | I like how Git Extensions now automatically fetches in teh background every few minutes | 13:10 | |
there's a bunch of nom commits in rakudo (back to 9 days) that aren't in moar-support | 13:11 | ||
(wouldn't that skew the test comparisons?) | 13:12 | ||
timotimo | my change for chained abverbs would, but only 2 tests in total i believe | ||
not sure if the "not set an encoding" commit changes test pass rate | 13:13 | ||
diakopter | oh | ||
13:24
cognominal joined
|
|||
JimmyZ | jnthn: I still got 'MAST::Local index out of range' after updating nqp && moarvm | 13:47 | |
timotimo | :o | 13:48 | |
but how? :( | |||
diakopter | JimmyZ: you almost certainly need to make realclean or hard reset | ||
JimmyZ | I did do `git clean -xdf' and `git reset HEAD --hard` | 13:51 | |
diakopter | did you update submodules? | 14:18 | |
(maybe something there?) | |||
all three, rakudo, nqp, and moarvm trees | |||
(clean/hard I mean) | |||
14:35
FROGGS[mobile] joined
14:41
lue joined
15:54
lue joined
16:51
FROGGS[mobile] joined
18:04
FROGGS[mobile] joined
18:07
ssutch joined
|
|||
dalek | arVM: 3293aaf | (Tobias Leich)++ | / (6 files): add op uniisblock to look up unicode blocks directly |
19:04 | |
arVM: 4e2fe17 | (Tobias Leich)++ | / (3 files): add block lookup, and (evil) hack for property value lookups |
|||
FROGGS creates his own rakudo.moar_summary.out | 19:21 | ||
diakopter | :) | 19:23 | |
[Coke] | oh, we probably have a new one.. | 19:44 | |
as of a few hours ago: | 19:46 | ||
19:46
FROGGS[mobile] joined
|
|||
[Coke] | +"total", 27135, 1060, 641, 1384, 29729, 28495 | 19:46 | |
r: say 27135/28454 # moar today | |||
camelia | rakudo-parrot 263aa0, rakudo-jvm 263aa0: OUTPUT«0.953644» | ||
19:58
FROGGS[mobile] joined
|
|||
FROGGS | mine: | 20:07 | |
"Synopsis", "pass","fail","todo","skip","plan","spec" | |||
"total", 27460, 686, 639, 1383, 29652, 28495 | |||
r: say 27460/28454 | 20:08 | ||
camelia | rakudo-parrot 263aa0, rakudo-jvm 263aa0: OUTPUT«0.965066» | ||
grondilu | 96.5% ? nice | 20:12 | |
20:18
FROGGS[mobile] joined
|
|||
nwc10 | slow thing is, well, slow: 7977 nicholas 30 10 382m 359m 14m R 99.4 0.4 3600:00 moar | 21:34 | |
FROGGS | :o) | ||
timotimo | but still awesome :D | 23:17 |