lizmat | Files=1136, Tests=52746, 226 wallclock secs (13.36 usr 3.66 sys + 1404.91 cusr 130.91 csys = 1552.84 CPU) | 06:55 | |
[Tux] | Drumrollllllllllllll … | 07:36 | |
This is Rakudo version 2016.08.1-146-gbb6da76 built on MoarVM version 2016.08-43-g3d04391 | |||
csv-ip5xs 9.327 | |||
test 15.392 | |||
test-t 6.996 | |||
csv-parser 16.610 | |||
llfourn | \o/ <6 | 07:48 | |
lizmat | whee! | 07:52 | |
[Tux] | <7 | 07:55 | |
Woodi | [Tux]: is it possible to add tests checking "other" domains performance ? there is a window for 4 more tests on the right :) numbers are close in last 2-3 monts, maybe it is possibility to test "other" parts of language ? | 08:20 | |
[Tux] | I have no idea what you are hinting at | 08:21 | |
Woodi | I cant hade ready answer, but maybe cvs checking use just only part of language functionalities ? | 08:22 | |
*don't have... | |||
[Tux] | Do you think *I* would be the one to perform such tests? (/me does not) | 08:23 | |
Woodi | [Tux]: but you run eg. cvs-parser tests ? | ||
anyway, just idea, don't fill forced... | 08:24 | ||
lizmat | Woodi: the Text::CSV tests are an itch that [Tux] wants to scratch | ||
and we're thankful for the canary it provides | 08:25 | ||
if you think other parts of the language should be exercised similarly | |||
feel free to run a test for that and post the result here every day :-) | |||
breakfast& | 08:26 | ||
Woodi | [Tux]: you have code for tests somewhere opensoursed ? | 08:27 | |
[Tux] | yes: github.com/Tux/CSV - look at time.pl | ||
Woodi | [Tux]++ thanx. I will check, *maybe* I can find some more numbers :) | 08:28 | |
dalek | ast: 34a0ff2 | (Zoffix Znet)++ | S16-io/lines.t: Capture :err too So we don't print out errors we don't care about during test run. |
09:12 | |
lizmat | looking at optimization targets, looks like Str.match would be a good one | 09:16 | |
it e.g. takes 15% of CPU when doing open("file") | |||
Zoffix | roast's Test::Util's is_run doesn't seem to capture stuff like "*** Error in `/home/zoffix/CPANPRC/rakudo/install/bin/moar': double free or corruption (!prev): 0x00002b37bc04c660 ***" | 09:30 | |
weird. gist.github.com/zoffixznet/4896a86...2df2331526 | 09:32 | ||
Hm, it relies on my shell to redirect stuff like this correctly :/ | 09:36 | ||
*lesigh* | |||
Smaller testcase: gist.github.com/zoffixznet/cf1ca3e...141ffa5333 | 09:40 | ||
lizmat | commute& | 09:47 | |
dalek | ast: c84a389 | (Zoffix Znet)++ | integration/weird-errors.t: Remove trailing whitespace |
10:02 | |
ast: 02d6988 | (Zoffix Znet)++ | integration/weird-errors.t: accessing Seq from multiple threads does not segfault RT#127208 |
|||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127208 | ||
Zoffix | committable6, 2015.12 use Test; is-deeply [^10 .hyper(:batch).map: *+0], [^10] | 10:07 | |
committable6 | Zoffix, ¦«2015.12»: not ok 1 - # Failed test at /tmp/tWwvxEP9rn line 1# expected: $[0, 1, 2, 3, 4, 5, 6, 7, 8, 9]# got: $[0, 2, 1, 3, 4, 5, 6, 7, 8, 9] «exit code = 1» | ||
timotimo | oh, that's interesting | 10:11 | |
timotimo remembers putting something into the hyper code to make the order at the end correct | |||
i probably didn't do it right :) | 10:12 | ||
Zoffix points at 2015.12 tag | |||
timotimo | oh! | ||
committable6, HEAD use Test; is-deeply [^10 .hyper(:batch).map: *+0], [^10] | 10:13 | ||
Zoffix | nice going :) | ||
timotimo | :( | ||
MasterDuke | luckily i'm up annoyingly early! | 10:14 | |
timotimo | .tell lizmat if Str.match is involved in open("file"), maybe there's a place where simplematch could be used | ||
yoleaux2 | timotimo: I'll pass your message to lizmat. | ||
MasterDuke | .tell lizmat Str.match as optimization target++, i've looked at a bunch of --profile-compile profiles and match is usually near the top | 10:15 | |
yoleaux2 | MasterDuke: I'll pass your message to lizmat. | ||
dalek | ast: c14b344 | (Zoffix Znet)++ | S07-hyperrace/hyper.t: .hyper preserves order The test can be simplified, but currently .hyper returns an empty list on some runs, which would cause the test to fail. The order *is* preserved, when we do get something, however. RT#127099 |
10:17 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127099 | ||
Zoffix | MasterDuke, so it just crashes? Stick into a loop | 10:18 | |
timotimo | SourceBaby: &open | 10:19 | |
s: &open | |||
SourceBaby | timotimo, Sauce is at github.com/rakudo/rakudo/blob/bb6d...rs.pm#L104 | ||
MasterDuke | brute force testing | 10:20 | |
Zoffix: we've done that for some other stuff, but if it leaves behind some files when it crashes it can block the other bots (or the restarted version of itself) | 10:25 | ||
Zoffix | ah | ||
MasterDuke | we've some thoughts on how to fix it, but my initial idea would have required building three new copies of all the builds, so we're trying to think of something better | 10:26 | |
timotimo | .tell lizmat the time spent in match when doing open("foobar") is from the .e test calling canonpath, which is basically all match calls (it does all those substitutions); maybe there's a simpler method for substitutions like this. | 10:28 | |
yoleaux2 | timotimo: I'll pass your message to lizmat. | ||
timotimo | do we have a substitution function for regexes that's faster than using ~~ s:g/.../.../? | 10:29 | |
nine | timotimo: we need that .e and .d only for catching someone trying to open a directory. Maybe we could just be smarter by checking for this error condition only in the error path? I.e. first trying to open it and if that fails try to find out why exactly. | 10:33 | |
timotimo | maybe | 10:35 | |
i'm looking at how much faster canonpath can get by replacing ~~ s:g with .subst-mutate | |||
in that process it seems like there's a :g missing | 10:36 | ||
m: say $*SPEC.canonpath("../../foo/bar/../../baz/quux") | |||
camelia | rakudo-moar bb6da7: OUTPUT«../../foo/bar/../../baz/quux» | ||
timotimo | oh, it's meant to only match at the beginning | 10:37 | |
okay, that's fair | |||
dalek | ast: 059bb84 | (Zoffix Znet)++ | integration/weird-errors.t: Check for successful exit too Once in a blue moon, the race condition won't cause a throw and the program will succeed. Watch for that case too, so the test fails only when we get SEGV and ilk. |
||
MasterDuke | Zoffix: timotimo: since you guys are around, any thoughts on github.com/perl6/nqp/pull/307? | 10:43 | |
Zoffix | MasterDuke, I looked at it but it's beyond my paygrade, so I've no idea if it can be merged or not :) | 10:44 | |
MasterDuke | fair enough, can't ask for more | 10:45 | |
timotimo | MasterDuke: i'm surprised that code fixes that bug :) | 10:46 | |
MasterDuke | ha. i'll admit i wouldn't have thought to go there first, but this github.com/MoarVM/MoarVM/issues/391 pointed me to it | 10:47 | |
timotimo | oh! | 10:48 | |
it's trying to numify even though it's a non-numbery name? | |||
nine | timotimo: seems like we can't as nqp::open on a directory does not actually fail | 10:49 | |
timotimo | oh, that sucks | 10:50 | |
currently, trying to install rakudo (while building) explodes in generate-moar-runner | |||
i might have the answer... | 10:51 | ||
Failure.new(exception => X::AdHoc.new(payload => "Failed to open file /home/timo/perl6/rakudo/perl6-m/: illegal operation on a directory"), backtrace => Backtrace.new) | |||
guess what. calling .print("foobar") on that gives you "wrong number of arguments" | |||
m: my $path = "hello/"; $path = $path.chomp if $path.ends-with('/'); say $path | 10:52 | ||
camelia | rakudo-moar bb6da7: OUTPUT«hello/» | ||
timotimo | m: my $path = "hello/"; $path = $path.chop if $path.ends-with('/'); say $path | ||
camelia | rakudo-moar bb6da7: OUTPUT«hello» | ||
timotimo | that's what i did wrong | ||
MasterDuke: i think your fix is OK to go in. do you have some spec tests to go with it ready to go into roast? | 10:54 | ||
dalek | p: 71b434f | MasterDuke17++ | src/QRegex/P6Regex/Actions.nqp: Properly numify non-ascii tokens in regex/grammars Fix RT #127075 |
||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127075 | ||
dalek | p: 4fd4b48 | timo++ | src/QRegex/P6Regex/Actions.nqp: Merge pull request #307 from MasterDuke17/RT127075 Properly numify non-ascii tokens in regex/grammars good catch: the code was trying to see if the string is actually a number by numifying it, but that stumbled over things outside the ascii range when we actually had a word-like token name. |
||
MasterDuke | yeah, but rakudo will obviously also need its nqp version bumped | ||
timotimo | before: | 10:56 | |
time perl6 -e '$*SPEC.canonpath("foobar") for ^50_000' | |||
12.37user 0.01system 0:12.39elapsed 99%CPU (0avgtext+0avgdata 79748maxresident)k | |||
after: | |||
after:time perl6 -e '$*SPEC.canonpath("foobar") for ^50_000' | |||
6.15user 0.01system 0:06.18elapsed 99%CPU (0avgtext+0avgdata 79864maxresident)k | |||
nine | oh nice! timotimo++ | ||
timotimo | before: | ||
time perl6 -e '$*SPEC.canonpath($*SPEC.rel2abs("foobar")) for ^20_000' | 10:57 | ||
19.00user 0.01system 0:19.02elapsed 99%CPU (0avgtext+0avgdata 82316maxresident)k | |||
after: | |||
dalek | ast: feb0022 | (Zoffix Znet)++ | S09-typed-arrays/arrays.t: using `of` does not affect arrays defined later RT#126136 |
||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=126136 | ||
timotimo | time perl6 -e '$*SPEC.canonpath($*SPEC.rel2abs("foobar")) for ^20_000' | ||
12.68user 0.02system 0:12.71elapsed 99%CPU (0avgtext+0avgdata 81808maxresident)k | |||
MasterDuke | timotimo++ | ||
timotimo | i might be able to push it down a bit more, even | ||
m: say "foobar".contains('oo') | |||
camelia | rakudo-moar bb6da7: OUTPUT«True» | 10:58 | |
timotimo | OK, the first test case now takes only 0.43s instead of 12.39 or 6.18 | 11:01 | |
i call that a win. | |||
Zoffix | :o | ||
timotimo | however | ||
how often do we have paths that don't have any /. or // in them | |||
timotimo runs the spec tests | 11:02 | ||
someone else might be interested in porting those changes to the windows spec, too | |||
t/spec/S32-io/socket-accept-and-working-threads.t seems to be extremely busy for a very long time and won't terminate | 11:13 | ||
or does it just take a very long time? | |||
dalek | ast: 457d3d8 | (Zoffix Znet)++ | S19-command-line/repl.t: no bizzare types returned from redeclared "returns an `of` Array" sub The bug used to return types like Array[Int][Int][Int][Int] RT#125412 |
11:15 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125412 | ||
Zoffix | timotimo, no, it's relatively quick | ||
dalek | kudo/nom: f17d9b4 | timotimo++ | src/core/IO/Spec/Unix.pm: make Unix's canonpath much cheaper for simple paths use subst-mutate instead of going through ~~, extract dominant literals and .contains or .starts-with before running a full regex |
||
Zoffix | Takes 0m4.740s on my box | 11:16 | |
timotimo | all of my 4 cores had been working for 2½ minutes each until i killed the main process | ||
Zoffix | 0.o | ||
MasterDuke | i don't remember how to bump the NQP version required for rakudo, does one of you guys mind doing it? | 11:18 | |
Zoffix | cd nqp; git describe > ../tools/build/NQP_REVISION | 11:19 | |
(if you're in rakudo checkout; basically stick `git desribe`'s output into tools/build/NQP_REVISION) | |||
timotimo | you shouldn't be in rakudo checkout | 11:20 | |
opening an existing file 20k times still spends 8.43% of its time in match, 5% additionaly in match-subst, apparently | |||
Zoffix | Why not? | 11:22 | |
timotimo | i should run a before-check, too. | ||
well, only if you're going to do the 'cd nqp' first | |||
Zoffix | How do you get those measurements? Is that from --profile? | 11:26 | |
timotimo | which measurements exactly? | ||
Zoffix | <timotimo> opening an existing file 20k times still spends 8.43% of its time in match, 5% additionaly in match-subst, apparently | ||
like how do you see that it's 8% in match? | 11:27 | ||
timotimo | yes, that's the profiler. sorted by "exclusive time" | ||
Zoffix | cool | 11:28 | |
timotimo | before: time perl6 -e 'open("CORE.setting.moarvm").close for ^50_000' | ||
44.10user 0.44system 0:44.55elapsed 99%CPU (0avgtext+0avgdata 100624maxresident)k | |||
MasterDuke | that's how i do a lot of my measuring, with --profile --profile-filename=<something.json> | ||
timotimo | after: time perl6 -e 'open("CORE.setting.moarvm").close for ^50_000' | ||
18.44user 0.36system 0:18.82elapsed 99%CPU (0avgtext+0avgdata 105392maxresident)k | |||
so it was actually noticably worth it in total | |||
Zoffix | neat | 11:29 | |
MasterDuke | very nice, i've found IO stuff pretty slow so far, all improvements very welcome here | ||
timotimo | i might have more winnings | 11:31 | |
but only for strings that have some things in them | |||
m: say "a//b///c////d".trans('//' => '/') | 11:33 | ||
camelia | rakudo-moar f17d9b: OUTPUT«a//b///c////d» | ||
timotimo | m: say "a//b///c////d".trans('//' => '/', :g) | ||
camelia | rakudo-moar f17d9b: OUTPUT«a//b///c////d» | ||
timotimo | that doesn't even do a single one. huh? | ||
Zoffix | m: say "a//b///c////d".trans(['//'] => ['/'], :g) | ||
camelia | rakudo-moar f17d9b: OUTPUT«a/b//c//d» | ||
timotimo | OK. it doesn't do more multiples, though | ||
don't think :g is actually even a thing trans does | 11:34 | ||
Zoffix | m: say "a//b///c////d".trans(['//'] => ['/']) | ||
camelia | rakudo-moar f17d9b: OUTPUT«a/b//c//d» | ||
timotimo | i could run that in a loop as long as any // exists in the string | 11:36 | |
might still be cheaper than going through subst-mutate with a regex | |||
dalek | kudo/nom: 4578e7d | timotimo++ | src/core/IO/Spec/Unix.pm: make removing ./ at the beginning of a path cheaper |
11:37 | |
timotimo | i didn't F that up, right? | 11:38 | |
oh, instead of trans i can use subst with strings for good measure | 11:39 | ||
dalek | ast: 2c617ab | (Zoffix Znet)++ | S03-operators/spaceship.t: Remove trailing whitespace |
11:45 | |
ast: bb4d3eb | (Zoffix Znet)++ | S03-operators/spaceship.t: <=> with non-numerics throws correct exception RT#126536: rt.perl.org/Ticket/Display.html?id=126536 |
11:47 | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=126536 | ||
dalek | kudo/nom: 6e2a559 | timotimo++ | src/core/IO/Spec/Unix.pm: remove '//' and others without using a regex. ought to be noticably faster because of the general overhead of invoking a regex match |
11:55 | |
Zoffix | s: "meow", 'contains', \('x') | 11:56 | |
SourceBaby | Zoffix, Sauce is at github.com/rakudo/rakudo/blob/f17d...tr.pm#L118 | ||
dalek | ast: 50b6080 | (Zoffix Znet)++ | integration/error-reporting.t: `BEGIN` suggested when `begin` is used RT#126264: rt.perl.org/Ticket/Display.html?id=126264 |
12:02 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=126264 | ||
MasterDuke | one of you guys mind merging github.com/rakudo/rakudo/pull/874? once that's in i'll add my tests for 127075 to roast | 12:07 | |
dalek | kudo/nom: 60b56ac | MasterDuke17++ | tools/build/NQP_REVISION: Bump NQP Picks up fix to properly numify non-ascii tokens in regex/grammars. |
12:13 | |
kudo/nom: 20ed9e2 | (Zoffix Znet)++ | tools/build/NQP_REVISION: Merge pull request #874 from MasterDuke17/RT127075 Bump NQP: properly numify non-ascii tokens in regex/grammars. |
|||
MasterDuke | Zoffix++, thanks | 12:16 | |
dalek | ast: cfbe28c | MasterDuke17++ | S05-match/capturing-contexts.t: Tests for RT #127075 Test that non-ascii tokens work in regexes/grammars. |
12:23 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127075 | ||
dalek | ast: e82a3df | MasterDuke17++ | S05-match/capturing-contexts.t: Merge pull request #156 from MasterDuke17/RT127075 Tests for RT #127075 |
||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127075 | ||
timotimo | would somebody be so kind as to make the for-windows Spec also have a faster canonpath? | 12:52 | |
i'll be AFK | |||
MasterDuke | to anyone with RT editing privileges, RT #127075 can be resolved, it's fixed and has tests | 12:59 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127075 | ||
MasterDuke | is there any reason not to add return types to existing subs/methods in rakudo? E.g., several of the methods in IO::Path have them, but not all, would it be good to fill out the rest? | 13:41 | |
[Tux] | This is Rakudo version 2016.08.1-151-g20ed9e2 built on MoarVM version 2016.08-43-g3d04391 | 14:18 | |
csv-ip5xs 8.958 | |||
test 15.319 | |||
test-t 6.912 | |||
csv-parser 17.111 | |||
timotimo | MasterDuke: a return type of --> Nil is very good for performance. other than that, return type annotations give us a tiny bit of a performance loss | 14:49 | |
MasterDuke | loss! why in the world is that? | 15:15 | |
timotimo | because it has to be checked on every return | 15:22 | |
and we're not 100% good at eliminating the check in every case we could | |||
japhb | [Tux]: Is that the first time test-t has been below 7 seconds? | 16:25 | |
[Tux] | japhb, yes | 16:32 | |
$ grep ' [56][.]' *log | |||
2016-09-10 09:35:03 test-t 6.996 | |||
2016-09-10 16:06:30 test-t 6.912 | |||
timotimo | hey, that's good | 16:43 | |
now we're on the way towards below 6 | |||
Zoffix | ugexe, yeah, it works well enough I was planning on writing an article (RE Mojolicious with Inline p5) | 17:58 | |
committable6, 2015.07 m: grammar Bug { token term { a }; token TOP { <term> % \n } } Bug.parse( 'a' ); | 18:04 | ||
committable6 | Zoffix, ¦«2015.07»: ===SORRY!=== Error while compiling /tmp/CdvXej13O_Missing quantifier on the left argument of %at /tmp/CdvXej13O_:1------> { token term { a }; token TOP { <term> %⏏ \n } } Bug.parse( 'a' ); «exit code = 1» | ||
Zoffix | Is this the oldest version the bot knows about? | ||
seems to be | 18:05 | ||
cygx | o/ | 18:14 | |
dalek | ast: d3d36db | (Zoffix Znet)++ | S05-grammar/example.t: Remove trailing whitespace |
18:15 | |
cygx | timotimo: re nqp::open, assuming we want aomicity, afaik the correct way to ensure we're dealing with an actual file is to use fstat after the file has been opened | ||
dalek | ast: d633c04 | (Zoffix Znet)++ | S05-grammar/example.t: `quantifier with %` error includes the token it appears in RT#124219: rt.perl.org/Ticket/Display.html?id=124219 |
||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=124219 | ||
cygx | so we need to expose an nqp::fstat, or fold the check into nqp::open | ||
note that nqp::open should already fail with directories in any mode other than read-only | 18:17 | ||
liztormato | cygx: fwiw, I think it was decided that an open on a directory should *always* fail, regardless of mode | 18:24 | |
cygx | liztormato: sure, but should that happen at nqp or p6 level? | 18:25 | |
the issue is that POSIX is happy to open directories | |||
liztormato | Yeah. Bur that doesn't bring us much on Win, or does it nowadays? | 18:26 | |
*But | |||
dalek | ast: 0b80402 | (Zoffix Znet)++ | S05-grammar/example.t: lookbehind that should not match does not match RT#81136: rt.perl.org/Ticket/Display.html?id=81136 |
18:27 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...l?id=81136 | ||
cygx | it doesn't really bring us much on *nix, as we do not expose fdopendir() | 18:28 | |
so I think making nqp::open fail would be the right choice | |||
liztormato | Feels right to me. If you really want that level of control, there's always NativeCall | 18:29 | |
Dinner& | 18:31 | ||
cygx | hm.. on win32, nqp::open() on a directory does indeed fail with mode "w", but not necessarily with other modes that also include write-access | ||
Zoffix | We have uniprop-[int|str|bool] but those aren't in the spectest... Should they be? github.com/rakudo/rakudo/blob/20ed...ol.pm#L107 | 19:12 | |
m: say 'x'.uniprop-int | 19:15 | ||
camelia | rakudo-moar 20ed9e: OUTPUT«Cannot resolve caller uniprop-int(Str); none of these signatures match: (Str:D $str, Stringy:D $propname) (Int:D $code, Stringy:D $propname) in block <unit> at <tmp> line 1» | ||
Zoffix | m: say '4'.uniprop-int('N') | ||
camelia | rakudo-moar 20ed9e: OUTPUT«1» | ||
Zoffix | m: say '4'.uniprop-str('N') | ||
camelia | rakudo-moar 20ed9e: OUTPUT«» | ||
Zoffix | m: say '4'.uniprop-bool('N') | ||
camelia | rakudo-moar 20ed9e: OUTPUT«True» | ||
Zoffix | :/ | ||
m: say '4'.uniprop-bool('NM') | 19:16 | ||
camelia | rakudo-moar 20ed9e: OUTPUT«False» | ||
Zoffix | m: say '4'.uniprop-int('NM') | ||
camelia | rakudo-moar 20ed9e: OUTPUT«0» | ||
AlexDaniel | Zoffix: “Is this the oldest version the bot knows about?” – yeah. Wait, you want us to build more fossils? :) | 19:17 | |
it is possible | 19:18 | ||
Zoffix | Nah, it's fine. | ||
It's useful when writing tests for old tickets where you can ensure your test fails before the bug was fixed, but it's not that important. | |||
AlexDaniel | Still, that's not a bad idea: github.com/perl6/whateverable/issues/30 | 19:21 | |
dalek | kudo/nom: b771bcc | (Zoffix Znet)++ | src/core/Cool.pm: Make Cool.split work same as Str.split It seems when extra adverbs were added to Str.split, they were forgotten in Cool. The .split(@needles...) form is also broken on Cool. We can get rid of multies in Cool and simply coerce the Cool to Stringy and let the Str.split figure out which multi to call. Fixes RT#129242: rt.perl.org/Ticket/Display.html?id=129242 |
19:33 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=129242 | ||
ast: 060cf7a | (Zoffix Znet)++ | S32-str/split.t: .split works on Cool same as it works on Str RT#129242: rt.perl.org/Ticket/Display.html?id=129242 |
19:34 | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=129242 | ||
nine | timotimo: I wonder why we even do a canonpath for methods like .e. Why not let the file system deal with resolving the path? It ought to be much better and faster at that kind of work. | 19:39 | |
timotimo | *shrug*, try it out and see :) | 19:40 | |
sounds like it would work | 19:41 | ||
dalek | ast: 3f98103 | (Zoffix Znet)++ | S05-match/capturing-contexts.t: Fix broken fudge It appears that adding a fudge before a subtest fudges more tests than what was specified in the fudge, so tests after the subtests were being fudged as well. Moving the fudge inside the subtest fixes the issue. |
||
AlexDaniel | .tell [Coke] please give MasterDuke (ddgreen@gmail.com) privileges to close RT tickets | ||
yoleaux2 | AlexDaniel: I'll pass your message to [Coke]. | ||
nine | timotimo: loading GTK::Simple is now down to 0.26 from 0.30 thanks to your improvements :) | 19:42 | |
timotimo | me?! | 19:44 | |
woohoo!! | |||
Zoffix | buggable, tags | 20:06 | |
buggable | Zoffix, Total: 1367; BUG: 529; UNTAGGED: 288; LTA: 116; NYI: 91; JVM: 63; RFC: 57; CONC: 51; SEGV: 37; REGEX: 34; UNI: 29; PERF: 27; @LARRY: 20; IO: 17; POD: 17; NATIVECALL: 15; TESTNEEDED: 14; BUILD: 11; PRECOMP: 11; TODO: 11; TESTCOMMITTED: 9; MATH: 7; OO: 7; STAR: 6; BOOTSTRAP: 5; GLR: 4; OSX: 4; REPL: 4; WEIRD: 3; CONFIGURE: 1; DOCS: 1; LIBRARY: 1; RT: 1; SITE: 1; SPEC: 1; See perl6.fail/ for details | ||
Zoffix | m: say "Delta in one day: total {1380-1367} / TESTNEEDED: {30-14}" | 20:07 | |
camelia | rakudo-moar b771bc: OUTPUT«Delta in one day: total 13 / TESTNEEDED: 16» | ||
Zoffix | :/ | ||
How can total closed testsneeded be more than total closed :/ | |||
Oh, right, I untagged a couple that weren't yet fixed and it wasn't clear what the fixed form would be | 20:08 | ||
ugexe | Zoffix: do you have any of that code on github by chance? | 20:10 | |
Zoffix | ugexe, github.com/zoffixznet/Rele6sr/blob...bin/app.p6 | 20:11 | |
ugexe, oh, and morbo/hypnotoad don't work IIRC. So I started the app with perl6 bin/app daemon and for dev was using this script instead of morbo for restart-on-change: github.com/zoffixznet/Rele6sr/blob...art-app.p6 | 20:13 | ||
mst | Zoffix: morbo/hypnotoad likely won't work right via Inline::Perl5, but may work fine if you load the app code via Inline::Perl6 | 20:26 | |
whether that helps you at all, I've no idea | 20:27 | ||
also, if neither works, I can try and debug it | |||
timotimo | nine: is that a GTK::Simple with "use" or with "require"? | 20:36 | |
apparently D calls lexically defined classes "voldemort types" | 20:43 | ||
nine | timotimo: the need variant | 20:47 | |
timotimo | that's the faster one, right? | 20:49 | |
masak | nwc10++ # alternative universe Frankfurt t-shirt | 21:17 | |
timotimo | bail-adore ... %) | 21:24 | |
nine | timotimo: yes. | 21:41 | |
timotimo | i wonder if i should just go ahead, make that change, close the issue | 21:45 | |
dalek | ast: 0cfcd4f | (Zoffix Znet)++ | S12-enums/basic.t: dynamically created lists can be used to define an enum RT#124251: rt.perl.org/Ticket/Display.html?id=124251 |
21:57 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=124251 | ||
dalek | ast: f71b8a6 | (Zoffix Znet)++ | S10-packages/use-with-class.t: Fix typo |
22:25 | |
nine | timotimo: I'm still wondering why the initial compilation of the need version is so much slower than the require version. Haven't even come up with any hypothesis at all. | 22:39 | |
timotimo | nanites. | 22:41 | |
aliens | 22:46 | ||
lizmat waves from home | 22:48 | ||
yoleaux2 | 10:14Z <timotimo> lizmat: if Str.match is involved in open("file"), maybe there's a place where simplematch could be used | ||
10:15Z <MasterDuke> lizmat: Str.match as optimization target++, i've looked at a bunch of --profile-compile profiles and match is usually near the top | |||
10:28Z <timotimo> lizmat: the time spent in match when doing open("foobar") is from the .e test calling canonpath, which is basically all match calls (it does all those substitutions); maybe there's a simpler method for substitutions like this. | |||
dalek | ast: 0c7d68f | (Zoffix Znet)++ | / (3 files): no Perl6/World.nqp in warning When one module is used in another, with $!foo is rw trait. RT#126302: rt.perl.org/Ticket/Display.html?id=126302 |
||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=126302 | ||
timotimo | lizmat: did you see my progress on canonpath? :D | 22:51 | |
lizmat | yes, it was along the lines I was thinking of | 23:00 | |
timotimo | i'm glad :) | 23:01 | |
lizmat | the last lines of canonpath I would do: | 23:02 | |
timotimo | canonpath on a path that has no . or / or whatever in it is already almost free | ||
lizmat | $path.chars > 1 && $path.ends-with('/') ?? $path.chop !! $path | ||
timotimo | but there's no equivalent change for the windows spec yet, and i imagine it's difficult because both \ and / are allowed? or perhaps it'd be tr'd immediately | ||
that sounds good, yes. do you want to commit that? | |||
lizmat | timotimo: sure, will do that | ||
but not now :-) | 23:03 | ||
fwiw, that was one of the premises of newio: that you *can* use / on Win | |||
it's just that on entry, you need to tr/\\/\// :-) | |||
so that this whole business can be integrated and $*SPEC can be put to sleep | 23:04 | ||
timotimo | ah | ||
lizmat | anyways.. pretty tired now | 23:05 | |
driving for about 6 hours | |||
I should probably sleep before I commit anything :-) | 23:06 | ||
timotimo | aye, good night! :) | ||
lizmat | nighty night! | ||
dalek | ast: e1704aa | (Zoffix Znet)++ | S06-signature/definite-return.t: using two `returns` produces sane error RT#125181: rt.perl.org/Ticket/Display.html?id=125181 |
23:31 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125181 | ||
Zoffix | committable6, 2016.02 say 2**10000000000 | 23:50 | |
committable6 | Zoffix, ¦«2016.02»: Numeric overflow in block <unit> at /tmp/oJNs2peL54 line 1Actually thrown at: in block <unit> at /tmp/oJNs2peL54 line 1 «exit code = 1» | ||
Zoffix | committable6, 2016.02 dd [ $*PERL.version ] | ||
committable6 | Zoffix, ¦«2016.02»: [Version.new('6.c')] | ||
Zoffix | committable6, 2016.02 dd [ $*PERL.compiler.version ] | ||
committable6 | Zoffix, ¦«2016.02»: [Version.new('2016.2')] | ||
Zoffix | I wonder how [Coke] got a 1 on a 2016.02 :/ rt.perl.org/Ticket/Display.html?id...xn-1392590 | ||
32bit box? | 23:51 | ||
hm | |||
:( | 23:52 | ||
dalek | ast: be4bb4c | (Zoffix Znet)++ | S03-operators/overflow.t: Remove trailing whitespace |
23:58 | |
ast: c7a5606 | (Zoffix Znet)++ | S03-operators/overflow.t: attempting to raise to a huge power throws RT#125938: rt.perl.org/Ticket/Display.html?id=125938 |
|||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125938 | ||
Zoffix | MasterDuke, AlexDaniel are you around? I killed committable6 again | 23:59 |