[00:00:22] It fails when editing the test case or implementation too 🤷‍♂️ (re @u99of9: Z27171 passes, but when I copy the exact same input into "Try this function" it returns void) [00:19:25] David has just fixed the JS converter! 🎉 (re @Al: Yes, but an initial debug in Z25997 gives no output for tests like Z26852, just a Z24 and Z511. To me, that suggests that the fa...) [00:19:43] Thanks so much for reporting this. My bad. As suggested in messages above, the definition of isVoid() in the converter function needed fixing. I've done that and the errors for Z27072 are gone now. (Most of the tests are still failing but I assume that's because it's work in progress.) I apologize for the time spent struggling with this! I will also check the other [00:19:43] converter fun [00:19:44] ctions where I previously included isVoid(). (re @wikilinksbot: T401162 – Wikidata quantities with void bounds are not converted into code [open]) [00:26:21] I’m not sure about that. An initial debug in Z25997 still fails to emit in any case where a void is in the argument. Not saying the converter’s the problem but it’s a problem before the Python executes. On a positive note, setting an argument to None does emit as a Z21 in the (incorrect) Actual result. (re @u99of9: David has just fixed the JS converter! 🎉) [00:30:20] He fixed JS, not Py (re @Al: I’m not sure about that. An initial debug in Z25997 still fails to emit in any case where a void is in the argument. Not saying ...) [00:51:44] I'm working on the other relevant JS converter functions and then will fix the Python. Shouldn't take long. (re @u99of9: He fixed JS, not Py) [02:22:17] Well I did update the python converter Z25784. Z25997 had 3 failing tests, and 2 of them are now working. I'm not sure why Z26944 still isn't working. The error that comes back includes "name 'K1' is not defined". I'm not sure if that means there's still an issue in the converter or maybe something in the implementation code? I'm out of time for now; will check back [02:22:18] soon. [07:25:12] Thanks! I think I fixed the implementation so that test now passes. (re @David: Well I did update the python converter Z25784. Z25997 had 3 failing tests, and 2 of them are now working. I'm not sure why Z2...) [10:05:43] Z6821 currently isn't working sometimes (try for example fetching Q144) [10:14:19] Looks okay now. It needed a couple of decent aliases! (re @dvd_ccc27919: Z6821 currently isn't working sometimes (try for example fetching Q144)) [10:17:16] With Q30 still doesn't work (re @Al: Looks okay now. It needed a couple of decent aliases!) [10:21:10] Countries often don’t, I find. Just too much data, I guess. (re @dvd_ccc27919: With Q30 still doesn't work) [15:33:28] Very weird; I didn't know about this and I don't understand it at this point. I agree it's most likely size-related. Will raise it with the team. (re @Al: Countries often don’t, I find. Just too much data, I guess.) [15:38:20] Sorry, I thought everyone knew… please also see [[Talk:Z23753]] (re @David: Very weird; I didn't know about this and I don't understand it at this point. I agree it's most likely size-related. Will raise...) [22:33:48] I created https://phabricator.wikimedia.org/T401358