[01:14:53] <legoktm>	 JeroenDeDauw: I think you'll enjoy https://spindas.dreamwidth.org/2765.html
[02:17:12] <sDrewth>	 anyone able to give me some guidance to fix enWP's authority control https://en.wikipedia.org/wiki/Module:Authority_control so that it pulls the preferred WD field for VIAF, rather than the first encountered
[04:42:44] <sDrewthedoff>	 anyone able to give me some guidance to fix enWP's authority control https://en.wikipedia.org/wiki/Module:Authority_control so that it pulls the preferred WD field for VIAF, rather than the first encountered
[07:08:37] <Amir1>	 I reverted 10 vandalisms today :)
[10:35:10] <JeroenDeDauw>	 legoktm: awezome!
[13:14:58] <sDrewthedoff>	 anyone able to give me some guidance to fix enWP's authority control https://en.wikipedia.org/wiki/Module:Authority_control so that it pulls the preferred WD field for VIAF, rather than the first encountered
[13:18:43] <hoo>	 sDrewthedoff: Still here?
[13:25:15] <multichill>	 hoo: Did you notice https://www.wikidata.org/wiki/User:Addshore/Identifiers/0#good_to_convert got a tiny bit longer? :-)
[13:31:24] <hoo>	 multichill: I did... I'll probably do another batch today
[13:31:43] <hoo>	 I did a few on Friday also
[13:43:17] <multichill>	 Maybe time to create a /done subpage to get the pages a bit smaller?
[13:44:03] <hoo>	 or one of these collapsible boxes?
[13:44:18] <multichill>	 That works too
[13:44:29] <multichill>	 Yeah, let's do that. We can always move it when the page gets really large
[13:47:39] <sDrewthedoff>	 hoo: still here
[13:47:54] <hoo>	 sDrewthedoff: Ah, great
[13:48:04] <sDrewthedoff>	 though not for a long time. nearly 1am
[13:48:16] <hoo>	 in getIdsFromWikidata
[13:48:34] <sDrewthedoff>	 let me pull up the module
[13:49:02] <hoo>	 instead of doing pairs( item.claims[property] ) you should do pairs( item:getBestStatements( property ) )
[13:49:17] <hoo>	 assuming item is a mw.wikibase.entity metatable
[13:49:22] <hoo>	 (didn't test)
[13:49:43] <hoo>	 but that should give you the list of just the best statements (highest rank only, no deprecated statements)
[13:50:25] <multichill>	 hoo: Any idea how to handle imdb? It has multiple formatter url's
[13:50:46] <multichill>	 (see https://www.wikidata.org/wiki/User:Addshore/Identifiers/0 )
[13:50:47] <hoo>	 ugh... and we only have on property?
[13:51:07] <hoo>	 I see
[13:51:13] <multichill>	 https://www.wikidata.org/wiki/Property:P345
[13:51:30] <multichill>	 Maybe we should just chop it up into smaller properties
[13:51:54] <hoo>	 Well, I think we have three options here: Build a tool on labs and use that as a target (yikes), use a gadget (yikes ^ 2) or make formatting on the Wikibase side smarter
[13:52:03] <hoo>	 or that, of course
[13:52:30] <multichill>	 Added advantage is the ability to set proper constraints
[13:52:59] <hoo>	 I've always said: Make properties small in scope
[13:53:11] <multichill>	 I completely agree
[13:53:31] <hoo>	 I'm happy people finally started splitting up website username on
[13:53:33] <multichill>	 This is suboptimization 
[13:53:45] <sDrewthedoff>	 hoo: above you changed from square brackets to  parentheses around "property" was that intentional?
[13:54:14] <hoo>	 sDrewthedoff: Yes, item.claims is a plain table item:getBestStatements is a function that you call
[13:54:26] <sDrewthedoff>	 k
[13:54:30] <hoo>	 see https://www.mediawiki.org/wiki/Extension:Wikibase_Client/Lua#mw.wikibase.entity:getBestStatements
[13:58:17] <sDrewthedoff>	 hoo: SO THIS ... https://en.wikipedia.org/w/index.php?title=Module:Authority_control/sandbox&diff=707373799&oldid=703166546
[13:58:43] <sDrewthedoff>	 ?
[13:58:56] <hoo>	 almost
[13:58:57] <hoo>	 no pairs in the if
[13:59:04] <sDrewthedoff>	 k
[13:59:12] <hoo>	 although it might not do any harm
[13:59:56] <sDrewthedoff>	 returned anyway
[14:00:36] <hoo>	 hm?
[14:01:53] <hoo>	 Does it work as expected now? I'm not sure I'm following
[14:01:57] <nikki>	 multichill: I would support splitting it, it's one of the only properties where we combine multiple types of identifier into a single property
[14:02:27] <sDrewthedoff>	 hoo https://en.wikipedia.org/w/index.php?title=Module%3AAuthority_control%2Fsandbox&type=revision&diff=707374002&oldid=703166546
[14:02:49] <sDrewthedoff>	 and the testcase there sucks!
[14:03:45] <multichill>	 hoo: nikki: Please comment at https://www.wikidata.org/wiki/Property_talk:P345#Split_this_property_up
[14:03:45] <hoo>	 does it even have test cases?
[14:04:32] <sDrewthedoff>	 something sucky as a subpage. I will roll it into place, check and revert if necessary
[14:04:34] <hoo>	 thanks for bringing that up, will comment
[14:09:12] <sDrewthedoff>	 hoo, works :-)
[14:09:18] <sDrewthedoff>	 thx!
[14:09:21] <hoo>	 Awesome :)
[14:11:18] <multichill>	 sDrewthedoff: You might want to apply the same logic to the other authority control properties too
[14:14:43] <sDrewthedoff>	 multichill: won't that do it to any wikidata id, not just VIAF?
[14:14:49] <sDrewthedoff>	 that change ^
[14:15:13] <multichill>	 Oh right, I didn't see you posted a diff
[14:15:17] <multichill>	 Sure looks like it :-)
[14:15:44] <sDrewthedoff>	 I managed to work out that there was no particular VIAF component where hoo said to change
[14:16:04] <sDrewthedoff>	 Useless at coding I may be, just not completely clueless
[14:16:22] <sDrewthedoff>	 close to completely, just a notch above
[14:16:29] <multichill>	 Oh, fun, hardcoded url's
[14:16:37] <multichill>	 Maybe time to grab those from Wikidata?
[14:17:18] <sDrewthedoff>	 One would think so
[14:17:49] <multichill>	 Seems to be about 50% of the code
[14:17:54] <hoo>	 also no url encoding... that can end up "funny"
[14:18:13] <hoo>	 (not scary though... it's still wikitext, so nothing to worry about)
[14:18:15] <sDrewthedoff>	 if you make changes to the sandbox, I am happy to move them over to the module
[14:19:18] <sDrewthedoff>	 this sort of code so needs to be universal, rather than so many xwiki copies
[14:19:36] <sDrewthedoff>	 one day
[14:20:03] * hoo still dreams of global modules
[14:23:20] <multichill>	 Yeah, global modules.....
[14:25:19] <sDrewthedoff>	 would be magic for the stats and graphs packages that Yuril is building.  Even the ability to have them situated at one wiki and sync'd
[14:25:29] <sDrewthedoff>	 Yuri
[14:28:16] <nikki>	 I've wondered for a while why there's nothing like commons for templates and modules... it seems like there are a lot of things just being copied and pasted between wikis
[14:31:18] <nikki>	 and it's more of a problem for smaller wikis where there are fewer people to maintain everything, so the templates and things they're used to on larger wikis won't be available (unless they copy them over themselves)
[14:32:20] <sDrewthedoff>	 especially when we have Modules for WD
[14:33:34] <sDrewthedoff>	 or the ability to install them xwiki like we do with gadgets
[14:35:44] <hoo>	 multichill: https://www.wikidata.org/wiki/User:Addshore/Identifiers/2#good_to_convert
[14:35:58] <hoo>	 think that's ok
[14:36:47] <multichill>	 Looks good!
[14:49:08] <sjoerddebruin>	 hoo: keep up the good work <3
[14:52:59] <hoo>	 :)
[15:11:32] <multichill>	 hoo: Can we figure out which templates use P345 based on usage? 
[15:11:59] <hoo>	 Hardly... no
[15:12:12] <hoo>	 We don't even do usage per property id (which we probably should)
[15:12:58] <multichill>	 hmm
[15:13:34] <multichill>	 So when someone vandalises a property we have no way of knowing which pages are affected?
[15:13:48] <hoo>	 essentially, yes
[15:13:50] <multichill>	 And how are pages being purged after a property update?
[15:14:02] <hoo>	 Not at all
[15:14:21] <hoo>	 only if the page updates or in case the property itself is being used
[15:14:36] <multichill>	 So when the formatter url gets changed (bbc your artists for example), how do we spread that change?
[15:14:43] <hoo>	 but just using the data (and upcoming derived data) doesn't make for a usage entry
[15:15:18] <hoo>	 wait for it :/
[15:15:26] <multichill>	 haha
[15:16:16] <hoo>	 multichill: Modules mentioning P345 https://phabricator.wikimedia.org/P2682
[15:17:02] <multichill>	 Mostly cycling race (?)
[15:17:15] <hoo>	 seems so
[15:18:05] <multichill>	 https://en.wikipedia.org/wiki/Template:IMDb_title ?
[15:18:19] <hoo>	 ugh
[15:18:19] <multichill>	 Why is that one not in the list?
[15:18:24] <hoo>	 the parser function
[15:18:35] <hoo>	 Because it's a template, not a module
[15:18:43] <hoo>	 also it shouldn't use the parser function
[15:18:52] <hoo>	 the function is supposed to return a URL at some point
[15:19:01] <hoo>	 (instead of the plain string)
[15:19:20] <multichill>	 It's possible, so people use it :P
[15:20:07] <hoo>	 true
[15:20:20] <hoo>	 also things like [[{{#property:P…}}]]
[15:20:20] <AsimovBot>	 Caracteres inválidos en el enlace «{{#property:P…}}»; no están permitidos: #<>[]|{}
[15:22:19] <multichill>	 We have it all over the place. One of the reasons is the lack of LUA adoption
[15:22:38] <multichill>	 If you really want to solve this, increase the LUA adoption rate
[15:23:25] <hoo>	 I'd love for that to happen
[15:26:30] <hoo>	 multichill: https://meta.wikimedia.org/wiki/Wikidata/Notes/Inclusion_syntax_v0.3 Ideas from a pre-Lua world…
[15:27:13] <multichill>	 :-)
[15:30:53] <HakanIST>	 is there any example for applying  properties on graph template?
[15:31:17] <hoo>	 There were some, but not sure where anymore
[15:31:29] <hoo>	 I think Jura1 had them in their user space somewhere
[16:37:02] <hoo>	 We have 325 external identifiers now!
[16:37:16] <sjoerddebruin>	 damn
[16:37:21] <hoo>	 hm?
[16:42:02] <multichill>	 hoo: Can you do P380 too?
[16:42:53] <multichill>	 It's an external identifier, we just made a mess of it on Wikidata :P
[16:43:50] <sjoerddebruin>	 hope ISNI reaches consensus, only one left on most items I visit
[16:49:21] <multichill>	 sjoerddebruin: It seems to have  4552 items with multiple links
[16:51:23] <hoo>	 multichill: hm... I sensed there might be an ongoing discussion there
[16:57:44] <hoo>	 multichill: Did it now
[18:07:33] <nikki>	 does anyone have any examples of selecting references in sparql?
[18:15:06] <sjoerddebruin>	 nikki: nothing between https://www.wikidata.org/wiki/Wikidata:SPARQL_query_service/queries ?
[18:16:15] <nikki>	 meh, unclickable tinyurl things... why do people do that :/
[18:17:34] <sjoerddebruin>	 url-shorters are blocked I think
[18:18:17] <sjoerddebruin>	 (all examples should be inside WDQS imo)
[18:18:20] <nikki>	 nobody makes people use url shorteners though, and I would argue that the query is actually interesting on a page of example queries
[18:19:04] <nikki>	 yeah, I was also wondering why we have a second page for examples (I didn't find anything relevant on the page linked from query.wikidata.org itself)
[18:29:04] <nikki>	 anyway thanks, I managed to find one eventually on there
[18:32:19] <sjoerddebruin>	 :)
[18:47:58] <sjoerddebruin>	 Meh. https://www.wikidata.org/w/index.php?title=Q2368598&type=revision&diff=308214630&oldid=299449934