Geth | roast: b477bee606 | MasterDuke17++ (committed using GitHub Web editor) | CONTRIBUTING.md Add a couple of suggestions Wording stolen nearly verbatim from zoffix++. |
01:05 | |||||||||||||||||||||||||||||||||||||
rakudo: ronaldxs++ created pull request #1706: Update 05-messages/03-errors.t test for RT #132238 |
01:11 | ||||||||||||||||||||||||||||||||||||||
synopsebot | RT#132238 [new]: rt.perl.org/Ticket/Display.html?id=132238 [LTA] $<> and “Corresponding … was at line” | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 27e77c8d03 | (Ronald Schmidt)++ | t/05-messages/03-errors.t Update 05-messages/03-errors.t test for RT #132238 clearer error message for unclosed <> hash quote index operator |
01:13 | |||||||||||||||||||||||||||||||||||||
rakudo: 9ff655645b | (Zoffix Znet)++ (committed using GitHub Web editor) | t/05-messages/03-errors.t Merge pull request #1706 from ronaldxs/rakudo-t-RT-132238-match-hash-2 Update 05-messages/03-errors.t test for RT #132238 |
|||||||||||||||||||||||||||||||||||||||
AlexDaniel | m: 42 xx ∞ | 02:52 | |||||||||||||||||||||||||||||||||||||
camelia | (timeout) | 02:53 | |||||||||||||||||||||||||||||||||||||
Geth | roast: b320464868 | (Aleks-Daniel Jakimenko-Aleksejev)++ | S03-operators/repeat.t Test xx not dying with large repetition count Closes RT#126014. |
03:43 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#126014 [open]: rt.perl.org/Ticket/Display.html?id=126014 Too many repetitions with xx operator causes out of memory; should it work lazily? | ||||||||||||||||||||||||||||||||||||||
Geth | roast: 19344b3c7f | (Aleks-Daniel Jakimenko-Aleksejev)++ | S03-operators/repeat.t Oops. Fix RT number in the last commit |
03:45 | |||||||||||||||||||||||||||||||||||||
roast: a7af87465e | (Aleks-Daniel Jakimenko-Aleksejev)++ | S32-io/io-handle.t Test that nl-in has \n and \r\n Closes RT#131858, but I still don't understand why it gives an array instead of a Junction. There's probably a good reason, but speccing it this way is probably wrong. Please adjust. |
04:22 | ||||||||||||||||||||||||||||||||||||||
synopsebot | RT#131858 [open]: rt.perl.org/Ticket/Display.html?id=131858 [REGRESSION] default $.nl-in on IO::Handle does not correctly work in subclasses | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | m: 42 xx 2⁹⁹⁹⁹⁹ | 04:26 | |||||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | m: 42 xx 2⁹⁹ | 04:27 | |||||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | m: 42 xx 2⁶³ | ||||||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | m: 42 xx 2⁶⁰ | ||||||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||||||
Geth | roast: ec2b4c3a7c | (Aleks-Daniel Jakimenko-Aleksejev)++ | S32-io/io-handle.t Change nl-in test to is-deeply |
05:07 | |||||||||||||||||||||||||||||||||||||
roast: 22001cb924 | (Aleks-Daniel Jakimenko-Aleksejev)++ | S04-statements/for.t Use is-deeply in hole tests |
05:12 | ||||||||||||||||||||||||||||||||||||||
rakudo: 5f256baec2 | (Zoffix Znet)++ | src/core/Rakudo/Iterator.pm6 Implement optimized Iterator methods for `42 xx ...` Fixes R#1708 github.com/rakudo/rakudo/issues/1708 |
05:25 | ||||||||||||||||||||||||||||||||||||||
synopsebot | R#1708 [open]: github.com/rakudo/rakudo/issues/1708 … xx ∞ should return a Seq | ||||||||||||||||||||||||||||||||||||||
Geth | roast: b4b0d584bc | (Zoffix Znet)++ | S03-operators/repeat.t Test opt methods/sinkage of `xx` with simple value Closes github.com/rakudo/rakudo/issues/1708 R#1708 Rakudo fix: github.com/rakudo/rakudo/commit/5f256baec2 |
05:29 | |||||||||||||||||||||||||||||||||||||
roast: 5fe5978a18 | (Zoffix Znet)++ (committed using GitHub Web editor) | CONTRIBUTING.md Use implementation-neutral language |
05:51 | ||||||||||||||||||||||||||||||||||||||
roast: 03ac03f16c | usev6++ | MISC/bug-coverage-stress.t [JVM] Skip tests using 'atomicint' |
06:21 | ||||||||||||||||||||||||||||||||||||||
Zoffix | ZOFHANG: t/spec/S07-hyperrace/stress.t | 07:18 | |||||||||||||||||||||||||||||||||||||
ZOFVM: Files=1294, Tests=153299, 142 wallclock secs (20.31 usr 3.18 sys + 3026.02 cusr 141.63 csys = 3191.14 CPU) | 07:22 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 4402839adb | (Zoffix Znet)++ | src/Perl6/Actions.nqp Fix crash in double-sink for for ... { Package::foo } Fixes R#1693 github.com/rakudo/rakudo/issues/1693 After adding a sink call to the last statement, we UNWANTED, the guts and because of the setup, we ended up adding another sink call into the QAST inside UNWANTED, causing the error in the Issue. ... (5 more lines) |
07:29 | |||||||||||||||||||||||||||||||||||||
synopsebot | R#1693 [open]: github.com/rakudo/rakudo/issues/1693 [regression][⚠ blocker ⚠] Cannot find method 'Nil' | ||||||||||||||||||||||||||||||||||||||
roast: 06d2abbc6c | (Zoffix Znet)++ | S04-statements/sink.t Test sinkage of package call in for Closes github.com/rakudo/rakudo/issues/1693 R#1693 Rakudo fix: github.com/rakudo/rakudo/commit/4402839adb |
07:30 | ||||||||||||||||||||||||||||||||||||||
Zoffix | ZOFHANG: t/spec/S07-hyperrace/stress.t | 07:47 | |||||||||||||||||||||||||||||||||||||
releasable6: next | 07:48 | ||||||||||||||||||||||||||||||||||||||
releasable6 | Zoffix, Next release in ≈13 days and ≈11 hours. 6 blockers. 0 out of 188 commits logged (⚠ 9 warnings) | ||||||||||||||||||||||||||||||||||||||
Zoffix, Details: gist.github.com/04a8b844fed55d64eb...030f104e51 | |||||||||||||||||||||||||||||||||||||||
Zoffix | huggable: star | ||||||||||||||||||||||||||||||||||||||
huggable | Zoffix, Estimated Rakudo Star releases for 2017: .01, .04, .07 & .10 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 9bcc8e523e | (Zoffix Znet)++ | src/core/CompUnit/PrecompilationUnit.pm6 Fix thinko in CURR conditional Part of fixing R#1662 github.com/rakudo/rakudo/issues/1662 |
07:52 | |||||||||||||||||||||||||||||||||||||
synopsebot | R#1662 [open]: github.com/rakudo/rakudo/issues/1662 [⚠ blocker ⚠] Windows Issues with "Replace error-prone file timestamp check by checksum when loading deps" | ||||||||||||||||||||||||||||||||||||||
Zoffix | ZOFFLOP: t/spec/S17-procasync/basic.t | 08:00 | |||||||||||||||||||||||||||||||||||||
Shoved t/spec/S07-hyperrace/stress.t into a loop, while stresstesting in another terminal, after ~5min it crashed with `Type check failed in binding to parameter '$batch'; expected Rakudo::Internals::HyperWorkBatch but got Mu (Mu)` Full output: gist.github.com/zoffixznet/5740c60...3416b87515 | 08:06 | ||||||||||||||||||||||||||||||||||||||
k, reproed a hang with a small bit of t/spec/S07-hyperrace/stress.t again | 08:13 | ||||||||||||||||||||||||||||||||||||||
Filed as github.com/rakudo/rakudo/issues/1709 R#1709 | 08:18 | ||||||||||||||||||||||||||||||||||||||
synopsebot | R#1709 [open]: github.com/rakudo/rakudo/issues/1709 [ASYNC][⚠ blocker ⚠] A race in HyperRace causing rare crash/hang | ||||||||||||||||||||||||||||||||||||||
Zoffix | ZOFFLOP: t/spec/S06-advanced/stub.t | 08:21 | |||||||||||||||||||||||||||||||||||||
ZOFFLOP: t/spec/S10-packages/basic.t | |||||||||||||||||||||||||||||||||||||||
ZOFFLOP: t/spec/S10-packages/basict | |||||||||||||||||||||||||||||||||||||||
ZOFFLOP: t/spec/S02-types/capture.t | 08:22 | ||||||||||||||||||||||||||||||||||||||
ZOFFLOP: t/spec/S06-other/main.t | |||||||||||||||||||||||||||||||||||||||
ZOFFLOP: t/spec/S17-procasync/basic.t | |||||||||||||||||||||||||||||||||||||||
That's the floppers from 7 stresstest runs | 08:23 | ||||||||||||||||||||||||||||||||||||||
S10-packages/basic.t and S17-procasync/basic.t are familiar offenders, but the rest are freshcomers | |||||||||||||||||||||||||||||||||||||||
Zoffix fires off another 10 runs | 08:27 | ||||||||||||||||||||||||||||||||||||||
And filed R#1710 not sure what else to say in the issue otehr than I see a lot more random floppers I've not seen before | |||||||||||||||||||||||||||||||||||||||
synopsebot | R#1710 [open]: github.com/rakudo/rakudo/issues/1710 New stresstest floppers | ||||||||||||||||||||||||||||||||||||||
FROGGS | o/ | 08:28 | |||||||||||||||||||||||||||||||||||||
Zoffix | \o | ||||||||||||||||||||||||||||||||||||||
nine | Zoffix: are they more reproducible with these env vars? MVM_SPESH_BLOCKING=1 MVM_SPESH_NODELAY=1 | 08:31 | |||||||||||||||||||||||||||||||||||||
Zoffix re-starts 10 runs with those vars set | 08:38 | ||||||||||||||||||||||||||||||||||||||
I got 4 files I've not seen before and t/spec/S17-procasync/basic.t on the first run. Gonna post results on the ticket after a few more runs | 08:46 | ||||||||||||||||||||||||||||||||||||||
Posted 5 runs: github.com/rakudo/rakudo/issues/17...-379534521 Looks like most of them aren't floppers but are just reproducible failures | 09:16 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | Zoffix: fwiw we can bisect with MVM_SPESH_BLOCKING=1 MVM_SPESH_NODELAY=1 if needed | 09:20 | |||||||||||||||||||||||||||||||||||||
so maybe someone can try golfing one of these and then we'll bisect | |||||||||||||||||||||||||||||||||||||||
and even if bisect points to a bump, we'll at least have a golfed version :) | |||||||||||||||||||||||||||||||||||||||
robertle | what do MVM_SPESH_NODELAY and MVM_SPESH_BLOCKING actually do? | 09:24 | |||||||||||||||||||||||||||||||||||||
Zoffix | MVM_SPESH_NODELAY Run dynamic optimization even for cold frames | 09:25 | |||||||||||||||||||||||||||||||||||||
MVM_SPESH_BLOCKING Blocks log-sending thread while specializer runs | |||||||||||||||||||||||||||||||||||||||
There's a bunch more in (see moar --help) | |||||||||||||||||||||||||||||||||||||||
robertle: basically makes spesh bugs pop up a lot easier, as far as I understand it | 09:26 | ||||||||||||||||||||||||||||||||||||||
spesh = our dynamic optimizer | |||||||||||||||||||||||||||||||||||||||
[Tux] |
|
09:28 | |||||||||||||||||||||||||||||||||||||
And I got this | 09:30 | ||||||||||||||||||||||||||||||||||||||
All tests successful. | |||||||||||||||||||||||||||||||||||||||
Files=30, Tests=22441, 25 wallclock secs ( 2.49 usr 0.25 sys + 84.88 cusr 2.84 csys = 90.46 CPU) | |||||||||||||||||||||||||||||||||||||||
Result: PASS | |||||||||||||||||||||||||||||||||||||||
Zoffix | ? Do you mean you didn't get that beoore? | ||||||||||||||||||||||||||||||||||||||
*before | 09:31 | ||||||||||||||||||||||||||||||||||||||
Geth | roast: JJ++ created pull request #414: Adds test for RT#126394 |
10:02 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#126394 [open]: rt.perl.org/Ticket/Display.html?id=126394 [BUG] Assigning Failure to typed variable swallows Failure information | ||||||||||||||||||||||||||||||||||||||
Geth | roast: ce173d4c66 | (JJ Merelo)++ | S04-exceptions/fail.t Adds test for RT#126394 Checking that it fails and effectively includes the important message. |
10:03 | |||||||||||||||||||||||||||||||||||||
roast: 295d101a09 | (Zoffix Znet)++ (committed using GitHub Web editor) | S04-exceptions/fail.t Merge pull request #414 from JJ/master Adds test for RT#126394 |
|||||||||||||||||||||||||||||||||||||||
roast: 39b4d2a83e | (Zoffix Znet)++ (committed using GitHub Web editor) | S04-exceptions/fail.t Remove needless ticket title (along with possibly-conflicting with fudger `#?` portion) |
10:04 | ||||||||||||||||||||||||||||||||||||||
nqp: d6061c5f36 | (Jonathan Worthington)++ | src/vm/moar/QAST/QASTRegexCompilerMAST.nqp Don't emit cstack handling code if unused Anything that doesn't capture or won't need to backtrack into a subrule call will not need a cstack. We already allocate it lazily, but this change means we don't even generate code to trim it when the null check guarding it would always come out true. This saves the check and makes the generated code more compact. This results in a bit over 3KB less bytecode from compiling the NQP grammar. This change combined with the previous one to not emit unused cclass setup save around 43KB off the compiled output of the Perl 6 grammar. |
10:07 | ||||||||||||||||||||||||||||||||||||||
roast: d3c04b47ca | (Zoffix Znet)++ (committed using GitHub Web editor) | S04-exceptions/fail.t Add test description (and toss unneeded `say` code) |
|||||||||||||||||||||||||||||||||||||||
robertle | thanks, I am trying to reproduce sporadic build failures on s390x and mips, and was wondering if these could affect behavior... doesn't look like they make a massive difference though | 10:15 | |||||||||||||||||||||||||||||||||||||
Geth | roast: 1f171a9d2f | (Aleks-Daniel Jakimenko-Aleksejev)++ | S07-hyperrace/basics.t Add tests for hyperized s/// Should also catch other reliability issues of .hyper, including segfaults. Closes RT#126752. |
10:44 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#126752 [open]: rt.perl.org/Ticket/Display.html?id=126752 [BUG] $/ vs .hyper gets confused | ||||||||||||||||||||||||||||||||||||||
Geth | roast: dogbert17++ created pull request #415: Add test for RT #127974 |
12:15 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#127974 [open]: rt.perl.org/Ticket/Display.html?id=127974 [CONC] sprintf() not threadsafe/reentrant if the format tokens use explicit indices | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: ronaldxs++ created pull request #1712: Update 05-messages/03-errors.t simpler and clearer tests for RT#132238 |
13:03 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#132238 [resolved]: rt.perl.org/Ticket/Display.html?id=132238 [LTA] $<> and “Corresponding … was at line” | ||||||||||||||||||||||||||||||||||||||
Geth | nqp/master: 4 commits pushed by pmurias++ | 13:37 | |||||||||||||||||||||||||||||||||||||
nqp: 95116aa460 | usev6++ | src/vm/jvm/runtime/org/perl6/nqp/io/AsyncProcessHandle.java [JVM] Make process ID of started child available Please note: I didn't test the code on Windows. |
14:39 | ||||||||||||||||||||||||||||||||||||||
nqp: f79f0ce1a4 | (Elizabeth Mattijsen)++ | tools/build/MOAR_REVISION Get latest Moar for nqp::floor_n JITting |
17:37 | ||||||||||||||||||||||||||||||||||||||
¦ nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...7-ga206cd8 | |||||||||||||||||||||||||||||||||||||||
rakudo: a53811cd4d | (Elizabeth Mattijsen)++ | tools/build/NQP_REVISION Bump NQP to get the nqp::floor_n jitting |
18:09 | ||||||||||||||||||||||||||||||||||||||
rakudo: version bump brought these changes: github.com/perl6/nqp/compare/2018....2-gf79f0ce d9869e53b5 | (Elizabeth Mattijsen)++ | src/core/Array.pm6 Hopefully that will mean they optimize better in the future |
|||||||||||||||||||||||||||||||||||||||
roast: 3727e0314a | (Jan-Olof Hendig)++ | S07-hyperrace/basics.t Add test for RT #127974 |
18:36 | ||||||||||||||||||||||||||||||||||||||
roast: 76d6344efd | dogbert17++ (committed using GitHub Web editor) | S07-hyperrace/basics.t Update basics.t Don't merge this yet as the added test seems to uncover another bug. AlexDaniel++ for the test. |
|||||||||||||||||||||||||||||||||||||||
synopsebot | RT#127974 [open]: rt.perl.org/Ticket/Display.html?id=127974 [CONC] sprintf() not threadsafe/reentrant if the format tokens use explicit indices | ||||||||||||||||||||||||||||||||||||||
Geth | roast: d21d2b33f9 | (Zoffix Znet)++ (committed using GitHub Web editor) | S07-hyperrace/basics.t Merge pull request #415 from dogbert17/test-rt-127974 Add test for RT #127974 |
||||||||||||||||||||||||||||||||||||||
rakudo: a2a6aad8be | (Ronald Schmidt)++ | t/05-messages/03-errors.t Update 05-messages/03-errors.t simpler and clearer tests for RT#132238 unclosed $< . |
18:39 | ||||||||||||||||||||||||||||||||||||||
rakudo: 318a9b090c | (Zoffix Znet)++ (committed using GitHub Web editor) | t/05-messages/03-errors.t Merge pull request #1712 from ronaldxs/rakudo-t-RT-132238-match-hash-2 Update 05-messages/03-errors.t simpler and clearer tests for RT#132238 |
|||||||||||||||||||||||||||||||||||||||
synopsebot | RT#132238 [resolved]: rt.perl.org/Ticket/Display.html?id=132238 [LTA] $<> and “Corresponding … was at line” | ||||||||||||||||||||||||||||||||||||||
lizmat | is the last test of S07-hyperrace/basics.t supposed to hang ? | 19:00 | |||||||||||||||||||||||||||||||||||||
it hangs for me reliably :- | |||||||||||||||||||||||||||||||||||||||
( | |||||||||||||||||||||||||||||||||||||||
Zoffix | lizmat: don't think so. And there's also R#1709 I filed last night | 19:02 | |||||||||||||||||||||||||||||||||||||
synopsebot | R#1709 [open]: github.com/rakudo/rakudo/issues/1709 [ASYNC][⚠ blocker ⚠] A race in HyperRace causing rare crash/hang | ||||||||||||||||||||||||||||||||||||||
lizmat | ok, would be surprised if it is my last local change causing this | 19:03 | |||||||||||||||||||||||||||||||||||||
nope, it's not | 19:06 | ||||||||||||||||||||||||||||||||||||||
how long is that test supposed to run? seconds ? | |||||||||||||||||||||||||||||||||||||||
Zoffix | Files=1, Tests=83, 4 wallclock secs ( 0.04 usr 0.00 sys + 4.24 cusr 0.38 csys = 4.66 CPU) | 19:07 | |||||||||||||||||||||||||||||||||||||
(doesn't hang for me) | |||||||||||||||||||||||||||||||||||||||
dogbert17 | I don't think that PR should have been merged just yet ... | 19:08 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: 5ec4a5ab44 | (Elizabeth Mattijsen)++ | 2 files Use nqp::elems instead of .elems for native arrays Saves 1 method call + a number of allocations. Makes e.g. copying native arrays about 9% faster |
19:09 | |||||||||||||||||||||||||||||||||||||
dogbert17 | "roast: Don't merge this yet as the added test seems to uncover another bug. AlexDaniel++ for the test." | 19:10 | |||||||||||||||||||||||||||||||||||||
Zoffix | dogbert17: well, it'd be helpful to specify that in the PR's description instead of burried in the commit messages. The PR was marged as approved: i.imgur.com/Q8wIhUY.png | 19:12 | |||||||||||||||||||||||||||||||||||||
s/merged/marked/; | |||||||||||||||||||||||||||||||||||||||
dogbert17 | Zoffix: it was indeed, I'm the one to blame for this | 19:13 | |||||||||||||||||||||||||||||||||||||
it seems as if there are still a few bugs in the implementation of 'race' | 19:14 | ||||||||||||||||||||||||||||||||||||||
samcv | Zoffix: what do you think about Int.random which will get a random integer? | 19:16 | |||||||||||||||||||||||||||||||||||||
or we could call it another name if we want | 19:17 | ||||||||||||||||||||||||||||||||||||||
Zoffix | .oO( YAGNI ) |
19:18 | |||||||||||||||||||||||||||||||||||||
samcv | there's no way to generate a random bigint though | ||||||||||||||||||||||||||||||||||||||
Zoffix | IMO that's going backwards: you see a way to implement a feature and try to think of the usecase, where the usecase should come first. | 19:20 | |||||||||||||||||||||||||||||||||||||
samcv | though i am thinking about making .rand more random. because our origin source of entropy is just "time * pid" | 19:21 | |||||||||||||||||||||||||||||||||||||
which is pretty unrandom | |||||||||||||||||||||||||||||||||||||||
Zoffix | Current .rand gives a range of quadrillion random numbers. Not all of them are representable in Num, in 2**53 every second one is representable and it increases as you increase or decrease that, but even if we dial down by three magnitudes, that's a trillion random numbers. | 19:22 | |||||||||||||||||||||||||||||||||||||
samcv | but i should be able to mix in some actual randomness | ||||||||||||||||||||||||||||||||||||||
Zoffix | So yeah, I can see trailing zeros in 1e300.rand, but I don't have a usecase where I need more than a trillion random numbers. | ||||||||||||||||||||||||||||||||||||||
And I can always combine multiple random portions of those Nums to form a very large Int if I ever need more than a trillion. | 19:23 | ||||||||||||||||||||||||||||||||||||||
samcv | and .rand acts based on the seed value. nqp::rand_I i believe doesn't if the platform has a random number system function | ||||||||||||||||||||||||||||||||||||||
so the usecase may be greater for getting actual random numbers from the os instead of a deterministic random number | 19:24 | ||||||||||||||||||||||||||||||||||||||
i'll think on that i suppose | |||||||||||||||||||||||||||||||||||||||
Zoffix | Perfect for a module. | 19:25 | |||||||||||||||||||||||||||||||||||||
samcv | it would have to use nqp though? that seems less than ideal | ||||||||||||||||||||||||||||||||||||||
Zoffix | Why? Every language has modules that produce true random numbers. I doubt they all rely on exposed guts of the language's interpreter/compiler. | 19:26 | |||||||||||||||||||||||||||||||||||||
But still IMO that's going backwards: you see a way to implement a feature and try to think of the usecase, where the usecase should come first. | 19:27 | ||||||||||||||||||||||||||||||||||||||
samcv | well they have to get it from somewhere. i mean i guess you could use nativecall. but not sure how well that'd work cross platform | ||||||||||||||||||||||||||||||||||||||
but as i said i'll think about this stuff | 19:28 | ||||||||||||||||||||||||||||||||||||||
Zoffix | What do you need true random numbers for? What are you building? What are you building that you need so many true random numbers that it'd be convenient to have it as a core feature? | ||||||||||||||||||||||||||||||||||||||
samcv | and in the meantime at least seed with something better than(epochtime * PID) | ||||||||||||||||||||||||||||||||||||||
Zoffix | This is the same as IO::Path.parent(UInt). It was argued about how needed it is and implemented but like half a year later it was still undocumented and untested and no one used it. Just shows the feature was implemented without a user actually needin git. | 19:29 | |||||||||||||||||||||||||||||||||||||
Same with true randomness. Do all OSes provide the means to produce true random numbers? And what'll be the behaviour when available entropy is exhausted? | 19:30 | ||||||||||||||||||||||||||||||||||||||
samcv | no i agree with what you're saying. and thank you for providing your opinion on this | 19:31 | |||||||||||||||||||||||||||||||||||||
Zoffix: .parent does seem to be used in modules. though i appologize for not adding it to the docs | 19:34 | ||||||||||||||||||||||||||||||||||||||
i'll be back a bit later. o/ | 19:36 | ||||||||||||||||||||||||||||||||||||||
lizmat_ | .tell jnthn for something like "my int @a = ^100; my int @b = @a" more than 80% of the time is spent in MetaModel code, which seems suboptimal :-) | ||||||||||||||||||||||||||||||||||||||
yoleaux | lizmat_: I'll pass your message to jnthn. | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 8c265f9baa | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/CurriedRoleHOW.nqp Micro-optimize CurriedRoleHOW.accept_type This makes it 1% faster. But with it being responsible for 40+% of time in something like "my int @a = ^100; my int @b = @a", it seemed worthwhile. |
19:39 | |||||||||||||||||||||||||||||||||||||
lizmat_ | .tell jnthn feels like we're missing a caching opportunity here | 19:40 | |||||||||||||||||||||||||||||||||||||
yoleaux | lizmat_: I'll pass your message to jnthn. | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: e8ec81db06 | (Elizabeth Mattijsen)++ | src/core/native_array.pm6 Make my int @a = ^100 about 40% faster Fewer method calls, 2 allocations less per call |
20:21 | |||||||||||||||||||||||||||||||||||||
rakudo: 6c2f24455c | (Zoffix Znet)++ | src/core/Rakudo/Internals.pm6 Include backtrace when printing THE-END exceptions |
20:32 | ||||||||||||||||||||||||||||||||||||||
lizmat | Zoffix++ | 20:33 | |||||||||||||||||||||||||||||||||||||
MasterDuke | zoffix++, but i think it would be good to put a '\n' before the backtrace | 20:37 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: 5fee5891c1 | (Zoffix Znet)++ | src/core/Rakudo/Internals.pm6 Include \n before backtrace; MasterDuke++ |
20:41 | |||||||||||||||||||||||||||||||||||||
rakudo: 91fe6b63d3 | (Zoffix Znet)++ | src/core/Rakudo/Internals.pm6 Indent backtraces in THE-END exceptions |
21:04 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | dogbert17: ok just fudge the test then | 21:07 | |||||||||||||||||||||||||||||||||||||
lizmat | this is about the last race test that hangs ? | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | yes | ||||||||||||||||||||||||||||||||||||||
lizmat | ok, I had already fudged it locally | ||||||||||||||||||||||||||||||||||||||
will push | 21:08 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | okay | ||||||||||||||||||||||||||||||||||||||
Geth | roast: f5246f8b3e | (Elizabeth Mattijsen)++ | S07-hyperrace/basics.t Fudge new test that is hanging on at least MacOS |
||||||||||||||||||||||||||||||||||||||
dogbert17 | lizmat++ | 21:13 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: 40815d16db | (Elizabeth Mattijsen)++ | 2 files Make initialization of int(8|16|32|64)/num(32|64) arrays 9x as fast Due to a bad signature, it was using the slow, generic path. |
21:17 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | c: HEAD say do for ^20 { [do for ^5 { ("a".."z").roll(3).join }] }; | 21:20 | |||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, gist.github.com/a2cdebffbf3da91d44...57f4a1cc80 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | c: HEAD do for ^1000 { [do for ^5 { ("a".."z").roll(3).join }] }; | 21:21 | |||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, ¦HEAD(91fe6b6): «» | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | c: HEAD do for ^10000 { [do for ^5 { ("a".."z").roll(3).join }] }; | ||||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, ¦HEAD(91fe6b6): «» | ||||||||||||||||||||||||||||||||||||||
timotimo | ooooh | 21:22 | |||||||||||||||||||||||||||||||||||||
lizmat++ | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | did anybody manage to reproduce github.com/MoarVM/MoarVM/issues/838 ? | 21:23 | |||||||||||||||||||||||||||||||||||||
lizmat | afraid I didn't | 21:24 | |||||||||||||||||||||||||||||||||||||
jnthn | Me either, but I don't have HEAD of everything | 21:25 | |||||||||||||||||||||||||||||||||||||
yoleaux | 19:36Z <lizmat_> jnthn: for something like "my int @a = ^100; my int @b = @a" more than 80% of the time is spent in MetaModel code, which seems suboptimal :-) | ||||||||||||||||||||||||||||||||||||||
19:40Z <lizmat_> jnthn: feels like we're missing a caching opportunity here | |||||||||||||||||||||||||||||||||||||||
jnthn | Suboptimal indeed :) | ||||||||||||||||||||||||||||||||||||||
Will have to look at that | 21:26 | ||||||||||||||||||||||||||||||||||||||
lizmat | it *may* have something to do with the candidates, looking at that now | ||||||||||||||||||||||||||||||||||||||
will get back to you on that :-) | |||||||||||||||||||||||||||||||||||||||
jnthn | Alright :) | 21:27 | |||||||||||||||||||||||||||||||||||||
lizmat | I think github.com/rakudo/rakudo/commit/40815d16db fixed it | 21:29 | |||||||||||||||||||||||||||||||||||||
but maybe not :-) | |||||||||||||||||||||||||||||||||||||||
jnthn | aha | 21:30 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: c015f08dec | (Zoffix Znet)++ | 2 files Fix crash with unknown column in RAKUDO_REPORT_COLUMNS Fixes github.com/rakudo/rakudo/issues/1714 Warn that we don't know what that is instead. |
21:35 | |||||||||||||||||||||||||||||||||||||
dogbert17 | was it a good conference? | 21:42 | |||||||||||||||||||||||||||||||||||||
lizmat | dogbert17: I thought it was pretty good... should have some videos of it before long :) | 21:43 | |||||||||||||||||||||||||||||||||||||
dogbert17 | any interesting speeches? | 21:44 | |||||||||||||||||||||||||||||||||||||
timotimo | three jonathan talks :) :) | 21:45 | |||||||||||||||||||||||||||||||||||||
dogbert17 | aha, interesting | 21:46 | |||||||||||||||||||||||||||||||||||||
lizmat | yes, they all were :-) | 21:48 | |||||||||||||||||||||||||||||||||||||
MasterDuke | Zoffix, lizmat: re github.com/rakudo/rakudo/commit/c015f08dec, tcd is a valid column/field according to docs.perl6.org/type/Telemetry::Ins...t::Thread, there just weren't any values recorded for it | 21:51 | |||||||||||||||||||||||||||||||||||||
would it make sense for the warning to mention that as a possibility? | 21:52 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: b5318e6ef2 | (Elizabeth Mattijsen)++ | 2 files Make my @a = ^100; my int @b = @a about 9x as fast - create a separate List:D source candidate that can use nqp::atpos - also for native str and num |
||||||||||||||||||||||||||||||||||||||
rakudo: ronaldxs++ created pull request #1715: Update 05-messages/03-errors.t to add test for RT #122980. |
|||||||||||||||||||||||||||||||||||||||
synopsebot | RT#122980 [open]: rt.perl.org/Ticket/Display.html?id=122980 [BUG] LTA error message on fairly strange input, complaining about the lack of a semicolon when the semicolon's right there in Rakudo | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 1c33453cd6 | (Ronald Schmidt)++ | t/05-messages/03-errors.t Update 05-messages/03-errors.t to add test for RT #122980 and check for the invalid "missing semicolon" message. The rakudo versions with the problem were over three years old and the Test libraries were not compatible with 2018 03-errors.t so testing with a symptomatic version had to be manually jury rigged. If test is still wanted think about the best that can be done. |
21:59 | |||||||||||||||||||||||||||||||||||||
rakudo: 95481e26bd | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | t/05-messages/03-errors.t Merge pull request #1715 from ronaldxs/test-RT-122980-wrong-msg-semi Update 05-messages/03-errors.t to add test for RT #122980. |
|||||||||||||||||||||||||||||||||||||||
Zoffix | MasterDuke: that column wasn't on my box. Maybe the available columns depend on the OS? I'm on Bodhi Linux. There's a bunch of columns (like IIRC `act`) that didn't have any values and don't show up in the output, but a warning wouldn't be shown for those. | 22:02 | |||||||||||||||||||||||||||||||||||||
or was it there.... | 22:03 | ||||||||||||||||||||||||||||||||||||||
Zoffix re-stricks debug prints | |||||||||||||||||||||||||||||||||||||||
MasterDuke: if I dump %format.keys at line 845 of lib/Telemetry.pm6, I get ["tw", "gw", "atc", "util\%", "volcsw", "nswp", "minf", "cpu", "ttq", "mrcv", "cpu-sys", "ttc", "s", "aw", "wallclock", "is-rss", "invcsw", "inb", "cpus", "gtc", "atq", "outb", "nsig", "max-rss", "ix-rss", "gtq", "cpu-user", "majf", "id-rss", "msnd"] | 22:04 | ||||||||||||||||||||||||||||||||||||||
And `act` is one of teh default columns for which no data is collected | |||||||||||||||||||||||||||||||||||||||
timotimo | you want atc, not act | ||||||||||||||||||||||||||||||||||||||
Zoffix | yeah, atc | 22:05 | |||||||||||||||||||||||||||||||||||||
timotimo | ah, that was just a typo | ||||||||||||||||||||||||||||||||||||||
Zoffix | And if I write RAKUDO_REPORT_COLUMNS=atc I don't get output nor warning, but if I write RAKUDO_REPORT_COLUMNS=act I get the "unknown column" warning | ||||||||||||||||||||||||||||||||||||||
timotimo | aw, that's not good | 22:06 | |||||||||||||||||||||||||||||||||||||
Zoffix | Which part? | 22:08 | |||||||||||||||||||||||||||||||||||||
MasterDuke | github.com/rakudo/rakudo/blob/mast...y.pm6#L234 | ||||||||||||||||||||||||||||||||||||||
line 247 even shows it as a default column. but maybe there is some OS dependence? | 22:09 | ||||||||||||||||||||||||||||||||||||||
Zoffix | Ah k | 22:13 | |||||||||||||||||||||||||||||||||||||
MasterDuke: If I set RAKUDO_TELEMETRY_INSTRUMENTS=Thread I get tcd and bunch of other columns | |||||||||||||||||||||||||||||||||||||||
With no data in them | 22:14 | ||||||||||||||||||||||||||||||||||||||
MasterDuke | ah, cool. let me try that... | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 6cdee60df3 | (Zoffix Znet)++ | lib/Telemetry.pm6 Suggest RAKUDO_TELEMETRY_INSTRUMENTS in warning |
22:16 | |||||||||||||||||||||||||||||||||||||
MasterDuke | heh, i once got tsd and thid, but usually it's blank | ||||||||||||||||||||||||||||||||||||||
Zoffix++ | |||||||||||||||||||||||||||||||||||||||
Geth | nqp: 463e56954b | (Zoffix Znet)++ | tools/build/MOAR_REVISION [MoarVM Bump] Brings 2 commits MoarVM bump brought: github.com/MoarVM/MoarVM/compare/2...9-g4234ab5 4234ab5 throw out problematic impl of floor_n in jit cfa1f2a Fix memory corruption when running with FSA_SIZE_DEBUG set |
22:24 | |||||||||||||||||||||||||||||||||||||
¦ nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...9-g4234ab5 | |||||||||||||||||||||||||||||||||||||||
rakudo: 0a08f1d0aa | (Zoffix Znet)++ | tools/build/NQP_REVISION [NQP Bump] 463e569 [MoarVM Bump] Brings 2 commits NQP bump brought: github.com/perl6/nqp/compare/2018....3-g463e569 MoarVM bump brought: github.com/MoarVM/MoarVM/compare/2...9-g4234ab5 4234ab5 throw out problematic impl of floor_n in jit cfa1f2a Fix memory corruption when running with FSA_SIZE_DEBUG set |
|||||||||||||||||||||||||||||||||||||||
¦ rakudo: version bump brought these changes: github.com/perl6/nqp/compare/2018....3-g463e569 | |||||||||||||||||||||||||||||||||||||||
roast: bf28c7796e | (Zoffix Znet)++ | MISC/bug-coverage.t Cover Illegal Instruction crash Covers M#838 github.com/MoarVM/MoarVM/issues/838 |
22:26 | ||||||||||||||||||||||||||||||||||||||
synopsebot | M#838 [closed]: github.com/MoarVM/MoarVM/issues/838 [⚠ blocker ⚠] SIGILL in MoarVM HEAD, works in 2018.03 | ||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build failed. Zoffix Znet 'Indent backtraces in THE-END exceptions' | 23:13 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/363872717 github.com/rakudo/rakudo/compare/5...fe6b63d3a5 | |||||||||||||||||||||||||||||||||||||||
buggable | [travis build above] ☠ Did not recognize some failures. Check results manually. | ||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Zoffix Znet 'Fix crash with unknown column in RAKUDO_REPORT_COLUMNS | 23:51 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/363880298 github.com/rakudo/rakudo/compare/4...15f08dec6b | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 404f30217b | (Zoffix Znet)++ | t/06-telemetry/01-basic.t Fix new test on Windows We flatten is-run code into a single line to avoid issues with windows runner, so we need a colon after the brace. |
23:59 |