Geth | rakudo: d21c31e1d7 | (Elizabeth Mattijsen)++ | lib/Telemetry.pm6 Add Telemetry msgsnd,msgrcv,nsignals,nvcsw,invcsw - Also add invcsw (involuntary context switches) to default report - this shows some data, at least on MacOS - This concludes all the fields in nqp::getrusage (*phew*) |
00:04 | |||||||||||||||||||||||||||||||||
lizmat | and that concludes my hacking for today | 00:06 | |||||||||||||||||||||||||||||||||
.tell gfldex hope this is enough to write a nice blogpost about Telemetry :-) | |||||||||||||||||||||||||||||||||||
yoleaux | lizmat: I'll pass your message to gfldex. | ||||||||||||||||||||||||||||||||||
lizmat | good night, #perl6-dev! | ||||||||||||||||||||||||||||||||||
Zoffix | \o | 00:19 | |||||||||||||||||||||||||||||||||
Zoffix commits to remaking the entire Introducing Perl 6 brochure | 00:49 | ||||||||||||||||||||||||||||||||||
Took me half a year to realize I don't have to use $work's tiny laptop to work remotely and instead can remote-desktop into it from my desktop with giant monitor. So I now have proper hardware for design software when I'm hope :P | 00:50 | ||||||||||||||||||||||||||||||||||
*home | |||||||||||||||||||||||||||||||||||
timotimo | oh, nice | ||||||||||||||||||||||||||||||||||
but what's wrong about inkscape!!! (lol omg) | 00:51 | ||||||||||||||||||||||||||||||||||
Zoffix | No idea. Never used it. Out of free tools, I've used Scribus 5 years ago and it kept crashing. | 00:52 | |||||||||||||||||||||||||||||||||
Or making 1GB PDFs :) | 00:53 | ||||||||||||||||||||||||||||||||||
timotimo | fair enough | ||||||||||||||||||||||||||||||||||
inkscape's UI has been in desperate need of a new UI for a long long time | |||||||||||||||||||||||||||||||||||
there are people capable of doing spectacular things with it, of course | |||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Elizabeth Mattijsen '-Inf .. Inf should be NaN, not 0' | 01:16 | |||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/295829183 github.com/rakudo/rakudo/compare/5...efe96ffb9d | |||||||||||||||||||||||||||||||||||
samcv | good * everybody | 01:42 | |||||||||||||||||||||||||||||||||
timotimo | yo samcv | 01:50 | |||||||||||||||||||||||||||||||||
samcv | hey | ||||||||||||||||||||||||||||||||||
timotimo, what do you think of a function for composing ropes. that we could use in concatenation and other functions. so you initialize an MVMstruct for the string, and the ncall a function to add strands onto it. where would that be a good place to use other than join and concatenation functions? anywhere else where that would be benefictial? | 01:51 | ||||||||||||||||||||||||||||||||||
timotimo | if you include repetitions, then the x operator maybe? | 01:53 | |||||||||||||||||||||||||||||||||
samcv | well. we're not talking about the x operator | 01:59 | |||||||||||||||||||||||||||||||||
timotimo | other than that i have no good ideas | 02:00 | |||||||||||||||||||||||||||||||||
samcv | gist.github.com/samcv/5c2eb18fa5d4...3e8bca8fc4 something like this | 02:02 | |||||||||||||||||||||||||||||||||
a mvm function that would let us easily compose ropes | |||||||||||||||||||||||||||||||||||
timotimo | i can see the appeal | 02:03 | |||||||||||||||||||||||||||||||||
samcv | one of the things i noticed is it's not trivial to compose ropes. we have concatenation but nothing else really does it. so i'm thinking if i have enough reasons for doing this | 02:04 | |||||||||||||||||||||||||||||||||
it may be worth it to add these functions | |||||||||||||||||||||||||||||||||||
i'm basically asking someone to convince me i should do this :) | 02:07 | ||||||||||||||||||||||||||||||||||
timotimo | the code looks good. it'll probably come out the same after gcc throws all its optimizations at it, but it's likely less error-prone | 02:12 | |||||||||||||||||||||||||||||||||
i'm going to bed, seeya! | 02:18 | ||||||||||||||||||||||||||||||||||
Zoffix | Man, I'm feeling the pain with all our Unicode ops and Unicode in outpout (e.g 「」). Even Noto font doesn't render them in InDesign; looks like whatever fallback mechanism usually exist doesn't work in typesetting software :( | 02:34 | |||||||||||||||||||||||||||||||||
Finally managed to typeset this snippet with 3 fonts :P gist.github.com/zoffixznet/a4f23c4...d22f514f5a | 02:37 | ||||||||||||||||||||||||||||||||||
Doesn't quite work still, 'cause I wanted to highlight it but can't apply two char styles together :| | 02:39 | ||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Elizabeth Mattijsen 'More Telemetry additions | 06:24 | |||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/295954168 github.com/rakudo/rakudo/compare/5...4f0c6ceb20 | |||||||||||||||||||||||||||||||||||
Rakudo build passed. Elizabeth Mattijsen 'Increase the size of gtc/ttc, gfldex++' | 07:12 | ||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/295979112 github.com/rakudo/rakudo/compare/7...e1faaa713b | |||||||||||||||||||||||||||||||||||
gfldex | . | 07:30 | |||||||||||||||||||||||||||||||||
yoleaux | 00:06Z <lizmat> gfldex: hope this is enough to write a nice blogpost about Telemetry :-) | ||||||||||||||||||||||||||||||||||
gfldex | .tell lizmat could you make %format available to members of the public please? | 07:32 | |||||||||||||||||||||||||||||||||
yoleaux | gfldex: I'll pass your message to lizmat. | ||||||||||||||||||||||||||||||||||
bartolin | samcv: it looks like we have a regression with some tests for "\c[BELL]" on jvm. I think that might be related to your recent work on control characters (nqp commit 7954f9b0c1). | 07:34 | |||||||||||||||||||||||||||||||||
m: say "\c[BELL]".uniname # gives <control-0007> on jvm | |||||||||||||||||||||||||||||||||||
camelia | BELL | ||||||||||||||||||||||||||||||||||
bartolin | samcv: I believe that happens because nqp::ord("\c[BELL]") returns 7 on jvm | 07:35 | |||||||||||||||||||||||||||||||||
samcv: do you have a pointer, perhaps, how "\c[BELL]" is processed? | 07:36 | ||||||||||||||||||||||||||||||||||
samcv: I've read through RT #130542 and if I understood correctly, BELL is a special case, because it was U+0007 in Unicode 1. So, maybe we need to add a correcsponding special case somewhere. (I saw that you already did that for nqp::codepointfromname in nqp commit deb8cb03e.) | 07:40 | ||||||||||||||||||||||||||||||||||
synopsebot | RT#130542 [resolved]: rt.perl.org/Ticket/Display.html?id=130542 [JVM] [UNI] \c[BELL] returns the U+0007 control code not U+1F514 BELL | ||||||||||||||||||||||||||||||||||
lizmat | Files=1229, Tests=75811, 318 wallclock secs (14.79 usr 5.38 sys + 2198.71 cusr 212.47 csys = 2431.35 CPU) | ||||||||||||||||||||||||||||||||||
yoleaux | 07:32Z <gfldex> lizmat: could you make %format available to members of the public please? | ||||||||||||||||||||||||||||||||||
lizmat | gfldex: I see that as an implementation detail, what would you like to achieve by that ? | 07:41 | |||||||||||||||||||||||||||||||||
bartolin | samcv: ah, one last thing: I've seen those failing tests in S32-str/parse-names.t | ||||||||||||||||||||||||||||||||||
work & | |||||||||||||||||||||||||||||||||||
gfldex | lizmat: I would like to display the descriptions. | 08:00 | |||||||||||||||||||||||||||||||||
lizmat: and when %format is exported &report could get a :$format | 08:02 | ||||||||||||||||||||||||||||||||||
samcv | bartolin, oh there's failing tests? | 08:08 | |||||||||||||||||||||||||||||||||
bartolin | samcv: here is a gist of runnning S32-str/parse-names.t with a current rakudo-j: gist.github.com/usev6/2e15b87350d4...fc3ae9f952 | 08:17 | |||||||||||||||||||||||||||||||||
samcv | bartolin, on jvm? | 08:18 | |||||||||||||||||||||||||||||||||
ah ok | |||||||||||||||||||||||||||||||||||
bartolin | *nods* | ||||||||||||||||||||||||||||||||||
the tests are fine on moar | |||||||||||||||||||||||||||||||||||
samcv | well. it's due to utf8-16 | ||||||||||||||||||||||||||||||||||
err utf-18 | |||||||||||||||||||||||||||||||||||
*16 | 08:19 | ||||||||||||||||||||||||||||||||||
bartolin | *g* | ||||||||||||||||||||||||||||||||||
those tests have been passing before, though (didn't bisect, but I thought it might happen since github.com/perl6/nqp/commit/7954f9b0c1 | 08:20 | ||||||||||||||||||||||||||||||||||
gfldex | m: class T does Associative { method AT-KEY(\k) { 42 but role :: { method desc { "answer" } } } }; say T.new<foo>.desc | 08:35 | |||||||||||||||||||||||||||||||||
camelia | answer | ||||||||||||||||||||||||||||||||||
gfldex | lizmat: maybe with a sneaky ↑↑↑ role? | ||||||||||||||||||||||||||||||||||
Geth | rakudo: 0dc4a0ebad | (Elizabeth Mattijsen)++ | lib/Telemetry.pm6 Telemetry tweaks #1 for today - handle 0 wallclock better, apparently Windows has bigger granularity - add initial memory size (max-rss) to report |
09:04 | |||||||||||||||||||||||||||||||||
lizmat | m: multi sub a(:@c is copy) { }; a # WOW, something we forgot in the GLR | 09:19 | |||||||||||||||||||||||||||||||||
camelia | replace this Array is copy logic in sub a at <tmp> line 1 in block <unit> at <tmp> line 1 |
||||||||||||||||||||||||||||||||||
lizmat | github.com/rakudo/rakudo/blob/mast...P.nqp#L443 | 09:21 | |||||||||||||||||||||||||||||||||
jnthn | Wow! | 09:27 | |||||||||||||||||||||||||||||||||
I see it's a highly used feature ;) | |||||||||||||||||||||||||||||||||||
lizmat | apparently :-) | 09:28 | |||||||||||||||||||||||||||||||||
oddly enough, uncommmenting the following lines appears to make it work | 09:29 | ||||||||||||||||||||||||||||||||||
lizmat runs a spectest just in case | 09:30 | ||||||||||||||||||||||||||||||||||
samcv | hah wow | ||||||||||||||||||||||||||||||||||
lizmat | jnthn: do you see anything obviously wrong with the code ? | ||||||||||||||||||||||||||||||||||
jnthn | Not immediately | 09:32 | |||||||||||||||||||||||||||||||||
lizmat | basically the same code as for :%h is copy | ||||||||||||||||||||||||||||||||||
samcv | lizmat, looks like more in Any.pm | ||||||||||||||||||||||||||||||||||
XXX GLR comments | |||||||||||||||||||||||||||||||||||
lizmat | yeah, those I know about :-) | ||||||||||||||||||||||||||||||||||
I think :-) | 09:33 | ||||||||||||||||||||||||||||||||||
m: sub a(:@c is copy) { }; a | 09:37 | ||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||
lizmat | jnthn: apparently only in the case of a multi sub | 09:38 | |||||||||||||||||||||||||||||||||
Geth | rakudo: be1e2879f3 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/BOOTSTRAP.nqp Make multi sub(:@c is copy) { } work Apparently this was looked over during the GLR |
||||||||||||||||||||||||||||||||||
[TuxCM] |
|
09:43 | |||||||||||||||||||||||||||||||||
Geth | rakudo: c2baf95e1b | (Elizabeth Mattijsen)++ | lib/Telemetry.pm6 Telemetry tweaks #2 for today - "columns" is now a named parameter to "report" - specify the columns you want to see, either by header or method name - %*ENV<RAKUDO_REPORT_COLUMNS> can now be used to set columns you want to see - if you don't specify any columns in the call to "report" |
09:47 | |||||||||||||||||||||||||||||||||
lizmat | afk for a few hours& | 09:48 | |||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Elizabeth Mattijsen 'Add Telemetry is-rss,min-flt,maj-flt,nswap,inblock,outblock | 10:00 | |||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/296009383 github.com/rakudo/rakudo/compare/9...751ee87c93 | |||||||||||||||||||||||||||||||||||
Rakudo build passed. Elizabeth Mattijsen 'Normalize all Telemetry x-rss to Kbytes | 10:35 | ||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/296013392 github.com/rakudo/rakudo/compare/9...00908cb272 | |||||||||||||||||||||||||||||||||||
Rakudo build failed. Elizabeth Mattijsen 'Add Telemetry msgsnd,msgrcv,nsignals,nvcsw,invcsw | 11:33 | ||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/296023846 github.com/rakudo/rakudo/compare/7...1c31e1d723 | |||||||||||||||||||||||||||||||||||
buggable | [travis build above] ✓ All failures are due to: GitHub connectivity (1 failure). | ||||||||||||||||||||||||||||||||||
Zoffix | .tell timotimo there looks to be a stale PR by you to mp6o: github.com/perl6/modules.perl6.org...l/86/files IIRC the problem with it was that data fetcher doesn't fetch travis files | 12:25 | |||||||||||||||||||||||||||||||||
yoleaux | Zoffix: I'll pass your message to timotimo. | ||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Elizabeth Mattijsen 'Telemetry tweaks #1 for today | 12:27 | |||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/296164052 github.com/rakudo/rakudo/compare/d...c4a0ebad21 | |||||||||||||||||||||||||||||||||||
Zoffix | Some feedback on our state: www.reddit.com/r/perl/comments/79w...s/dp6r8j7/ | 12:46 | |||||||||||||||||||||||||||||||||
lizmat | m: class A { has Int @.a[4] = die }; dd A.new # clearly the initializer is not being run | 12:49 | |||||||||||||||||||||||||||||||||
camelia | A.new(a => Array[Int].new(:shape(4,), [Int, Int, Int, Int])) | ||||||||||||||||||||||||||||||||||
lizmat | m: class A { has Int @.a[4] is required }; dd A.new # not even the required check ??? | 12:50 | |||||||||||||||||||||||||||||||||
camelia | A.new(a => Array[Int].new(:shape(4,), [Int, Int, Int, Int])) | ||||||||||||||||||||||||||||||||||
pmurias | Zoffix: re hard to navigate the docs search sucks | ||||||||||||||||||||||||||||||||||
Zoffix | pmurias: ok, good. That's one of the things that'll be improved by the redesign. | 12:51 | |||||||||||||||||||||||||||||||||
That post actual points out a counter-point to people who say a new name for the language will nullify google results. It's actually a good thing in this case, where the user's saying they're getting outdated results :P | 12:52 | ||||||||||||||||||||||||||||||||||
lizmat | m: class A { has %.a is BagHash is required }; dd A.new # looks like "is required" is a noop in these cases :-( | 12:53 | |||||||||||||||||||||||||||||||||
camelia | A.new(a => ().BagHash) | ||||||||||||||||||||||||||||||||||
Zoffix & | 12:54 | ||||||||||||||||||||||||||||||||||
pmurias | Zoffix: keeping the renaming debate in #perl6 out of #perl6-dev seems like a good idea ;) | ||||||||||||||||||||||||||||||||||
Zoffix | pmurias: why? | ||||||||||||||||||||||||||||||||||
To me, the reverse is true. | |||||||||||||||||||||||||||||||||||
The devs are the ones who should be the most involved in the discussion that concerns the marketing of this thing we're making. | 12:55 | ||||||||||||||||||||||||||||||||||
This is the boardroom with all the directors. You're making it seem like the suggestion to move the marketing discussions to the customer service deparment :) | 12:57 | ||||||||||||||||||||||||||||||||||
Zoffix leaves to do @urgent-things | |||||||||||||||||||||||||||||||||||
pmurias: oh and there's an easy way to reduce the amount of these discussions. Resolve the 6.d blocker so we could move forward with 6.d release: github.com/perl6/6.d-prep/blob/mas...s-language The naming debate culminates then. | 13:11 | ||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Elizabeth Mattijsen 'Make multi sub(:@c is copy) { } work | 13:24 | |||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/296175854 github.com/rakudo/rakudo/compare/0...1e2879f37e | |||||||||||||||||||||||||||||||||||
Rakudo build passed. Elizabeth Mattijsen 'Telemetry tweaks #2 for today | 14:22 | ||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/296179262 github.com/rakudo/rakudo/compare/b...baf95e1b37 | |||||||||||||||||||||||||||||||||||
Geth | rakudo: 9f54bc989d | (Elizabeth Mattijsen)++ | src/core/Exception.pm Add X::NYI.workaround So we can give workaround information if necessary |
15:10 | |||||||||||||||||||||||||||||||||
rakudo: 0973b307ad | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/BUILDPLAN.nqp Temporary fix for GH #1226 - throw an X::Comp::NYI for buildplan code 9 with a 4,5,6,7 - show a work around Can't help but wonder that if we recommend this workaround, why not automatically generate that code then? This could be done by adding another code block like TWEAK to the BUILDPLAN, with the correct code pre-generated. |
15:31 | ||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1226 [open]: github.com/rakudo/rakudo/issues/1226 Broken default value assignment on shaped array attributes | ||||||||||||||||||||||||||||||||||
Zoffix | m: class F { has @.color[4]; submethod TWEAK (:@!color = <1 2 3 4>) {} }; F.new.color.say; # shorter FWIW | 15:43 | |||||||||||||||||||||||||||||||||
camelia | [1 2 3 4] | ||||||||||||||||||||||||||||||||||
lizmat | m: class F { has @.color[4]; submethod TWEAK (:@!color = <1 2 3 4>) {} }; F.new(color => <f g h i>).color.say; | 15:44 | |||||||||||||||||||||||||||||||||
camelia | [f g h i] | ||||||||||||||||||||||||||||||||||
lizmat | ah, good point ! | 15:45 | |||||||||||||||||||||||||||||||||
Geth | rakudo: 6dab5aad89 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/BUILDPLAN.nqp Improve GH #1226 workaround suggestion, Zoffix++ |
15:52 | |||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1226 [open]: github.com/rakudo/rakudo/issues/1226 Broken default value assignment on shaped array attributes | ||||||||||||||||||||||||||||||||||
Zoffix wonders if that'll make it come out as `(:@.color …` instead of @! twigil | 15:54 | ||||||||||||||||||||||||||||||||||
lizmat | 'class F { has @.color[4] = <1 2 3 4> }' comes out as 'method TWEAK(:@!color = (initial values)) { }' for me | 15:55 | |||||||||||||||||||||||||||||||||
Zoffix | \o/ | 15:56 | |||||||||||||||||||||||||||||||||
lizmat | m: use Telemetry; say T<max-rss> | 15:59 | |||||||||||||||||||||||||||||||||
camelia | 91504 | ||||||||||||||||||||||||||||||||||
lizmat | m: use Telemetry; say T<max-rss> | ||||||||||||||||||||||||||||||||||
camelia | 87740 | ||||||||||||||||||||||||||||||||||
lizmat | wonder where these differences come from | ||||||||||||||||||||||||||||||||||
m: use Telemetry :COLUMNS; say max-rss # wow | 16:00 | ||||||||||||||||||||||||||||||||||
camelia | 116860 | ||||||||||||||||||||||||||||||||||
ugexe | what does it mean if there is no max-rss? | ||||||||||||||||||||||||||||||||||
yoleaux | 08:54Z <tbrowder> ugexe: how can i get a local install of rakudo to find Debugger::UI::CommandLine (and its dependencies) in system-wide COMPUNITS? alternatively, how can i get zef to —install-to=/some/path? | ||||||||||||||||||||||||||||||||||
lizmat | ugexe: that your OS doesn't support it | 16:01 | |||||||||||||||||||||||||||||||||
man7.org/linux/man-pages/man2/getrusage.2.html | |||||||||||||||||||||||||||||||||||
ugexe | lizmat: on osx for 1..10 { snap } shows max-rss twice but the rest are blank | ||||||||||||||||||||||||||||||||||
lizmat | ah, blank means no change, it hides 0 as that makes for a very noisy overview | 16:02 | |||||||||||||||||||||||||||||||||
ugexe | i see, same for ics/gw/etc? | ||||||||||||||||||||||||||||||||||
lizmat | yup | ||||||||||||||||||||||||||||||||||
otherwise you're just looking at a page with 0's | 16:03 | ||||||||||||||||||||||||||||||||||
and some none zero values | |||||||||||||||||||||||||||||||||||
Zoffix | tbrowder: what I usually do is run Step 12 from the release guide except for the last `zef install ...` replaced with the module I need: github.com/rakudo/rakudo/blob/nom/..._guide.pod | ||||||||||||||||||||||||||||||||||
tbrowder | Zoffix: thanks! BTW, any updates to yr markdown to html for advent entries since last year? | 16:07 | |||||||||||||||||||||||||||||||||
Zoffix | Nope. I'm not sure where the code is at even. Probably can be improved a bit and shared with all the authors. | 16:10 | |||||||||||||||||||||||||||||||||
So yesterday I was whinging because there wasn't an easy way to style code parts in InDesign to have syntax highlights. Turns out it supports PCRE in paragraph styles to apply substyles. And a bunch of regex hackery later, I managed to make a single paragraph style that reasonably-well highlights Perl 6 code up in it: i.imgur.com/1GsZsBZ.png | 16:21 | ||||||||||||||||||||||||||||||||||
Not that anyone here cares about InDesign, but it'll make creating marketing brochures with code about 1000x faster :) | |||||||||||||||||||||||||||||||||||
ZofBot: do the happy dance, bud | 16:22 | ||||||||||||||||||||||||||||||||||
ZofBot | Zoffix, 'Would you have done all that for me?' she said | ||||||||||||||||||||||||||||||||||
Zoffix | For you, I would've! | ||||||||||||||||||||||||||||||||||
timotimo | the other day i was trying something with List | 16:24 | |||||||||||||||||||||||||||||||||
the reason why i even started trying the nqp::attrinited stuff on List was that $!todo.DEFINITE will autovivify a Scalar container even for a List object that'll never have (and never had) an iterator part | 16:25 | ||||||||||||||||||||||||||||||||||
so this is perhaps an opportunity for improvement. though maybe we actually want to do this in the optimizer | 16:26 | ||||||||||||||||||||||||||||||||||
lizmat | hmmm... good one | 16:38 | |||||||||||||||||||||||||||||||||
timotimo | okay, so ... | ||||||||||||||||||||||||||||||||||
i outputted every "demanded" object and stable with Telemetry vs Telemetry :COLUMNS | |||||||||||||||||||||||||||||||||||
and with :COLUMNS it's actually fewer things in there | 16:39 | ||||||||||||||||||||||||||||||||||
it could be that more stuff hangs off of these, though | |||||||||||||||||||||||||||||||||||
nope, even with the work loop itself instrumented it's fewer things if :COLUMNS is specified | 16:44 | ||||||||||||||||||||||||||||||||||
time for the heap analyzer | |||||||||||||||||||||||||||||||||||
nothing very obvious | 16:48 | ||||||||||||||||||||||||||||||||||
Geth | rakudo: e1a1b8d865 | (Elizabeth Mattijsen)++ | lib/Telemetry.pm6 Add support for Telemetry affinity-tasks-completed - also add this to the default report |
17:05 | |||||||||||||||||||||||||||||||||
jnthn | Urgh, a bug I've been hunting for the last few hours might boil down to something wrong with the CLOSE phaser | 17:14 | |||||||||||||||||||||||||||||||||
lizmat | whee | 17:16 | |||||||||||||||||||||||||||||||||
also: if you can't start another thread, should we act on that somehow more than just ignoring ? | 17:17 | ||||||||||||||||||||||||||||||||||
jnthn | In what sense? | 17:18 | |||||||||||||||||||||||||||||||||
Thread.start explodes pretty forcefully, and perhaps too much so, if it fails | |||||||||||||||||||||||||||||||||||
(It's a VM panic) | 17:19 | ||||||||||||||||||||||||||||||||||
lizmat | m: await start { sleep 1 } xx 64; say T.general-workers # never goes beyond 47 for me | ||||||||||||||||||||||||||||||||||
camelia | 5===SORRY!5=== Error while compiling <tmp> Undeclared name: T used at line 1 |
||||||||||||||||||||||||||||||||||
lizmat | m: use Telemetry; await start { sleep 1 } xx 64; say T.general-workers # never goes beyond 47 for me | ||||||||||||||||||||||||||||||||||
camelia | MoarVM panic: Could not spawn thread: errorcode -11 | ||||||||||||||||||||||||||||||||||
lizmat | ooh wow | ||||||||||||||||||||||||||||||||||
jnthn | That's 'cus of limits in the environment camelia runs under | ||||||||||||||||||||||||||||||||||
lizmat | on MacOS it just says 47 | ||||||||||||||||||||||||||||||||||
if run with RAKUDO_SCHEDULER_DEBUG, I get a lot of: Heuristic queue progress deadlock situation detected | 17:20 | ||||||||||||||||||||||||||||||||||
followed by Could not create a new Thread: too many open files | |||||||||||||||||||||||||||||||||||
jnthn | Ah | ||||||||||||||||||||||||||||||||||
Well, I think it's probably doing the right thing | |||||||||||||||||||||||||||||||||||
No reason to bring down a program that will eventually work | 17:21 | ||||||||||||||||||||||||||||||||||
Instead of leaving it to complete | |||||||||||||||||||||||||||||||||||
Zoffix | I still get "too many open files" even if I set RAKUDO_MAX_THREADS=10 | 17:24 | |||||||||||||||||||||||||||||||||
ilmari | what does 'ulimit -n' output? | 17:25 | |||||||||||||||||||||||||||||||||
Zoffix | hm, nm, setting it to 7 does go through the "max threads reached" message. | 17:26 | |||||||||||||||||||||||||||||||||
perlpilot | "too many open files" is a strange way to say ENOMORETHREADS | ||||||||||||||||||||||||||||||||||
Zoffix | ilmari: I set it to 50 | ||||||||||||||||||||||||||||||||||
But I guess there are other things using files, so the left over is 7 for threads :) | |||||||||||||||||||||||||||||||||||
jnthn | perlpilot: Probably really means "handles" | ||||||||||||||||||||||||||||||||||
tbrowder | Zoffix: I have a copy of your md2html from last year. It's at github gist: gist.github.com/tbrowder/32154ec02...8fd4823d8, plus my code wrapping it for my md source. Both are at gist | 17:29 | |||||||||||||||||||||||||||||||||
ignore words after the gist, please | 17:30 | ||||||||||||||||||||||||||||||||||
Zoffix | m: await start { sleep 1 } xx 30 | 17:33 | |||||||||||||||||||||||||||||||||
camelia | MoarVM panic: Could not spawn thread: errorcode -11 | ||||||||||||||||||||||||||||||||||
Zoffix | So what limitations does camelia's env have? I set all the same ulimit values (a few days ago) and didn't have that issue locally. | ||||||||||||||||||||||||||||||||||
It's kind'f an unwanted issue IMO. The work can be done on a couple of threads really and the program I wrote that works elsewhere crashes here for reasons I don't know. | 17:35 | ||||||||||||||||||||||||||||||||||
(this also didn't exist in 2017.08) | |||||||||||||||||||||||||||||||||||
*.07 | |||||||||||||||||||||||||||||||||||
star: await start { sleep 1 } xx 30 | |||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||
jnthn | Then set RAKUDO_MAX_THREADS=16 and you'll have the 2017.07 max pool size back | 17:36 | |||||||||||||||||||||||||||||||||
Zoffix | m: BEGIN %*ENV<RAKUDO_MAX_THREADS> = 16; await start { sleep 1 } xx 30 | ||||||||||||||||||||||||||||||||||
camelia | MoarVM panic: Could not spawn thread: errorcode -11 | ||||||||||||||||||||||||||||||||||
jnthn | I think it actually has to be in the env | 17:37 | |||||||||||||||||||||||||||||||||
Zoffix | m: BEGIN %*ENV<RAKUDO_MAX_THREADS> = 2; await start { sleep 1 } xx 30 | ||||||||||||||||||||||||||||||||||
camelia | MoarVM panic: Could not spawn thread: errorcode -11 | ||||||||||||||||||||||||||||||||||
jnthn | As the scheduler instance is created at CORE.setting load time | ||||||||||||||||||||||||||||||||||
Zoffix | 2 just worked in #zofbot | ||||||||||||||||||||||||||||||||||
jnthn | Can perhaps also look into whether the panic can be turned into a throw | 17:38 | |||||||||||||||||||||||||||||||||
In which case the sueprvisor will be able to silently swallow it | |||||||||||||||||||||||||||||||||||
Zoffix | m: use nqp; nqp::bindattr(nqp::decont($*SCHEDULER), ThreadPoolScheduler, q|$!max_threads|, 16); await start { sleep 1 } xx 30 | ||||||||||||||||||||||||||||||||||
jnthn | Just needs some care to make sure that we don't end up with a corrupt internal VM state as a result | ||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||
Zoffix | k, yeah, works with 16 | ||||||||||||||||||||||||||||||||||
jnthn | Which may be why it's a panic now | 17:39 | |||||||||||||||||||||||||||||||||
Looking very much like the CLOSE phaser bug was the root cause of the problem I was hunting. Thankfully this case is trivial to rewrite in terms of .on-close, which is safe | 17:40 | ||||||||||||||||||||||||||||||||||
I'll file an issue later, or mebbe tomorrow | 17:41 | ||||||||||||||||||||||||||||||||||
Phew. That was some debugging. But, we'll get Cro 0.7.1 this week (or weekend) thanks to it. :) | 17:48 | ||||||||||||||||||||||||||||||||||
jnthn disappers to cook pau bahji | 17:49 | ||||||||||||||||||||||||||||||||||
Geth | rakudo: b380230dd9 | (Elizabeth Mattijsen)++ | lib/Telemetry.pm6 Add support for Telemetry affinity-tasks-queued |
18:00 | |||||||||||||||||||||||||||||||||
lizmat steps away for a few hours to grok | 18:01 | ||||||||||||||||||||||||||||||||||
Zoffix | ZofBot: twerk it: www.youtube.com/watch?v=-pqvQ8k308I | 18:50 | |||||||||||||||||||||||||||||||||
ZofBot | Zoffix, It's a hobby of mine | ||||||||||||||||||||||||||||||||||
Zoffix rofls | |||||||||||||||||||||||||||||||||||
buggable | New CPAN upload: bamboo-0.000.003_20171102.tar.gz by PSIXDISTS cpan.metacpan.org/authors/id/P/PS/...102.tar.gz | 19:06 | |||||||||||||||||||||||||||||||||
New CPAN upload: DateTime-Parse-0.000.003_20171102.tar.gz by PSIXDISTS cpan.metacpan.org/authors/id/P/PS/...102.tar.gz | |||||||||||||||||||||||||||||||||||
ugexe | Supply has no .close, but there was a multi sub close(Supply:D) added | 19:07 | |||||||||||||||||||||||||||||||||
Geth | rakudo: ef84aafc01 | (Nick Logan)++ (committed using GitHub Web editor) | src/core/io_operators.pm Remove invalid close() candidate Supply has no close method |
19:11 | |||||||||||||||||||||||||||||||||
roast: 02f72506a6 | usev6++ | 4 files [jvm] Unfudge some passing tests |
19:32 | ||||||||||||||||||||||||||||||||||
[Coke] | ... does the cpan upload of bamboo also depend on panda? | 19:33 | |||||||||||||||||||||||||||||||||
Zoffix | I don't know why the PSIXDISTS user is uploading anything. It's ignored by modules.perl6.org | ||||||||||||||||||||||||||||||||||
I thought we were doing those uploads only as an experimental measure. They're violating the license of some modules. | 19:34 | ||||||||||||||||||||||||||||||||||
lizmat | mst should know more | 19:39 | |||||||||||||||||||||||||||||||||
but I was also under the impression that PSIXDIST was hated | |||||||||||||||||||||||||||||||||||
*halted | |||||||||||||||||||||||||||||||||||
oops | |||||||||||||||||||||||||||||||||||
Zoffix | :P | ||||||||||||||||||||||||||||||||||
mst | I'll make sure the cron job gets turned off | 19:40 | |||||||||||||||||||||||||||||||||
it was really just a way to remind perl6 that cpan actually existed | |||||||||||||||||||||||||||||||||||
lizmat | ugexe: good catch: I just acked for "method close", saw there was one in Supply.pm and added it | ||||||||||||||||||||||||||||||||||
mst | now you appear to have finally noticed it can go away | 19:41 | |||||||||||||||||||||||||||||||||
lizmat | afk again& | ||||||||||||||||||||||||||||||||||
Zoffix | mst++ | ||||||||||||||||||||||||||||||||||
mst: FWIW, I'm including this blurb about Perl in a new Rakudo Perl 6 brochure: gist.github.com/zoffixznet/b8e883c...dfbe2ed17a | 19:42 | ||||||||||||||||||||||||||||||||||
If it needs to be tweaked, let me know. | |||||||||||||||||||||||||||||||||||
mst | looks fine to me | 19:43 | |||||||||||||||||||||||||||||||||
Zoffix | \o/ | ||||||||||||||||||||||||||||||||||
[Coke] | Zoffix: yup, looks good. | 20:12 | |||||||||||||||||||||||||||||||||
jnthn | Yup, it's Tap that has close, not Supply | 20:14 | |||||||||||||||||||||||||||||||||
perlpilot | Zoffix: check the comments on that gist | 20:18 | |||||||||||||||||||||||||||||||||
Zoffix++ btw | |||||||||||||||||||||||||||||||||||
mst | Zoffix: cronjob is now off. | 20:36 | |||||||||||||||||||||||||||||||||
Zoffix | Sweet. Thanks. | 20:49 | |||||||||||||||||||||||||||||||||
Geth | nqp: usev6++ created pull request #380: [jvm] Fix \c[BELL] returning U+0007 |
21:24 | |||||||||||||||||||||||||||||||||
synopsebot | NQP#380 [open]: github.com/perl6/nqp/pull/380 [jvm] Fix \c[BELL] returning U+0007 | ||||||||||||||||||||||||||||||||||
bartolin | samcv: ^^ if you have a minute or two to look at that PR, that would be great | 21:27 | |||||||||||||||||||||||||||||||||
samcv | thanks bartolin will look in a sec | 21:55 | |||||||||||||||||||||||||||||||||
bartolin, i thought i fixed that previously hmm | |||||||||||||||||||||||||||||||||||
oh. awesome thanks bartolin | 21:56 | ||||||||||||||||||||||||||||||||||
Geth | nqp: 66e58e8f59 | usev6++ | src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java [jvm] Fix \c[BELL] returning U+0007 |
21:58 | |||||||||||||||||||||||||||||||||
nqp: 5a03f1e9c6 | (Samantha McVey)++ (committed using GitHub Web editor) | src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java Merge pull request #380 from usev6/jvm_bell [jvm] Fix \c[BELL] returning U+0007 |
|||||||||||||||||||||||||||||||||||
synopsebot | NQP#380 [closed]: github.com/perl6/nqp/pull/380 [jvm] Fix \c[BELL] returning U+0007 | ||||||||||||||||||||||||||||||||||
bartolin | \o/ thanks samcv++ # all that work on unicode stuff | 22:00 | |||||||||||||||||||||||||||||||||
samcv | you're welcome :-) | 22:05 | |||||||||||||||||||||||||||||||||
Geth | nqp: de0f955305 | (Samantha McVey)++ | tools/build/MOAR_REVISION Bump MoarVM for join and Unicode property improvements Full changelog: e76788819 unicode: Make sure 'Cn' codepoints match with C enum 25bbba8c5 unicode: make sure to return 0 not "" from get_property_int e4b39aa1d show the actual type when "no such attribute" error occurs 87191078c Add MVMROOT2..MVMROOT5 to make rooting multiple variables easier 4136ae256 speed up join of longer strings by a significant amount 11ab1dcbe join: factor code into join_get_str_from_pos() function |
22:10 | |||||||||||||||||||||||||||||||||
nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...ge76788819 8e0cd7a4e8 | MasterDuke17++ | src/vm/moar/QAST/QASTOperationsMAST.nqp |
|||||||||||||||||||||||||||||||||||
travis-ci | NQP build failed. Samantha McVey 'Bump MoarVM for join and Unicode property improvements | 22:30 | |||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/296505978 github.com/perl6/nqp/compare/5a03f...0f955305b3 | |||||||||||||||||||||||||||||||||||
Zoffix | .ask lizmat are you able to be to advise on the brochure or have access to woolfy to ask? I see it has grown considerably in size, driving up printing costs. I don't think my version will be any larger than the current 60-page one, but should I compress it as much as I can to keep the costs down or are the costs less important than more pleasant aesthetics? | 22:58 | |||||||||||||||||||||||||||||||||
yoleaux | Zoffix: I'll pass your message to lizmat. | ||||||||||||||||||||||||||||||||||
lizmat | . | 23:05 | |||||||||||||||||||||||||||||||||
yoleaux | 22:58Z <Zoffix> lizmat: are you able to be to advise on the brochure or have access to woolfy to ask? I see it has grown considerably in size, driving up printing costs. I don't think my version will be any larger than the current 60-page one, but should I compress it as much as I can to keep the costs down or are the costs less important than more pleasant aesthetics? | ||||||||||||||||||||||||||||||||||
lizmat | :-) | ||||||||||||||||||||||||||||||||||
nqp: b8c85f824b | (Zoffix Znet)++ (committed using GitHub Web editor) | src/vm/moar/QAST/QASTOperationsMAST.nqp Merge pull request #378 from MasterDuke17/add_coerce_II_op Add the new coerce_II op as fromI_I |
|||||||||||||||||||||||||||||||||||
synopsebot | NQP#378 [closed]: github.com/perl6/nqp/pull/378 Add the new coerce_II op as fromI_I | ||||||||||||||||||||||||||||||||||
jnthn | Zoffix: Just in case you forgot: that NQP merge'll need a MoarVM version bump | 23:46 | |||||||||||||||||||||||||||||||||
Zoffix | yup | 23:50 | |||||||||||||||||||||||||||||||||
samcv | i bumped MVM already | 23:54 | |||||||||||||||||||||||||||||||||
oh looks like new commits. nvm | |||||||||||||||||||||||||||||||||||
Zoffix | TODO passed: 543-545, 549 t/spec/S05-mass/properties-general.t | ||||||||||||||||||||||||||||||||||
ZOFFLOP: t/spec/S11-modules/nested.t t/spec/S11-modules/require.t | |||||||||||||||||||||||||||||||||||
ZOFVM: Files=1283, Tests=152820, 155 wallclock secs (21.58 usr 3.17 sys + 3322.77 cusr 193.19 csys = 3540.71 CPU) | 23:55 | ||||||||||||||||||||||||||||||||||
Geth | nqp: a3f7bb8177 | (Zoffix Znet)++ | tools/build/MOAR_REVISION Bump MoarVM |
||||||||||||||||||||||||||||||||||
¦ nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...7-gf88ab16 | |||||||||||||||||||||||||||||||||||
rakudo: dbf0a21f90 | (Zoffix Znet)++ | tools/build/NQP_REVISION Bump NQP |
|||||||||||||||||||||||||||||||||||
travis-ci | NQP build failed. Zoffix Znet 'Merge pull request #378 from MasterDuke17/add_coerce_II_op | 23:56 | |||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/296540731 github.com/perl6/nqp/compare/de0f9...c85f824b30 | |||||||||||||||||||||||||||||||||||
Geth | rakudo: 6cb7ebfb9f | MasterDuke17++ | src/core/Int.pm Use the new nqp::fromI_I in Int.new The op will create a new Int using the value of the given Int, but remove any mix-ins. |
||||||||||||||||||||||||||||||||||
rakudo: 264eef7988 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/core/Int.pm Merge pull request #1217 from MasterDuke17/use_fromI_I_in_Int.new Use the new nqp::fromI_I in Int.new |
|||||||||||||||||||||||||||||||||||
samcv | Zoffix, you can untodo those now passing tests now that rakudo has been bumped if you want Zoffix. otherwise i'll get to it a little later | ||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1217 [closed]: github.com/rakudo/rakudo/pull/1217 Use the new nqp::fromI_I in Int.new | ||||||||||||||||||||||||||||||||||
samcv | err actually wait. gotta push some changes | ||||||||||||||||||||||||||||||||||
Geth | roast: b42fcb389c | (Zoffix Znet)++ | S05-mass/properties-general.t Unfudge passing todos |
23:58 |