00:02 reportable6 joined 00:36 MasterDuke joined 03:18 MasterDuke left 06:00 reportable6 left 06:02 reportable6 joined 08:10 sena_kun joined 08:11 finanalyst joined
ab5tract [Coke]: why not? (genuine question as I don't know whether its a complicated process) 08:17
lizmat .tell lizmat hello 08:54
tellable6 lizmat, I'll pass that message to your doctor
tellable6_ lizmat, I'll pass that message to your doctor
lizmat hmmm
08:55 ChanServ sets mode: +o lizmat, ChanServ sets mode: -o lizmat 09:18 sena_kun left
finanalyst .tell [Coke] I've answered your question about PR #364. Hoping for a review. 09:50
tellable6 finanalyst, I'll pass your message to [Coke]
09:55 finanalyst left
patrickb jdv: Release build is done. 10:03
ab5tract thanks patrickb++! 11:53
12:00 reportable6 left
ab5tract m: use nqp; my $Block := nqp::unipropcode("Block"); say "with space: " ~ nqp::unipvalcode($Block, "Basic Latin"); say "w/o space: " ~ nqp::unipvalcode($Block, "Basic_Latin"); 12:02
camelia with space: 0
w/o space: 1
12:03 reportable6 joined
ab5tract it's interesting that the nqp version of unipvalcode has signature (int, str) but the MVM one has (int, int) 12:03
m: use nqp; my $a = nqp::unipropcode("Basic Latin"), my $b = nqp::unipropcode("Basic_Latin"), $a == $b ==> say() 12:06
camelia (0 6 False)
ab5tract I actually have no idea how the spaced-version is working :)
lizmat m: my int $a; my str $b = "42"; $a = $b; say $a 12:07
camelia 42
lizmat if the native string coerces ok, it is automagically converted
only works with natives
m: sub a(int $a) { dd $a }; a my str $ = "42" 12:08
camelia ===SORRY!=== Error while compiling <tmp>
Calling a(str) will never work with declared signature (int $a)
at <tmp>:1
------> sub a(int $a) { dd $a }; āa my str $ = "42"
lizmat m: sub a(int $a) { dd $a }; a(my str $ = "42")
camelia ===SORRY!=== Error while compiling <tmp>
Calling a(str) will never work with declared signature (int $a)
at <tmp>:1
------> sub a(int $a) { dd $a }; āa(my str $ = "42")
lizmat pretty sure with optimization off that would work :-) 12:09
ab5tract ah, sorry, I meant I don't understand how "a" ~~ /<:Block("Basic Latin")>/ is working. `DELEGATE-ACCEPTS` gets "Basic Latin" as the property name and the target, but I can't see how the MAST operations are returning "Basic Latin" instead of "Basic_Latin" 12:15
it's actually irrelevant in the long run, just a bit of confusion that my mind would rather see resolved :) 12:16
Geth rakudo/r5486: d27303a6b8 | ab5tract++ | 2 files
Add name check and exception to General_Category uasge

Add a check that we should actually be bothering with General_Category lookups at all.
By doing so, we are able to add a targeted exception in the case where an unknown Unicode General_Category ... (5 more lines)
12:18
ab5tract nine & lizmat, let me know if you still approve of R#5555 and N#820 -- I've added exceptions for when unknown categories and category/property pairs are provided. 12:23
linkable6 R#5555 [open]: github.com/rakudo/rakudo/pull/5555 Add lookup for translating General_Category longnames
N#820 [open]: github.com/Raku/nqp/pull/820 Add lookup for translating General_Category longnames
12:24 linkable6_ left 12:26 linkable6_ joined
ab5tract dang, I thought I had t/12-rakuast/regex-assertion.rakutest fixed.. hold off on review for now 12:27
lizmat too late :-) 12:28
ab5tract :)
but yeah, basically what seems weird to me is that `nqp::unipvalcode(nqp::unipropcode($category), $constraint-property)` doesn't work for Block and "Basic Latin" 12:29
lizmat is this in Raku or NQP ? 12:30
ab5tract I could solve this with a .subst call on $constraint-property, but wasn't sure how heavy that would be 12:31
this call is coming from Raku
lizmat so what is the failing case? 12:32
ab5tract Subtest: Character property with expression assertion works \n # / <+:Block("Basic Latin")>/ 12:34
but.. from the CLI this works: `set-rakuast; r ' "As" ~~ / <+:Block("Basic Latin")> / ==> say()'` 12:35
šŸ™ƒ
ah, I think I see what's going wrong 12:37
from some debug: :constraint-property("Basic Latin") \n :target("Supplemental Symbols and Pictographs") 12:38
hmm, but how to fix it.. 12:43
\o/ 12:53
but I did have to resort to .subst(' ', '_', :g) 12:56
except something is still up.. new tests failing that hadn't failed before 12:59
Okay, one more chance and then I'm abandoning this as an exception site. It would probably be cleaner to do this in RakuAST anyway... 13:09
I think this whole section of regex's could use a revisit 13:15
TIL that 'bc' is a valid "category" 13:16
Geth rakudo/r5486: 6a7171d565 | ab5tract++ | src/core.c/Match.rakumod
Add lookup for translating General_Category longnames

Also fix broken shortnames "L", "M", "N", "P", "S", "Z", and "C".
This fixes #5486 and #5372.
The problem this addresses is three-fold: ... (23 more lines)
13:18
ab5tract back to basics :) 13:19
okay, I'm flummoxed now.. I know I ran full spectest successfully with the older proposals 13:23
will have to investigate further later on 13:24
that's what I get for forgetting to `rm ./install/bin/nqp*` šŸ˜­ 13:28
*phew* 13:31
16:59 sena_kun joined 18:00 reportable6 left, reportable6 joined
ab5tract m: "e".uniprop("Block").say; "e".uniprop("Script").say; "e".uniprop("General_Category").say; "e".uniprop("Uppercase_Letter").say 18:16
camelia Basic Latin
Latin
Ll
Ll
ab5tract I feel like the uni* methods must be some of the earliest pieces of Rakudo... even the method names are a bit more archaic than usual (though`.unicode-property` does feel a bit wordy) 18:19
[Coke] m: "e".uniprop("Random Stuff").say;
camelia 0
tellable6 2024-04-26T09:50:38Z #raku-dev <finanalyst> [Coke] I've answered your question about PR #364. Hoping for a review.
ab5tract m: "e".uniprop("Basic Latin").say 18:20
camelia 0
ab5tract note that it doesn't even roundtrip there
[Coke] Is 0 the right response? Not clear from docs.raku.org/type/Cool#routine_uniprop 18:21
ab5tract I would argue that an exception or failure would be more appropriate 18:23
I think I'll craft a problem-solving ticket for this $sometime 18:25
[Coke] could you write up a sentence as to what cases would return a 0 here for the docs? I'll edit it and add it in 18:33
ab5tract Sure, will get back to you on that 18:44
[Coke] also just found m Stuff").say; 18:45
14:19 < camelia> rakudo-moar 735fc4f3
linkable6 (2024-04-25) github.com/rakudo/rakudo/commit/735fc4f3d1 Merge pull request #5556 from rakudo/release-2024.04
[Coke] er, github.com/Raku/doc/issues/4449
18:46 linkable6_ left 18:48 linkable6_ joined
ab5tract I wonder how muc 18:57
*I wonder how much community fallout there would be from changing 0 to Nil here.. 18:58
I would imagine that there would be very little working code out there that depends on looking up a nonexistent property 19:02
jdv community or ecosystem? 19:09
ab5tract ecosystem 19:11
related, it would be an amazing thing to rework blin to be able to run on a branch commit 19:13
jdv idk if it cant. you can spec a "new" sha iirc 19:15
ab5tract by sending an incantation to buildable? 19:16
my understanding from the code was that could only do what's been buildable'd 19:17
but maybe (probably) I missed something..
jdv i meant blin. itd have to be on gh so a build could be fetched 19:18
yeah.
youre probably right 19:19
it would be cool
i guess the first hurdle would be getting that build in the set on the "mothership" 19:21
one issue is its all run by alexdaniel so its not easy to change 19:22
ab5tract indeed... 19:36
my thought was maybe it could be refactored to take either a capital-B Build as usual or to work with a local path 19:38
jdv the sha would have to in the git tree it looks to 19:40
right now thats gh
point both local maybe 19:41
thats if you want bisecting
ab5tract bisecting is a great feature but for testing a branch just simple verification was the goal I had in mind 19:42
unless its a monster of a branch, I wouldn't expect bisecting to be too important 19:43
jdv yeah
Geth rakudo/main: aee3d2599a | ab5tract++ | src/core.c/unicodey.rakumod
Return '' instead of 0 when a uniprop is not found

The previous behavior seems to be an unintended consequence of setting the "pref" to 'I' when no uniprop is found in codename2proppref.
Using '' here as that is what is returned in the bottom of the nested if's.
20:06
ab5tract ^^ pushed this directly to main so that it can be tested with blin. if there's any issue, I will of course revert it 20:07
m: "e".uniprop("Basic Latin").say 20:13
camelia Nil
ab5tract buildable is fast :) 20:14
Geth rakudo/main: 05dcd08151 | ab5tract++ | src/core.c/unicodey.rakumod
Actually return '' (not Nil) from uniprop
20:20
ab5tract Dang: `WARNING: image platform (linux/amd64) does not match the expected platform (linux/arm64)` 20:44
nine Looks like my grant proposal is up for comments: news.perlfoundation.org/post/sseif...as-apr2024
I'd appreciate feedback :) The committee is voting next weekend
[Coke] nine++ 20:47
# commented
lizmat nine++ .oO( develoer ?? ) 20:49
[Coke] ... they never took away my edit rights, I could fix that typo. :| 20:54
lizmat hehe :-) 20:56
[Coke] .tell Util - hey, can you let Saif know there's a typo "develoer" in news.perlfoundation.org/post/sseif...s-apr2024)
tellable6 [Coke], I'll pass your message to Util
20:57 MasterDuke joined
MasterDuke ab5tract, jdv: i think blin can do branches, assuming they're pushed to the rakudo repo 21:01
since for a while now the whateverable build process builds branch commits also 21:02
ab5tract ah, that should indeed enable it 21:03
MasterDuke heh, "interpreteter" and "interpretter" in the grant proposal 21:05
21:29 sena_kun left
ab5tract nine++ , I really hope you get this 22:27