| lizmat | [Coke] you around ? | 00:02 | |
| .tell [Coke] my blog post is ready for publication: if you need more time for yours, please feel free to swap mine for yours | 00:05 | ||
| yoleaux | lizmat: I'll pass your message to [Coke]. | ||
| lizmat | sleep& | 00:06 | |
| tyil | gn | 00:09 | |
| Geth | rakudo: 2a8287cf89 | (Zoffix Znet)++ | 2 files Rename Str.parse-names to Str.uniparse The original name was chosen to align with `.parse-base` that parses out base-X numbers out strings. However, we have a whole block of more closely related routines all named in `.uni*` format, so it makes sense for this routine's naming to align with those. They are: `infix:<unicmp>`, `unimatch`, `uniname`, `uninames`, `unival`, `univals`, `uniprop`, ... (5 more lines) |
00:28 | |
| roast: 3efe6cb8da | (Zoffix Znet)++ | 2 files [6.d REVIEW] Rename Str.parse-names to Str.uniparse The original name was chosen to align with `.parse-base` that parses out base-X numbers out strings. However, we have a whole block of more closely related routines all named in `.uni*` format, so it makes sense for this routine's naming to align with those. They are: `infix:<unicmp>`, `unimatch`, `uniname`, `uninames`, `unival`, `univals`, `uniprop`, ... (7 more lines) |
00:29 | ||
| 6.d-prep: 075c98619c | (Zoffix Znet)++ (committed using GitHub Web editor) | TODO/FEATURES.md Update .parse-names -> .uniname status |
00:31 | ||
| 6.d-prep: a23c68002e | (Zoffix Znet)++ (committed using GitHub Web editor) | TODO/FEATURES.md Fix markdown error |
00:32 | ||
| Zoffix | .tell lizmat .uninames added, if you wanted to mention it in your blog post: github.com/perl6/doc/commit/bff42f80b1 (.parse-names will still exist as deprecated until 6.e) | 00:33 | |
| yoleaux | Zoffix: I'll pass your message to lizmat. | ||
| Zoffix | .tell lizmat I meant .uniparse :) | ||
| yoleaux | Zoffix: I'll pass your message to lizmat. | ||
| Zoffix | m: '482,PENGUIN'.uniparse.say | 00:49 | |
| evalable6 | (exit code 1) Unrecognized character name [482] in block <unit> at /tmp/UtWehELyOE line 1 |
||
| Zoffix | m: '\c[482,PENGUIN]'.say | ||
| evalable6 | \c[482,PENGUIN] | ||
| Zoffix | m: "\c[482,PENGUIN]".say | ||
| evalable6 | Ǣ🐧 | ||
| Zoffix | :/ I thought .uniparse/.parse-names used the same nqp routine as \c :/ | ||
| u: 482 | |||
| unicodable6 | Zoffix, U+3482 <CJK Ideograph Extension A-3482> [Lo] (㒂) | ||
| Zoffix, U+4482 <CJK Ideograph Extension A-4482> [Lo] (䒂) | |||
| Zoffix, 789 characters in total: gist.github.com/edfe37f7a695aa16d7...88b98ac1ba | |||
| Zoffix | m: 482.chr.say | 00:50 | |
| evalable6 | Ǣ | ||
| Zoffix | Well, don't wanna add it. overengineering-- | 00:54 | |
| AlexDaniel | fwiw I remember that mine is on 23rd :) | 01:05 | |
| Geth | 6.d-prep: 72a65c80e3 | (Zoffix Znet)++ (committed using GitHub Web editor) | TODO/README.md Add HTML version of `glop` 6.d review alias |
01:40 | |
| roast: 0fdb8da9b1 | (Zoffix Znet)++ | S04-statements/return.t Remove trailing whitespace |
01:43 | ||
| roast: 4e7e4006f4 | (Zoffix Znet)++ | S04-statements/return.t [6.d REVIEW] Remove unsureness comment |
01:44 | ||
| roast: ce4968703b | (Zoffix Znet)++ | S32-num/is-prime.t Remove trailing whitespace |
01:56 | ||
| roast: 0d78a135c7 | (Zoffix Znet)++ | S32-num/is-prime.t [6.d REVIEW] Include Num is-prime cases Orig: github.com/perl6/roast/commit/4c91...9e89061116 |
01:57 | ||
| roast: 66d29b7aa8 | (Zoffix Znet)++ | S32-temporal/DateTime.t [6.d REVIEW] Add couple more later/earlier leap sec tests Orig: github.com/perl6/roast/commit/5e60...da0415f591 |
02:11 | ||
| roast: ddd5420ba7 | (Zoffix Znet)++ | S02-types/WHICH.t [6.d REVIEW] Add IO::CatHandle to WHICH tests Orig: github.com/perl6/roast/commit/d042...698afc3dea |
02:37 | ||
| roast: a049fff21e | (Zoffix Znet)++ | S04-statements/while.t [6.d REVIEW] Don't stringify Seq being tested Orig: github.com/perl6/roast/commit/b777...707cb68b5b |
02:39 | ||
| roast: 1ecd6171da | (Zoffix Znet)++ | S04-statements/while.t [6.d REVIEW] Don't stringify Seq being tested Orig: github.com/perl6/roast/commit/a8bb...669d7f0481 |
02:41 | ||
| roast: 84a4645934 | (Zoffix Znet)++ | S03-operators/range-int.t [6.d REVIEW] Expand empty range sum tests Orig github.com/perl6/roast/commit/dffb...9f667f0f73 |
02:53 | ||
| roast: b939ce4a90 | (Zoffix Znet)++ | S32-temporal/Date.t Remove trailing whitespace |
03:01 | ||
| roast: 7b86b235a6 | (Zoffix Znet)++ | 2 files [6.d REVIEW] Do not stringify tested Arrays Orig: github.com/perl6/roast/commit/5e7d...528d127b5f |
|||
| roast: e3383ba4d0 | (Zoffix Znet)++ | 2 files [6.d REVIEW] Consolidate similar tests in one file Orig: github.com/perl6/roast/commit/bfd3...14b198fca2 github.com/perl6/roast/commit/b0c1...a34d10f8f8 |
03:21 | ||
| rakudo: 1f99aa145a | (Zoffix Znet)++ | t/spectest.data Remove S03-binding/nonsense.t from list of tests It was consolidated into another file: github.com/perl6/roast/commit/e3383ba4d0 |
03:22 | ||
| roast: 76b3925762 | (Zoffix Znet)++ | S32-num/power.t [6.d REVIEW] Reformat for clarity - Get rid of file-global confusingly-named var - Use clearer names for numeric constants - Use `is-deeply` to check types are right Partial orig: github.com/perl6/roast/commit/f884...8288f46482 |
03:46 | ||
| 6.d-prep: 480be92a8f | (Zoffix Znet)++ (committed using GitHub Web editor) | TODO/README.md Update Zoffix's 6.d reviewer commit mark |
03:48 | ||
| roast: 629b223fab | usev6++ | 5 files [jvm] Fudge new test failures Also replaces some tabs with whitespaces in S17-supply/syntax.t |
12:54 | ||
| jnthn | lizmat_: "there is B implicit locking"...B? ) | 13:14 | |
| lizmat_ | I think I wrote B<no> originally, then wordpress ate the <no> | 13:15 | |
| jnthn | ah | 13:16 | |
| lizmat++ | |||
| lizmat | jnthn++ # fixed | ||
| tbrowder | nqp question please: do *all* individual objects, including children of objects (e.g., array elements, hash keys and values) need to have a $*W.add_constant entry for types Str, Int, Num, etc.? | 13:18 | |
| jnthn | No | ||
| The serializer will walk them recursively | 13:19 | ||
| tbrowder | jnthn: thnx. how do i know when i need to do it? for instance, i’m expermenting with taking a config string representing a list or hash and ensuring bare numbers are treated as such rather than as a string as they initially arrive in Pod.nqp. | 13:21 | |
| so far that seems to be working but i’m trying to avoid double stringification (i.e., extra enclosing quotes when not needed) and wonder how serialization affects that. | 13:25 | ||
| jnthn | It just serializes object graphs with the types they have | 13:26 | |
| You only need to add_object something if it will be referred to by a QAST::WVal node really | |||
| (Which something like install_lexical_symbol will make) | 13:27 | ||
| tbrowder | so a constant str or num in a hash or list don’t need any special treatment as long as their types are known? | 13:29 | |
| e.g., my int $a := 3 | 13:31 | ||
| $a retains its int type even when put into a generic list or hash key’s value. | 13:33 | ||
| jnthn | A hash or list can't contain a native int anyway | ||
| It'll be automatically boxed | |||
| But yeah, the boxed object will then know its type | |||
| Be careful that in NQP code, you won't get things auto-boxed into Perl 6 types | |||
| Which is something else add_constant would do for you | 13:34 | ||
| Though looking up the Perl 6 Int type, for example, and then using nqp::box_i, would do it | |||
| tbrowder | hm, that’s not what i think i’m seeing. when creating a list from an array of strings of numbers they keep str unless i convert them by assigning to an int or num. | 13:35 | |
| ah, autoboxing? maybe that’s part of the problem! | |||
| so, watch out for unnecessary use of add_constant, then! | 13:38 | ||
| jnthn: got to leave till later, but helpful hints much appreciated. time to review yr nqp course again! | 13:41 | ||
| [Coke] | ]/win 2 | 16:47 | |
| samcv | /say 99 | 16:54 | |
| /win 99 | |||
| buggable | samcv, Thank you for entering Accidental /win Lottery! The next draw will happen in 1 week, 2 days, 7 hours, 5 minutes, and 51 seconds | ||
| AlexDaniel | uhhhhh | 20:01 | |
| did we just… lose an optimization | |||
| c: 2017.11,2017.12 6c: for 0..20 { $ = ~(+combinations($^row,$_) for 0..$row) }; say now - BEGIN now | 20:02 | ||
| committable6 | AlexDaniel, ¦2017.11,2017.12: «04===SORRY!04=== Error while compiling /tmp/pxV5Hb7PUAConfusedat /tmp/pxV5Hb7PUA:1------> 03608⏏04c: for 0..20 { $ = ~(+combinations($^row expecting any of: whitespace «exit code = 1»» | ||
| AlexDaniel | c: 2017.11,2017.12 for 0..20 { $ = ~(+combinations($^row,$_) for 0..$row) }; say now - BEGIN now | ||
| committable6 | AlexDaniel, ¦2017.11: «0.12636975» ¦2017.12: «8.28950117» | ||
| AlexDaniel | bisect: old=2017.11 for 0..20 { $ = ~(+combinations($^row,$_) for 0..$row) }; say 5 > (now - BEGIN now) | ||
| bisectable6 | AlexDaniel, Bisecting by output (old=2017.11 new=1f99aa1) because on both starting points the exit code is 0 | 20:03 | |
| AlexDaniel, bisect log: gist.github.com/0f7ea65bb208cca3d2...a53e6193cd | |||
| AlexDaniel, (2017-12-08) github.com/rakudo/rakudo/commit/af...5bd89e74eb | |||
| AlexDaniel | c: af9812fa73^ for 0..20 { $ = ~(+combinations($^row,$_) for 0..$row) }; say 5 > (now - BEGIN now) | 20:04 | |
| committable6 | AlexDaniel, ¦af9812fa73^: «True» | ||
| AlexDaniel | c: af9812fa73^ for 0..20 { $ = ~(+combinations($^row,$_) for 0..$row) }; say now - BEGIN now | ||
| committable6 | AlexDaniel, ¦af9812fa73^: «0.1250381» | ||
| AlexDaniel | c: af9812fa73 for 0..20 { $ = ~(+combinations($^row,$_) for 0..$row) }; say now - BEGIN now | ||
| committable6 | AlexDaniel, ¦af9812f: «8.65331035» | ||
| AlexDaniel | c: af9812fa73^ say combinations 50, 50 | 20:05 | |
| committable6 | AlexDaniel, ¦af9812fa73^: ««timed out after 10 seconds» «exit signal = SIGHUP (1)»» | ||
| AlexDaniel | c: af9812fa73^ say +combinations 50, 50 | 20:06 | |
| committable6 | AlexDaniel, ¦af9812fa73^: «1» | ||
| AlexDaniel | Filed it here: github.com/rakudo/rakudo/issues/1330 | 20:12 | |
| it should've been tested, really :) | |||
| timotimo | blame bdfoy for finding that count-only and bool-only are kind of broken in most places | 20:34 | |
| AlexDaniel | :D | 20:35 | |
| Zoffix | AlexDaniel: that's "by design" really | 20:37 | |
| I didn't look too hard for a way around it, but aeveral opts were removes in that commit because they were broken for partially consumed iterators | |||
| *several | 20:38 | ||
| AlexDaniel | that sucks! :) | ||
| Zoffix | Tho I thin in case of combinations simply sticking a counter that keeps track could be a fix. gonna look later tonight | ||
| b2gills | The original reason for count-only was so that you could get a count without iterating. The other way of getting a count would exhaust the Iterator. So it wasn't really intended that it would be used in addition with pull-one. | 20:44 | |
| Zoffix | ah hehe I even XXX TODOed the Combinations iterator and yeah I'll fix it tonight | ||
| b2gills: except there's no way to tell whether an iterator was ever pulled from and under the paradigm you describe it's unsafe to re-use an iterator you did not create, so you lose all potential benefits re-making iteratorss all over the place | 20:46 | ||
| at least with current count-only impl; it could be made to return some sentinel value, like IterationEnd | 20:47 | ||
| b2gills | It was added to speed up .elems, it was later used for more internal uses | ||
| I'm not sure if it should be extended, I'm also not sure it shouldn't | 20:49 | ||
| AlexDaniel | hmm where's the markdown→wordpress converter? | ||
| Zoffix | eco: advent | ||
| buggable | Zoffix, Found 3 results: Game::Crypt, Adventure::Engine, Acme::Advent::Highlighter. See modules.perl6.org/s/advent | ||
| Zoffix | eco: advent:;high | ||
| AlexDaniel | cool thanks! | ||
| buggable | Zoffix, Nothing found | ||
| Zoffix | eco: advent::high | ||
| buggable | Zoffix, Acme::Advent::Highlighter 'Preprocessor for Perl 6 Advent Articles': github.com/zoffixznet/perl6-Acme-A...ighlighter | ||
| b2gills | anyway I'm replacing my CPU cooler with one that doesn't have a broken fan, so will be off-line for however long that takes | 20:50 | |
| Zoffix | \o | ||
| b2gills | also gives me a chance to run an update | 20:51 | |
| That took longer than it should have. It would be nice if they would have taken into consideration the thickness of the protective pads on the brackets when designing the plastic clips that hold it in place. I ended up using screws and plastic washers that I made out of a margarine container. | 23:14 | ||
| At least it isn't vibrating from the missing fan blade on the CPU fan, and the case fans are so much quieter than the worn out ones they replace. | |||
| Geth | rakudo: 5eed2b0020 | (Zoffix Znet)++ | 4 files Re-implement Combinations iter's .[count|bool]-only Fixes github.com/rakudo/rakudo/issues/1330 |
23:43 | |
| roast: 9a1d5e7ed9 | (Zoffix Znet)++ | MISC/bug-coverage.t Fix off-by-one in a test |