Geth | rakudo: vrurg++ created pull request #3689: Fix main-version.nqp dependencies |
00:03 | |||||||||||||||||||||||||||||||||||||
00:42
lucasb left
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo: b71cb3ae7c | (Vadim Belman)++ | 2 files Fix main-version.nqp dependencies Make it depend on the list of files used to build SHA digest. |
01:17 | |||||||||||||||||||||||||||||||||||||
rakudo: b5f23aa0d0 | (Vadim Belman)++ (committed using GitHub Web editor) | 2 files Merge pull request #3689 from vrurg/fix-main-version-deps Fix main-version.nqp dependencies |
|||||||||||||||||||||||||||||||||||||||
01:21
pamplemousse left
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo: c988f83291 | (Tom Browder)++ | 2 files Improve error message for too many directives in routine `sprintf` Fix GH issue #3682 The problem is the erroneous use of a '$' symbol in the sprintf format statement does not give a hint of that possibility in the original exception code. That, coupled with incomplete sprintf documentation, ... (8 more lines) |
02:06 | |||||||||||||||||||||||||||||||||||||
linkable6 | RAKUDO#3682 [open]: github.com/rakudo/rakudo/issues/3682 LTA error msg with improper use of '$' in format string | ||||||||||||||||||||||||||||||||||||||
rakudo: c75d25c570 | (Tom Browder)++ (committed using GitHub Web editor) | 2 files Merge pull request #3688 from tbrowder/sprintf-err Improve error message for too many directives in routine `sprintf` |
|||||||||||||||||||||||||||||||||||||||
04:32
sena_kun joined
04:37
sena_kun left
04:38
sena_kun joined
06:04
Altai-man_ joined
06:06
sena_kun left
07:18
jjmerelo joined
07:49
MasterDuke left
08:04
MasterDuke joined
08:05
sena_kun joined
08:07
Altai-man_ left
|
|||||||||||||||||||||||||||||||||||||||
sena_kun | 08:24 | ||||||||||||||||||||||||||||||||||||||
08:43
jjmerelo left
|
|||||||||||||||||||||||||||||||||||||||
[Tux] |
|
09:18 | |||||||||||||||||||||||||||||||||||||
lizmat | Files=1306, Tests=111281, 213 wallclock secs (28.57 usr 8.28 sys + 2994.20 cusr 271.76 csys = 3302.81 CPU) | 09:34 | |||||||||||||||||||||||||||||||||||||
09:45
squashable6 left,
squashable6 joined
10:04
Altai-man_ joined
10:07
sena_kun left,
MasterDuke left
10:14
patrickb joined
10:18
lichtkind joined
|
|||||||||||||||||||||||||||||||||||||||
nine | MasterDuke: I got a change log entry "When we're (int|num)ifying a list or hash..." in NQP. That's rather unhelpful. The commit subject should describe in very few words the user visible change (Fix blah, Speed up bleh, Add support for whatever...) | 10:53 | |||||||||||||||||||||||||||||||||||||
tellable6 | nine, I'll pass your message to MasterDuke | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 3ee1777a6b | (Elizabeth Mattijsen)++ | src/core.c/Str.pm6 Make "42".Int about 4.3x faster - by the magic of removing code - other types of values remain as fast / slow as they were |
10:54 | |||||||||||||||||||||||||||||||||||||
lizmat | grrr... the same change that yesterday made Inine::Perl5 fail because of version mismatch with Actions.nqp | 11:06 | |||||||||||||||||||||||||||||||||||||
now works like a charm, probabiy because there have been other changes | |||||||||||||||||||||||||||||||||||||||
so it's starting to feel to me some kind of boundary problem that isn't being tickled anymore | |||||||||||||||||||||||||||||||||||||||
nine | lizmat: vrurg fixed the build system, so changes in Actions.nqp no longer break installed modules | 11:09 | |||||||||||||||||||||||||||||||||||||
lizmat | aaah,, b71cb3ae7c79ea5e95fa | ||||||||||||||||||||||||||||||||||||||
linkable6 | (2020-05-12) github.com/rakudo/rakudo/commit/b71cb3ae7c Fix main-version.nqp dependencies | ||||||||||||||||||||||||||||||||||||||
lizmat | ok.... vrurg++ | 11:10 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: 37533ee66e | (Elizabeth Mattijsen)++ | src/Perl6/Actions.nqp Use the more idiomatic FLATTENABLE_LIST Because that's what it is for: to de-HLLize a HLL Array in NQP. |
11:12 | |||||||||||||||||||||||||||||||||||||
rakudo: cfc3e57f62 | (Donald Hunter)++ | src/core.c/Rakudo/Internals.pm6 Truncate .gist output for multidim arrays |
11:16 | ||||||||||||||||||||||||||||||||||||||
rakudo: 4bfe5bd5b7 | (Donald Hunter)++ | src/core.c/Rakudo/Internals.pm6 Clean up whitespace |
|||||||||||||||||||||||||||||||||||||||
rakudo: 1464310b1d | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | src/core.c/Rakudo/Internals.pm6 Merge pull request #3685 from donaldh/multidim-gist Truncate .gist output for multidim arrays |
|||||||||||||||||||||||||||||||||||||||
roast: f607f389ea | (Donald Hunter)++ | S09-multidim/methods.t Update .gist tests for multidim arrays This updates tests to match github.com/rakudo/rakudo/pull/3685 |
|||||||||||||||||||||||||||||||||||||||
roast: 708faddae3 | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | S09-multidim/methods.t Merge pull request #643 from donaldh/multidim-gist Update .gist tests for multidim arrays |
|||||||||||||||||||||||||||||||||||||||
rakudo: 8d5a0f7026 | (Tim Smith)++ | 2 files Fix .comb(Str) to ensure non-overlapping results It is expected that .comb() returns non-overlapping results, and .comb('aa') should yield the same results as .comb(/aa/). |
11:23 | ||||||||||||||||||||||||||||||||||||||
rakudo: a4ea2c78b4 | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 2 files Merge pull request #3668 from softmoth/comb-str Fix .comb(Str) to ensure non-overlapping results |
|||||||||||||||||||||||||||||||||||||||
roast: c9425490dd | (Tim Smith)++ | 3 files Test "aaaa".comb("aa") gives non-overlapping results |
11:24 | ||||||||||||||||||||||||||||||||||||||
roast: 1ca0b17c4f | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 3 files Merge pull request #639 from softmoth/comb-str Test "aaaa".comb("aa") gives non-overlapping results |
|||||||||||||||||||||||||||||||||||||||
11:25
MasterDuke joined
11:28
jjmerelo joined
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 3b4794f4ae | (Ben Davies)++ | src/core.c/Mu.pm6 Allow typechecking of any instance against Mu Smartmatching of Mu.new against Mu itself would throw before. This fixes that, but causes problems with smartmatching with junctions. |
11:36 | |||||||||||||||||||||||||||||||||||||
rakudo: 840d3e8101 | (Ben Davies)++ | 3 files Improve smartmatching with junctions The last commit breaks smartmatching for junctions. While the compiler could handle their threading when smartmatching, this would break any existing code that has ACCEPTS candidates for junctions. Giving it a default ACCEPTS candidate allows existing code to work again. While not strictly necessary for this to work, this also moves handling of junction smartmatching from Any to Mu. This allows `any(Mu, Any) ~~ Mu` to return `True` instead of `False`. |
|||||||||||||||||||||||||||||||||||||||
rakudo: f0594084e8 | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 3 files Merge pull request #3614 from Kaiepi/mu-any-junction-smartmatching Improve smartmatching against Mu/Any/Junction |
|||||||||||||||||||||||||||||||||||||||
11:49
patrickb left
11:54
patrickb joined
12:00
travis-ci joined
|
|||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build errored. Elizabeth Mattijsen 'Make "42".Int about 4.3x faster | 12:00 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/686518347 github.com/rakudo/rakudo/compare/c...e1777a6b78 | |||||||||||||||||||||||||||||||||||||||
12:00
travis-ci left
|
|||||||||||||||||||||||||||||||||||||||
lizmat | ^^ some transient connectivity failure, can't restart the job though :-( | 12:02 | |||||||||||||||||||||||||||||||||||||
12:05
sena_kun joined
12:07
Altai-man_ left
|
|||||||||||||||||||||||||||||||||||||||
patrickb | .tell rba Prebuilt releases of rakudo for 2020.05.1 are available for download from rooster.uber.space/patcloud/index....EJLQDZcjbf (as always). Can you upload to rakudo.org? Thank you! | 12:08 | |||||||||||||||||||||||||||||||||||||
tellable6 | patrickb, I'll pass your message to rba | ||||||||||||||||||||||||||||||||||||||
12:09
jjmerelo left
|
|||||||||||||||||||||||||||||||||||||||
patrickb | sena_kun: Sorry for messing up the release. And thank you for pulling 2020.05.1 off. I hope we'll manage to prevent such fallouts in the future. | 12:10 | |||||||||||||||||||||||||||||||||||||
tellable6 | 2020-05-09T18:23:17Z #raku-dev <El_Che> patrickb thanks for the heads-up. Weirdly enough my relocatable tar.gz for linux does not seem to be broken. | ||||||||||||||||||||||||||||||||||||||
2020-05-09T18:23:46Z #raku-dev <El_Che> patrickb: ~/Downloads/rakudo-2020.05/bin$ ./raku -v\nThis is Rakudo version 2020.05 built on MoarVM version 2020.05\nimplementing Raku 6.d. | |||||||||||||||||||||||||||||||||||||||
2020-05-09T18:27:09Z #raku-dev <El_Che> patrickb Oh, I found the github issue for the Windows problem. You're right, NativeCall seems broken on Linux too | |||||||||||||||||||||||||||||||||||||||
2020-05-10T15:00:52Z #raku-dev <El_Che> patrickb: The problem is fixed for relocatable linux as well. patrickb++ | |||||||||||||||||||||||||||||||||||||||
12:10
patrickb left
|
|||||||||||||||||||||||||||||||||||||||
sena_kun | patrickb++ # no problem as long as we can have more tests. :) | 12:11 | |||||||||||||||||||||||||||||||||||||
12:25
travis-ci joined
|
|||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build errored. Sylvain Colinet 'Make the 'is' routine in Test not throwing an error on failure when the type can't be stringified, falling back on .raku instead. | 12:25 | |||||||||||||||||||||||||||||||||||||
travis-ci.com/Skarsnik/rakudo/builds/165709125 github.com/Skarsnik/rakudo/compare...d3bc11fbee | |||||||||||||||||||||||||||||||||||||||
12:25
travis-ci left
|
|||||||||||||||||||||||||||||||||||||||
nine | Why would BEGIN time EVAL stuff cause us to throw serialization errors due to missing static frames for totally random seeming frames like { $obj.HOW.compose($obj, :$compiler_services) }? | 12:28 | |||||||||||||||||||||||||||||||||||||
s/frames/closures/ | |||||||||||||||||||||||||||||||||||||||
12:31
samcv left,
samcv joined
12:44
[TuxCM] left
12:53
travis-ci joined
|
|||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build errored. Elizabeth Mattijsen 'Use the more idiomatic FLATTENABLE_LIST | 12:53 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/686523244 github.com/rakudo/rakudo/compare/3...533ee66e65 | |||||||||||||||||||||||||||||||||||||||
12:53
travis-ci left
13:00
travis-ci joined
|
|||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build errored. Sylvain Colinet 'Make the 'is' routine in Test not throwing an error on failure when the type can't be stringified, falling back on .raku instead. | 13:00 | |||||||||||||||||||||||||||||||||||||
travis-ci.com/Skarsnik/rakudo/builds/165710315 github.com/Skarsnik/rakudo/compare...f8f4345866 | |||||||||||||||||||||||||||||||||||||||
13:00
travis-ci left
|
|||||||||||||||||||||||||||||||||||||||
lizmat | tyil: I think the module bot is awol again | 13:08 | |||||||||||||||||||||||||||||||||||||
or maybe I'm too fast :-) | |||||||||||||||||||||||||||||||||||||||
[Coke] | I saw an update for your module on #raku | 13:12 | |||||||||||||||||||||||||||||||||||||
lizmat | argh, looking in the wrong places again :-( | 13:22 | |||||||||||||||||||||||||||||||||||||
tyil: sorry for the noise | 13:23 | ||||||||||||||||||||||||||||||||||||||
tyil | np | 13:29 | |||||||||||||||||||||||||||||||||||||
13:42
pamplemousse joined
13:44
MasterDuke left
13:52
MasterDuke joined
14:04
Altai-man_ joined
14:07
sena_kun left
|
|||||||||||||||||||||||||||||||||||||||
Geth | nqp/new-disp: 031d8f9ff7 | (Jonathan Worthington)++ | src/vm/moar/QAST/QASTOperationsMAST.nqp Basic compilation of new dispatch operation |
14:12 | |||||||||||||||||||||||||||||||||||||
nqp/new-disp: 582cf22c86 | (Jonathan Worthington)++ | t/moar/53-dispatch.t An initial test for the new dispatchers Failing at time of commit. |
|||||||||||||||||||||||||||||||||||||||
nqp/new-disp: 69f9a65419 | (Jonathan Worthington)++ | 2 files Mark literal call arguments in callsites |
|||||||||||||||||||||||||||||||||||||||
15:06
lichtkind left
|
|||||||||||||||||||||||||||||||||||||||
gfldex | Is there a reason why IO::Path can tell me that a file is not 0 size but can't give me the actual size? | 15:47 | |||||||||||||||||||||||||||||||||||||
sorry my bad | 15:48 | ||||||||||||||||||||||||||||||||||||||
it's just hard to ctrl-f | |||||||||||||||||||||||||||||||||||||||
[Coke] | how to force a rebuild of --moar when building rakudo? (trying to use --moar-option and it's not even bothering because Moar is new enough) | 15:59 | |||||||||||||||||||||||||||||||||||||
MasterDuke | rm -rf nqp/MoarVM ? | 16:00 | |||||||||||||||||||||||||||||||||||||
[Coke] | ah, --force-rebuild. :) | 16:01 | |||||||||||||||||||||||||||||||||||||
nine | cd nqp/MoarVM ; make clean ; make install | ||||||||||||||||||||||||||||||||||||||
[Coke] | nine++ | ||||||||||||||||||||||||||||||||||||||
16:05
sena_kun joined
16:07
Altai-man_ left
16:22
camelCaser left,
ccamel joined
16:25
jjmerelo joined
16:58
guifa2_ joined
17:01
guifa2 left
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo/master: 4 commits pushed by (Ben Davies)++, (Elizabeth Mattijsen)++ | 17:21 | |||||||||||||||||||||||||||||||||||||
18:04
Altai-man_ joined
18:06
sena_kun left
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo: c3b1290668 | (Elizabeth Mattijsen)++ | 25 files Remove return constraint from .WHICH methods These methods are basically very simplistic, so it's clear that what is being expected, will actually be returned. Since for some applications this is hot code, this makes sense. For instance, this makes Str.WHICH about 2x as fast. |
18:19 | |||||||||||||||||||||||||||||||||||||
lizmat | :q | 18:22 | |||||||||||||||||||||||||||||||||||||
jnthn | Yes, but the point of return constraints is also introspection... | 18:25 | |||||||||||||||||||||||||||||||||||||
Each one you remove is also removing information seen by IDE users. | 18:26 | ||||||||||||||||||||||||||||||||||||||
Plus making the effects of my new-disp work look like less, 'cus it'd probably speed up these cases anyway :P | 18:27 | ||||||||||||||||||||||||||||||||||||||
18:28
pamplemousse left
18:30
travis-ci joined
|
|||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Elizabeth Mattijsen 'Merge pull request #3585 from Kaiepi/capture | 18:30 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/686675825 github.com/rakudo/rakudo/compare/f...6dee327d02 | |||||||||||||||||||||||||||||||||||||||
18:30
travis-ci left
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo: be36ec3822 | (Elizabeth Mattijsen)++ | 25 files Revert "Remove return constraint from .WHICH methods" Make sure IDE users have the return constraint introspection. It also looks like the GDO will ameliorate. |
18:32 | |||||||||||||||||||||||||||||||||||||
lizmat | jnthn ^^ | 18:37 | |||||||||||||||||||||||||||||||||||||
MasterDuke | i didn't realize return constraints were still so expensive. i remember that being the case a while ago, but for some reason thought that wasn't true anymore | 18:38 | |||||||||||||||||||||||||||||||||||||
jnthn | I didn't think it was either, fwiw...spesh often rips them out. I do wonder if it somehow isn't in this case. | 18:41 | |||||||||||||||||||||||||||||||||||||
lizmat | well, for very small methods it becomes visible, such as Str.WHICH | ||||||||||||||||||||||||||||||||||||||
which gets pretty hot for something like $str.comb.Bag | 18:42 | ||||||||||||||||||||||||||||||||||||||
jnthn | In new-disp, though, there will not be any invocation of bytecode at all for the return type check in the case it's identity. | ||||||||||||||||||||||||||||||||||||||
At the moment the spesh plugin invokes an identity function and then relies on it getting inlined to achieve better peak performance. | |||||||||||||||||||||||||||||||||||||||
lizmat | yeah, but this would not be identity | 18:43 | |||||||||||||||||||||||||||||||||||||
jnthn | ? | ||||||||||||||||||||||||||||||||||||||
Does it have to be decont'd? | |||||||||||||||||||||||||||||||||||||||
lizmat | Str.WHICH returns a ValueObjAt, not a Str | ||||||||||||||||||||||||||||||||||||||
jnthn | Ah, I should explain better | ||||||||||||||||||||||||||||||||||||||
The return type check spesh plugin isn't just checking, it's also decont'ing or recont'ing | 18:44 | ||||||||||||||||||||||||||||||||||||||
Or in the case we have here, just saying "ok, the value as it is will be fine" | |||||||||||||||||||||||||||||||||||||||
*that* is the identity bit | |||||||||||||||||||||||||||||||||||||||
lizmat | ok, indeed, no decont needed | 18:45 | |||||||||||||||||||||||||||||||||||||
nine | I.e. if it doesn't have to decont or recont and the return value has a known type that's matching, it won't bother to check? | ||||||||||||||||||||||||||||||||||||||
jnthn | nine: Today the spesh plugin sees it's not a container, and that the type matches. It makes a guard entry that is effectively: if it's ValueObjAt, then run -> \x { x } on the return value - e.g. identity | 18:47 | |||||||||||||||||||||||||||||||||||||
Spesh can then inline this, and it inlines to nothing. However, *until* inlining happens, we have to actually pay for an invoke of the identity function there. | |||||||||||||||||||||||||||||||||||||||
nine | Ah, ok, that makes perfect sense now | ||||||||||||||||||||||||||||||||||||||
jnthn | new-disp, however, is capable of more outcomes than "invoke bytecode", including "produce this value out of the set of input arguments" | 18:48 | |||||||||||||||||||||||||||||||||||||
Or at least, will be soon :) | |||||||||||||||||||||||||||||||||||||||
Thus the identity case doesn't need to run any bytecode at all. | |||||||||||||||||||||||||||||||||||||||
nine | Btw. music recommendation for deep and hard coding sessions (3:46 hours): www.youtube.com/watch?v=Q-vsx84hsbs | 18:49 | |||||||||||||||||||||||||||||||||||||
jnthn | It'll just say "ah, grab the first arg to the dispatch and set it as the result of the dispatch and we're done" | ||||||||||||||||||||||||||||||||||||||
nine | (was engaged in all my successes in the past two months) | ||||||||||||||||||||||||||||||||||||||
jnthn | Spesh itself will have an easier time of this too; it won't need to take the whole inlining path, it can instead just say "well, I'll turn the dispatch instruction into a set" | 18:50 | |||||||||||||||||||||||||||||||||||||
18:50
vrurg joined
|
|||||||||||||||||||||||||||||||||||||||
nine | When looking at the extops I've wondered whether new-disp may give a way to implement FIRST efficiently without the extop (by dispatching to a version running the FIRST code on the first run and a NULL code on subsequent ones) | 18:51 | |||||||||||||||||||||||||||||||||||||
MasterDuke | i don't really listen to music much nowadays, but i used to listen to the descent (don't remember if it was 1 or 2) soundtrack back when game cds would also work as music cds | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: dwarring++ created pull request #3690: expose IO::Handle.forget-about-closing($fd) method |
18:59 | |||||||||||||||||||||||||||||||||||||
19:15
travis-ci joined
|
|||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Elizabeth Mattijsen 'Remove return constraint from .WHICH methods | 19:15 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/686696707 github.com/rakudo/rakudo/compare/8...b129066882 | |||||||||||||||||||||||||||||||||||||||
19:15
travis-ci left
19:25
jjmerelo left
19:30
lichtkind joined
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo/master: 4 commits pushed by (Elizabeth Mattijsen)++ | 19:50 | |||||||||||||||||||||||||||||||||||||
20:05
sena_kun joined
20:06
Altai-man_ left
|
|||||||||||||||||||||||||||||||||||||||
nine | github.com/rakudo/rakudo/pull/3690 is a good example for why I'm sometimes reluctant to review a PR. Probably took me longer than the author needed for the PR to look at it, dig into the topic, find out why I don't like the solution and write up my reasoning all just to end up saying "no" to a contributor, which is never a good feeling. | 20:27 | |||||||||||||||||||||||||||||||||||||
lizmat | nine++ | 20:29 | |||||||||||||||||||||||||||||||||||||
nine | On the other hand it's vital that we do anyway. Because the only alternatives are getting a screwed up mishmash of an API (and having to break stuff when we want to fix it) or never gaining contributors. | ||||||||||||||||||||||||||||||||||||||
20:38
Altai-man_ joined
20:40
sena_kun left
20:41
MasterDuke left
21:52
ufobat__ joined
21:56
ufobat_ left
22:08
MasterDuke joined
22:11
Altai-man_ left
23:07
lichtkind left
|
|||||||||||||||||||||||||||||||||||||||
Geth | rakudo: d9defb6c08 | (Tom Browder)++ (committed using GitHub Web editor) | src/core.c/Exception.pm6 Tweak error message |
23:15 | |||||||||||||||||||||||||||||||||||||
23:32
gfldex joined
|