AlexDaniel | samcv: any news on RT #131961? Or if not, was there anything interesting you were able to find? I'm thinking maybe we can leave something in a comment on that ticket to help the thing go | 01:14 | |
synopsebot6 | Link: rt.perl.org/rt3/Public/Bug/Display...?id=131961 | ||
samcv | well i am not sure what's causing it. though it now doesn't leak memory when it throws, and now frees the bytes in the buffer | 01:15 | |
but otherwise no | |||
AlexDaniel | wait, I didn't bisect this ticket? | 01:26 | |
.oO( why???? ) |
01:27 | ||
makes no sense. I'll get some tea and will bisect it. Who knows, maybe it'll give a clue | 01:28 | ||
bisected to github.com/rakudo/rakudo/commit/51...29cde39c1f | 01:55 | ||
but I don't know if it means anything | 01:57 | ||
but I was seeing something like this a few weeks before that commit: github.com/perl6/whateverable/issues/153 | 01:58 | ||
it may be a different issue | |||
how can I test it? I guess simply try it with a bigger file? | |||
samcv: yes, that's the right commit. Before that it can go through 251MB file just fine | 02:06 | ||
… “commit”. One of 15 MoarVM commits I mean | 02:07 | ||
.tell [Coke] A *tiny* update on the Malformed UTF-8 issue. There's hope! rt.perl.org/Ticket/Display.html?id...xn-1486102 | 02:10 | ||
yoleaux | AlexDaniel: I'll pass your message to [Coke]. | ||
samcv | AlexDaniel, well decodestream was changed | 02:17 | |
it's possible that or the utf8 changes caused it | |||
i can't compile nqp right now though.. | 02:18 | ||
anyone else have this? | |||
geekosaur | nqp built for me a couple hours ago | 02:21 | |
ugexe | I wrote a quick example of how to get appveyor build status changes in an irc channel by slapping together Cro and IRC::Client - gist.github.com/ugexe/a7724886674f...93f0b8a8cf | 02:34 | |
Zoffix | wooo \o/ ugexe++ | 02:35 | |
AlexDaniel | alright, it's github.com/MoarVM/MoarVM/commit/a6...b3a2b05762 | 04:05 | |
why can't I teach bisectable to do it? Why? :-/ | 04:06 | ||
samcv | AlexDaniel++ should let jnthn know maybe he can figure it out better than I, since he wrote it | 04:08 | |
AlexDaniel | .tell jnthn FWIW rt.perl.org/Ticket/Display.html?id...xn-1486128 | ||
yoleaux | AlexDaniel: I'll pass your message to jnthn. | ||
AlexDaniel | .tell timotimo ping me when you have 15 minutes for me | 04:30 | |
yoleaux | AlexDaniel: I'll pass your message to timotimo. | ||
samcv | that nqp commit that disabled repl on tty i think broke roast tests | 04:42 | |
that wanted the output of the REPL | |||
maybe we need an ENV var to enable it or a flag | |||
like one to force repl and one to force no-repl | 04:43 | ||
.u { .uniprop('Block') eq 'Malayalam' } | 05:12 | ||
yoleaux | U+0020 SPACE [Zs] ( ) | ||
U+0027 APOSTROPHE [Po] (') | |||
U+0028 LEFT PARENTHESIS [Ps] (() | |||
samcv | u: { .uniprop('Block') eq 'Malayalam' } | 05:14 | |
unicodable6 | samcv, U+0D01 MALAYALAM SIGN CANDRABINDU [Mn] ( ?) | ||
samcv, U+0D02 MALAYALAM SIGN ANUSVARA [Mc] (?) | |||
samcv, 114 characters in total: gist.github.com/fa65e9e3f6629185e9...53f219912e | |||
Geth | roast: 640d40ee81 | (Samantha McVey)++ | S05-mass/properties-block.t Fix InMalayam test Previously this test was a CJK base with a Malayalam extending character. Replace it so the base character is also Malayam. |
05:41 | |
[Tux] | This is Rakudo version 2017.08-25-g7d1ece805 built on MoarVM version 2017.08.1-32-gcd41322e | 06:05 | |
csv-ip5xs 1.324 - 1.352 | |||
test 10.438 | |||
test-t 3.513 - 3.640 | |||
csv-parser 11.803 | |||
Geth | nqp: fc57c9a203 | (Samantha McVey)++ | tools/build/MOAR_REVISION Bump MoarVM for Unicode 10 update * Adds Hangul codepoint names * Property values are now unique for each property |
06:39 | |
¦ nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...-g79654248 | |||
rakudo/nom: 64dd94c250 | (Samantha McVey)++ | tools/build/NQP_REVISION Bump MoarVM/nqp for Unicode 10 update * Adds Hangul codepoint names * Property values are now unique for each property |
06:43 | ||
¦ rakudo/nom: version bump brought these changes: github.com/perl6/nqp/compare/2017....gfc57c9a20 | |||
lizmat | Files=1223, Tests=67663, 293 wallclock secs (13.91 usr 5.21 sys + 1972.40 cusr 206.74 csys = 2198.26 CPU) | 08:00 | |
Geth | nqp: 589a3dd5cd | (Samantha McVey)++ | docs/ops.markdown Add docs for a few variations of index/eqat |
08:22 | |
nqp/master: 4 commits pushed by pmurias++ | 08:39 | ||
lizmat | afk for a while& | 09:22 | |
timotimo | AlexDaniel: i think i might have the time for you now. what's up? | 09:33 | |
yoleaux | 04:30Z <AlexDaniel> timotimo: ping me when you have 15 minutes for me | ||
timotimo | correction: i have to head out and do some errands | 09:41 | |
Geth | rakudo/jit_nativecall: 3a48c66de1 | (Stefan Seifert)++ | lib/NativeCall.pm6 Unbox HLL objects passed to JITed native functions |
10:14 | |
dogbert17 | timotimo: irclog.perlgeek.de/moarvm/2017-08-30#i_15092872 | 10:16 | |
timotimo | dogbert17: i believe we're on a non-release commit of libtommath that already has that | 10:17 | |
dogbert17 | yes but the fix for RT #129829 would be nice to have as well | 10:24 | |
synopsebot6 | Link: rt.perl.org/rt3/Public/Bug/Display...?id=129829 | ||
Skarsnik | Hello | 11:38 | |
timotimo | AlexDaniel: i have time for you, are you there? | 11:47 | |
AlexDaniel | timotimo: now I am :) | 12:43 | |
timotimo | OK! | ||
Skarsnik | I just watched Damian Conway talk about three little words. I love the your keyword x) | 12:46 | |
Zoffix | ??? New blog post: "On Troll Hugging, Hole Digging, and Improving Open Source Communities": rakudo.party/post/On-Troll-Hugging...ommunities | 12:56 | |
pmurias | Zoffix: hole digging seems like a more "permanent" way to get rid of trolls ;) | 12:57 | |
Zoffix | :) | 12:58 | |
AlexDaniel | awesome blog post | 13:21 | |
MasterDuke | Zoffix++ nice post. fyi, "barelly" | 13:23 | |
Zoffix | Thanks. Fixed. | 13:26 | |
AlexDaniel | huggable: ? :is: rakudo.party/post/On-Troll-Hugging...ommunities | 13:27 | |
huggable | AlexDaniel, Added ? as rakudo.party/post/On-Troll-Hugging...ommunities | ||
AlexDaniel | huggable: <3 :is: rakudo.party/post/On-Troll-Hugging...ommunities | ||
huggable | AlexDaniel, Added <3 as rakudo.party/post/On-Troll-Hugging...ommunities | ||
nine | OMG there's a quote of me | 13:55 | |
nine blushes | |||
AlexDaniel | by the way, maybe somebody has some interesting ideas on this: github.com/perl6/whateverable/issu...-326300469 | 13:58 | |
mainly the question is: if nqp bump in rakudo brought more than one nqp commit, what's the right way to bisect it? | |||
nine | Zoffix++ # really awesome blog post | 14:02 | |
lizmat | someone should put Zoffix' blog post on HN | ||
brrt | someone with a flame-retardant suit | 14:10 | |
or a 'thick skin' or what have you | |||
timotimo | i just realized i never told anyone my lightning talk would be okay to record & upload | 14:15 | |
so it probably won't appear on youtube | |||
which is a good thing because i could re-make it as a much more structured (and rehearsed) screencast or something | |||
brrt | (also, Zoffix++, i enjoyed that post) | 14:18 | |
timotimo | yes, yes. good post, Zoffix++ | 14:21 | |
MasterDuke | are the SPW videos going up under a particular account? | ||
perlpilot | Zoffix++ I haven't finished reading it yet, but it resonates well with me. Good job! | 14:22 | |
timotimo | MasterDuke: lee manages that account | ||
MasterDuke | timotimo: k. hopefully some links get posted, i'd like to watch them | 14:30 | |
timotimo | www.youtube.com/channel/UCV3oXF6JS...IDa_7KMuEA | 14:33 | |
Geth | rakudo: MasterDuke17++ created pull request #1145: Simplify ignore(mark|case) handling in INTERPOLATE |
||
timotimo | not sure if the comment about lee managing it is correct | ||
lee was the one with the camera in any case | |||
MasterDuke | timotimo++ thanks | ||
AlexDaniel | ? Community Bug SQUASHathon blogs.perl.org/users/zoffix_znet/20...athon.html | 14:42 | |
Geth | rakudo/nom: 79604a8803 | (Elizabeth Mattijsen)++ | src/core/Junction.pm Refine Junction.INFIX-TWO one() infix one() shouldn't merge, because any merge would change the semantics. This *is* ok for any(), all() and none(). |
14:55 | |
rakudo/nom: 8b5d283cb6 | (Elizabeth Mattijsen)++ | src/core/Junction.pm Complete support for Junction ~ Junction Even for two differently typed junctions or if one of the junctions is a one(). |
15:05 | ||
rakudo/nom: 215a5fa731 | MasterDuke17++ | src/core/Match.pm Simplify ignore(mark|case) handling in INTERPOLATE By switching to use the relatively new nqp::index(ic|im|icim) ops. |
15:08 | ||
rakudo/nom: 067f5407ef | lizmat++ (committed using GitHub Web editor) | src/core/Match.pm Merge pull request #1145 from MasterDuke17/simplify_ignoremark_and_ignorecase_in_INTERPOLATE Simplify ignore(mark|case) handling in INTERPOLATE |
|||
timotimo | .o( squashable ) | 15:14 | |
Geth | rakudo: MasterDuke17++ created pull request #1146: Document the $m argument in INTERPOLATE's comment |
15:48 | |
|Tux| | .hug Zoffix | 15:58 | |
Zoffix | |Tux|: oops. I guess IRC::Client still has a bug where the IRC grammar doesn't match nicks with `|` chars in them | 16:14 | |
|Tux| | (hug still applies) | ||
Zoffix | Thanks :) | ||
Geth | rakudo/nom: ea57cbec8a | MasterDuke17++ (committed using GitHub Web editor) | src/core/Match.pm Document the $m argument in INTERPOLATE's comment |
16:19 | |
rakudo/nom: e717d14db1 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/core/Match.pm Merge pull request #1146 from MasterDuke17/patch-1 Document the $m argument in INTERPOLATE's comment |
|||
rakudo: MasterDuke17++ created pull request #1147: s/index/eqat/g in INTERPOLATE literal handling |
19:50 | ||
rakudo/nom: c6aacafd1f | MasterDuke17++ | src/core/Match.pm s/index/eqat/g in INTERPOLATE literal handling The simplification in 215a5fa731 used nqp::index* for the ignore* cases, but if that change is made to the regular case some spectests break. Both 215a5fa731 and this change pass all spectests, which I think indicates that there probably aren't enough (any?) tests for regex interpolation with ingnore* modifiers. Using index* is more efficient, since it doesn't require walking through the haystack and checking at each position, but it somehow needs to be done more correctly. |
20:07 | ||
rakudo/nom: d135728b71 | lizmat++ (committed using GitHub Web editor) | src/core/Match.pm Merge pull request #1147 from MasterDuke17/use_eqat_instead_of_index_in_INTERPOLATE s/index/eqat/g in INTERPOLATE literal handling |
|||
MasterDuke | lizmat: thanks. btw, istr you were working on MATCH a little while ago. do you have any thoughts on my recent comments/questions re INTERPOLATE? | 20:18 | |
lizmat | none, apart from being glad you're working on it :-) | ||
MasterDuke: and that we could use some more tests :-) | 20:19 | ||
MasterDuke | heh, ain't that always the case... | 20:20 | |
lizmat | indeed :-) | ||
MasterDuke | well, i'm certainly open to help/advice/correction/etc from anybody, i'd be lying if i said i completely understand how INTERPOLATE is working (or should) | 20:23 | |
japhb | huggable: speed | 21:27 | |
huggable | japhb, tux.nl/Talks/CSV6/speed4.html | ||
japhb | buggable: speed | ||
buggable | japhb, ?????????????????????????????????????????????????? data for 2017-07-30–2017-08-31; range: 3.471s–4.925s; 21% faster | ||
AlexDaniel | this graph needs a logarithmic scale | 21:29 | |
maybe | 21:30 | ||
japhb | AlexDaniel: the problem with logarithmic scale is what your zero point is, because it can't actually be zero. | 21:45 | |
Plus that last pixel or two makes a HUGE difference to the experienced speed out of proportion to the visual difference. | |||
The current code is designed to look kinda like it feels: The above is "Slow for a while, then a bit faster, then slow again (with one measurement REALLY slow), then getting back to faster again. | 21:46 | ||
" | 21:47 | ||
Sadly unicode doesn't offer 16- or 32-unit-high bars | 21:48 | ||
For increased precision | |||
Though now that I think of it, I suppose buggable could rapidly answer *twice* and stack the two rows into a double-high chart for more precision. | 21:49 | ||
Oooh, I think I'm going to use this locally. | |||
Where is the buggable code? | 21:50 | ||
buggable: source | |||
buggable | japhb, See: github.com/zoffixznet/perl6-buggable | ||
jdv79 | is the optimization techniques deployed in moar similar to a tracing jit? | 21:53 | |
i just remember the term and not any specifics | |||
is the irc log site going to be fixed for unicode anytime soon? | 21:57 | ||
I'd look into it if its something i could fix. its not great to keep seeking ? chars all the time when backlogging. | 21:58 | ||
MasterDuke | jdv79: moritz and Zoffix have chatted about it a couple times, i think moritz said it's a mysql thing. could try searching the logs (either here or #perl6, don't remember which) for details | 22:02 | |
hm, maybe it was moritz and AlexDaniel | |||
jdv79 | didn't it used to work? | 22:03 | |
i feel like it did cause i think i saw some unicode stuff in those logs before i updated my fonts so my term didn't bork them | 22:04 | ||
moritz: what's the status of that? is there a ticket or anything else? | |||
AlexDaniel | jdv79: I looked at it a little bit but didn't make much progress | 22:08 | |
MasterDuke | jdv79: moritz said he was going to sleep a little while ago, so you'll probably have to wait until tomorrow to chat with him | ||
AlexDaniel | jdv79: here's the last discussion: irclog.perlgeek.de/perl6/2017-08-22#i_15056532 | 22:12 | |
MasterDuke | another question about INTERPOLATE: i'm trying to correctly use index instead of eqat. just doing s/eqat/index/ fails this test | 22:20 | |
m: use Test; is "foobar" ~~ /$( $_.substr(3) )/, "bar", "Contextualizer with functions calls"; | |||
camelia | ok 1 - Contextualizer with functions calls? | ||
MasterDuke | instead i get: # Failed test 'Contextualizer with functions calls' # at -e line 1 # expected: 'bar' # got: 'foobar' | 22:21 | |
fyi, this is my patch: gist.github.com/MasterDuke17/c9512...64ad04103f | |||
afk for a bit, but i'll backlog for suggestions | 22:22 | ||
japhb | If I want to return something multi-line from a buggable plugin, do I need to return a list or something \n separated? | 22:47 | |
Zoffix: ^^ | |||
Also, will each line be prefixed with "$nick, " ? | 22:48 | ||
FWIW, the code I have now can handle anything from 1-4 rows (the code can handle any number, but I figured > 4 was rude to the channel). It looks like this: | 22:50 | ||
??? ? ?? ????????????? dates: 2017-07-30–2017-08-31 | |||
????????????? ? ???????????????? range: 3.471s–4.925s | |||
????????????? ?? ???????????????? ??? ? speed: 21% faster | |||
?????????????????????????????????????????????????? | |||
buggable: help | 22:54 | ||
buggable | japhb, tags | tag SOMETAG | eco | eco Some search term | author github username | speed | testers CPANTesters report ID |