Module talk:GetShortDescription

subst

Hi there, just noticed a bit of oddness when using subst:

{{#invoke:GetShortDescription|main |name=Brazil |only=explicit}} → table
{{#invoke:GetShortDescription|main |name=Brazil}} → table
{{subst:#invoke:GetShortDescription|main |name=Brazil |only=explicit}} →
{{subst:#invoke:GetShortDescription|main |name=Brazil}} → country in South America

Olivaw-Daneel (talk) 10:19, 23 January 2023 (UTC)[reply]

Huh. Looks like the last one is getting the wikidata, which means the penultimate, requesting only the explicit, is not getting the wikidata as a fallback and is thus empty. Sorry this came up now; I'm genuinely on my last legs for the day. I will attack it after some sleep. The explicit request requires preprocessing to handle weirdly configured (gotta account for weird stuff or it falls apart when it meets it) short desc templates, because lua's pattern matching is no match for grown-up regex. My guess is that the preprocessing is being bullied during the substitution. Possibly a frame problem; frames are weird. I'll figure it out ASAP. Fred Gandt · talk · contribs 10:57, 23 January 2023 (UTC)[reply]

Yeah; almost certainly a frame issue; via template it's fine:

{{subst:User:Fred Gandt/sandbox/Get short description |name=Brazil |only=explicit }} → table

Fred Gandt · talk · contribs 11:06, 23 January 2023 (UTC)[reply]

oh I derp; of course it just dumped the invoke on the page and left; I really need to sleep Fred Gandt · talk · contribs 11:09, 23 January 2023 (UTC)[reply]

The substitution happens before the preprocessing, so the preprocessing never happens. This leaves the found short desc template in its raw state and the pattern matching for the content fails, so we get the empty string for explicit, and the wikidata fallback otherwise. I will have to manually process the short desc template. This isn't so bad. But to put into perspective why I am not doing it right now; I just past 25 hours awake and it really feels like it. See ya tomorrow 😜 Fred Gandt · talk · contribs 13:04, 23 January 2023 (UTC)[reply]

Thanks, no hurry! Olivaw-Daneel (talk) 17:07, 23 January 2023 (UTC)[reply]

Btw, there's really no rush wrt to the GAN usage; don't want to hurry your development time. For {{GAN}} I was thinking of starting with the code used in {{Annotated link}}, as I've already gotten that to work, and it seems good enough to start with. We can later switch to your module version when it's ready; perhaps in sync with when Template_talk:Annotated_link#Module concludes. How does that sound to you? Olivaw-Daneel (talk) 04:29, 24 January 2023 (UTC)[reply]

Absolutely great if that works for you 😊 This all started to replace the template code of {{annotated link}} which is already horrible to maintain and will only get worse, so eventually that template will being using this module anyway 😁 Substituting is always a pain; the existence of various kludges to get around the issue is proof of it; safesubst, subst and all kinds of weird breaking of inclusion tags... The removal of preprocessing from the module will improve performance anyway, so it's good to do even if not for fixing the substitution issue. Fred Gandt · talk · contribs 04:44, 24 January 2023 (UTC)[reply]

Fixed in my test Module; needs more testing before pushing

{{subst:#invoke:User:Fred Gandt|main |name=Brazil |only=explicit}} → Country in South America
{{subst:#invoke:User:Fred Gandt|main |name=Brazil}} → Country in South America

Fred Gandt · talk · contribs 09:26, 24 January 2023 (UTC)[reply]

Pushed to release; nice little performance boost too:

Old

NewPP limit report
Parsed by mw1351
Cached time: 20230124093803
Cache expiry: 1814400
Reduced expiry: false
Complications: [vary‐revision‐sha1, vary‐page‐id, show‐toc]
CPU time usage: 0.276 seconds
Real time usage: 0.339 seconds
Preprocessor visited node count: 566/1000000
Post‐expand include size: 20103/2097152 bytes
Template argument size: 744/2097152 bytes
Highest expansion depth: 14/100
Expensive parser function count: 7/500
Unstrip recursion depth: 0/20
Unstrip post‐expand size: 4974/5000000 bytes
Lua time usage: 0.208/10.000 seconds
Lua memory usage: 15322087/52428800 bytes
Number of Wikibase entities loaded: 0/400

New
NewPP limit report
Parsed by mw1478
Cached time: 20230124094020
Cache expiry: 1814400
Reduced expiry: false
Complications: [vary‐revision‐sha1, vary‐page‐id, show‐toc]
CPU time usage: 0.184 seconds
Real time usage: 0.237 seconds
Preprocessor visited node count: 566/1000000
Post‐expand include size: 20103/2097152 bytes
Template argument size: 744/2097152 bytes
Highest expansion depth: 14/100
Expensive parser function count: 7/500
Unstrip recursion depth: 0/20
Unstrip post‐expand size: 4974/5000000 bytes
Lua time usage: 0.138/10.000 seconds
Lua memory usage: 15322564/52428800 bytes

Number of Wikibase entities loaded: 0/400

Fred Gandt · talk · contribs 09:44, 24 January 2023 (UTC)[reply]

For the record: there's still a problem; it's just a different problem. Fred Gandt · talk · contribs 12:50, 24 January 2023 (UTC)[reply]

I think I fixed it; it's plugged into Module:Annotated link and the results are pretty nifty:

Code: {{#invoke:AnnotatedLink|main |name=The Partisan |quote=yes |only=wikidata |desc_first_letter_case=upper |wedge=from the album ''[[Songs from a Room]]'' |aka=La Complainte du partisan |aka_lang=fr }}

Result: Script error: No such module "AnnotatedLink".

And yes, it works if it's substed 😉 Fred Gandt · talk · contribs 21:28, 24 January 2023 (UTC)[reply]

Alright, now testing Oceania, where the WP shortdesc is "Geographical region in the Pacific Ocean":

{{#invoke:GetShortDescription|main |name=Oceania |only=explicit}} → table
{{#invoke:GetShortDescription|main |name=Oceania}} → table
{{subst:#invoke:GetShortDescription|main |name=Oceania |only=explicit}} →
{{subst:#invoke:GetShortDescription|main |name=Oceania}} → geographic region that includes Australasia, Melanesia, Micronesia and Polynesia

Olivaw-Daneel (talk) 07:29, 26 January 2023 (UTC)[reply]

I wonder when I broke it? Poor timing; I have to go out for a bit; I'll get right on it when I get home. Fred Gandt · talk · contribs 07:42, 26 January 2023 (UTC)[reply]

Fixed. I did check it was working, but obviously wasn't paying close enough attention. Fred Gandt · talk · contribs 10:30, 26 January 2023 (UTC)[reply]