[03:54:37] testing bug [12:40:27] Hello I am back again and want to start contribute to "RevisionSlider" can I get any preffered tasks to start? [13:08:19] Gopa_: hi! I think addshore has some [13:09:28] Gopa_: HI! [13:09:29] yes :) [13:17:46] Gopa_: where are you? (: [13:53:15] Hello [14:00:21] I wonder strongly about https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/412822/ [14:01:52] the recommendation is now `require( 'mediawiki.String' ).byteLength` [14:03:13] but I found no documentation about the global require() function (except for Lua and PHP) [14:04:46] Hi Gopa_ ! :-) [14:08:17] seems a shortcut for mw.loader.require [14:31:11] Technical Advice IRC meeting *Wikimania Special* starting in 30 minutes in channel #wikimedia-tech, hosts: @addshore & @CFisch_WMDE - all questions welcome, more infos: https://www.mediawiki.org/wiki/Technical_Advice_IRC_Meeting [14:31:44] :D [14:31:55] :) [14:32:12] \o/ [14:33:04] Sadly it seems freenode has some technical issues right now :/ I hope noone is excluded from todays TAIM because of that� [14:52:24] Silly freenode [14:53:36] Are the people at wikimania running into a Freenode rate limit (first phab, then freenode)? [14:54:09] Good question... [14:55:49] Oooh, maybe not! [14:55:59] Or maybe, I haven't seen anyone complain yet [15:00:27] \o/ [15:00:39] It's time! [15:01:12] Welcome to the Technical Advice IRC Meeting! [15:01:41] CFisch_WMDE: Hello ! [15:01:57] Wikimania Special Edition! [15:02:19] Live from Cape Town [15:02:21] and Berlin [15:02:22] addshore: Sorry for the Inconvenience and late reply, I went out for food. [15:02:50] Gopa_: no problem :-) [15:03:06] CFisch_WMDE: Thanks [15:03:28] o/ [15:03:53] addshore: More over I am from India. [15:04:49] Yay! Shall I start with a question? I've been wondering how you should hide html headers (h2 etc) intended for screenreaders. Looking at Vector, there are three ways it appears to be done [15:05:18] Hi mainframe98 [15:05:25] Hi mainframe98 ! [15:05:35] Less mixin that turns the header in 1px; CSS with display:none; or CSS with top: -9999px; [15:05:50] Hello ;) [15:05:51] addshore: Can I get some tasks to work on RevisionSlider that would guide me in understanding the code base much better. [15:05:56] Hi Niridya [15:06:20] I got one question now : why isn't the User info template not translated on my meta page ? https://meta.wikimedia.org/wiki/User:Niridya [15:06:37] Gopa_: so the starting place would be https://phabricator.wikimedia.org/T199877 [15:06:47] mainframe98: so your question is, what method to pick? [15:06:55] And there are 3 smallish tickets located as subtickets [15:07:18] addshore: Thanks ;) I will look over it. [15:07:22] CFisch_WMDE: Yes, that is correct. [15:07:29] Let us know which takes your fancy [15:07:58] mainframe98: so I would assume that display none will be ignored by screenreaders [15:08:21] addshore: >>Let us know which takes your fancy Me? [15:08:22] so if you still want screenreaders to work [15:08:28] Gopa_: yes! [15:08:30] I would go for the 1px method [15:09:52] Are any wikibase + RDF people here to tackle the curious case of the canonicalUriProperty setting not resulting in any changes to RDF output (at least on my instance, built using addshore's wonderful docker images) [15:10:45] Niridya: it looks good to me when viewing it on fr, https://meta.wikimedia.org/wiki/User:Niridya?uselang=fr [15:10:59] dbs: Lucas_WMDE might be your guy here :-) [15:11:13] Hi dbs, so let me have a dig around with you [15:11:15] well, I was going to mention that setting [15:11:22] but it seems you already found that? [15:11:23] aaah, Lucas_WMDE is also here! [15:11:40] * addshore will leave it to Lucas_WMDE :) [15:11:42] that wasn’t clear to me from the text on the TAIM page [15:11:45] yes - curiously, formatterUrlProperty is working as expected in the HTML [15:11:47] addshore: I am very excited to work on MediaWiki project's in between I saw this project and was quite interesting to me and working in a team, together for two months, Interacting with new developers was really amazing. [15:11:57] CFisch_WMDE: Thank you. That should be easy. I'm curious though, when looking at Vector's personalMenu.Less, it does use display:none for specific screenreader headers. [15:12:14] addshore some parts are working (the sentences for example) but the template itseilf (the About me title for example) remain in English [15:12:25] Lucas_WMDE: well I've updated that topic a few times today as I made progress; this is something I've been trying to get straight for 3 or 4 days now [15:12:25] CFisch_WMDE: I'll just stick to the 1px method and see where I'll end up. [15:12:46] mainframe98: Hmm interesting, - Volker_E__ do you have a note on that? [15:12:55] mainframe98: https://developer.mozilla.org/en-US/docs/Web/CSS/display#display_none says display: none hides content from accessibility tools [15:13:19] hmmmm Niridya I'll have a look at the template [15:13:43] thank you ! [15:14:02] Lucas_WMDE: That's exactly why I found it so strange Vector uses it in such way. [15:14:39] dbs: how are you checking the RDF representation? are you looking directly at Special:EntityData? [15:15:50] Niridya: so in Template:User_info I see {{ {{TNTN|Template:User info| uselang = {{{uselang|{{CURRENTCONTENTLANGUAGE}}}}} }} [15:16:14] Lucas_WMDE: using curl to request text/turtle representation at entity/Q# [15:16:22] addshore and is it good ? (i know how work templates but not well for translations) [15:16:33] * addshore is digging some more [15:16:41] Lucas_WMDE: 'curl -LH "Accept: text/turtle" http://localhost:8181/entity/Q16' specifically [15:16:46] dbs: can you perhaps add something like ?breakCache=387102459870 just to make sure it’s not caching anything? [15:17:19] mainframe98: So Volker_E__ says, display: none hides it from the screenreaders [15:17:25] but there is a mixin [15:17:29] in the mediawiki mixins [15:17:40] called screenreader text [15:17:55] that can be used for elements that should be visible to screenreaders only [15:18:07] but hidden for everything else [15:19:09] Lucas_WMDE: okay - tried appending the cache-busting GET param but it made no difference. good thought though! [15:19:22] okay, then I’m not sure why this wouldn’t work [15:19:37] unless perhaps this feature simply wasn’t available in whatever version Docker uses? [15:19:52] * Lucas_WMDE checks [15:20:19] CFisch_WMDE: Excellent. Thank you. I'll stick to mediawiki mixins [15:20:50] addshore: I hope you read the above message. [15:21:20] dbs: hm, that feature should be available in 1_30 [15:21:38] Gopa_: sorry, missed it but just saw it :) [15:21:49] Lucas_WMDE: I can try rebuilding the images from scratch. This is a pretty important use case for libraries who are getting on-board with the idea of maintaining their own linked data stores so happy to work through as much as possible [15:22:07] Gopa_: I hear that you already signed up for the season of RevisionSlider too [15:22:09] addshore: No worries :) [15:22:50] dbs: hm, looking at the implementation, it seems you might need to rebuild the property info after configuring that setting? [15:22:54] I’ll check how that’s done [15:23:07] Niridya: I did a test @ https://meta.wikimedia.org/wiki/User:Addshore/sandbox?uselang=de and the code in the template doesn't appear to be doing what I would expect [15:23:21] Lucas_WMDE: cool, thanks [15:23:55] dbs: I think you need this https://tools.wmflabs.org/sal/log/AV9Y7xNAF4fsM4DBdUqD [15:24:03] addshore and what did you expected to appear ? [15:24:09] php extensions/Wikidata/extensions/Wikibase/repo/maintenance/rebuildPropertyInfo.php --rebuild-all --force [15:24:17] addshore: Yes I am talking about the RevisionSlider project, I kept a message in that token. [15:24:28] So Niridya with uselang=de in the URL i was expecting to see Template:User info/de [15:24:28] hm, probably without the wikidata build in there [15:24:34] i. e. php extensions/Wikibase/repo/maintenance/rebuildPropertyInfo.php --rebuild-all --force [15:25:08] addshore i was just reading this in the template's documentation^^ [15:25:17] by the way, I do have a request for technical advice of my own :) [15:25:23] addshore do you know from where does this bug come from ? [15:25:24] I would be very happy if anyone could take a look at https://discourse-mediawiki.wmflabs.org/t/registering-an-optional-service-in-mediawikiservices/579 [15:25:25] Niridya: yes, something sounds odd [15:25:30] Niridya: not yet [15:25:36] Gopa_: So for the team project we want to start beginning of August there will be a detailed introduction at the Kick-Off [15:25:59] addshore okey ;) (you think you will find before the end of the IRC meeting ?) [15:26:05] Niridya: maybe ;) [15:26:09] CFisch_WMDE: yes I read about it. [15:26:09] ;) [15:26:16] i let you search ;) [15:26:24] Gopa_: But for now you could also look at the taskboard addshore linked earlier and see if something seems doable for you [15:26:38] CFisch_WMDE: Ok sure [15:26:55] CFisch_WMDE: Thanks I will start working on that. [15:27:36] It's pretty good already, if it's working for you locally and then you can start playing around a bit to see how it's all connected [15:28:33] Alright, now that the internet connection has been restored, I have another question: If I wanted to access something from the database during extension registration, how should I do that? [15:28:47] mainframe98_: wb [15:28:48] Lucas_WMDE: sounded promising but didn't see any change in the results, alas! [15:28:49] CFisch_WMDE: mainframe98: That's the mentioned mixin https://github.com/wikimedia/mediawiki/blob/master/resources/src/mediawiki.less/mediawiki.mixins.less#L192 [15:28:59] mainframe98_: what do you need exactly? [15:29:08] CFisch_WMDE: yes I already cloned, Installed locally. Now I am doing a case study over the code base. [15:29:11] and why do you need it before the extension registration [15:29:37] Currently, trying to call wfGetDB() in the extension registration fails because the chronology protection isn't set yet. I need it to retrieve settings from the database, which are needed before the service locator is reset [15:30:10] Certain config in setup.php depends on certain globals, and I need to have the configuration from the database before that point (at the extension registration) [15:30:12] mainframe98_: where in extension registration? in a hook or? [15:30:32] addshore: in the extension registration callback hook. [15:30:41] legoktm: ^^ [15:30:55] Lucas_WMDE: I think I'm going to try a fresh docker image to ensure there's nothing I've done previously contaminating the build [15:31:53] Lucas_WMDE: short-term, I'm hoping to convince a research group to consider wikibase for a LOD graph store but if I can't get RDF mappings working, that won't go too far--heh [15:32:10] mainframe98_: is the code available somewhere already? [15:32:12] The registry fires at Setup.php#124, which is before the global setup process at line 144 and later. If I just used the service locator, it would be too late, since that occurs at Line 614 [15:32:17] dbs: can you perhaps look at the property info in the database? [15:32:22] to see if it was imported or not [15:32:30] (the same as what you previously had hand-edited, I assume) [15:33:19] Niridya: I tried to demonstrate the issue a bit more at https://meta.wikimedia.org/wiki/User:Addshore/sandbox [15:33:28] CFisch_WMDE: Not yet. I had planned to release it after I would get a Proof of concept, but this is blocking that. I can probably create a gist though. [15:34:59] Niridya: I think you might actually have to mark your page as a translatable page instead of using the langswitch [15:35:22] Lucas_WMDE: sure thing - property_info.pi_info has {"type":"external-id","formatterURL":"https:\/\/example.org\/vocab-url\/$1","canonicalURI":"https:\/\/example.org\/vocab\/$1"} for the property in question [15:35:45] Niridya: looking at the docs of https://www.mediawiki.org/wiki/Template:CURRENTCONTENTLANGUAGE which are used in that user info template [15:36:02] "This template returns the "content language" of the current page; more precisely it checks if the current title ends with a language code, and if it is the case it returns this language code, else {{CONTENTLANGUAGE}} (the default content language; on this wiki: "en" for English)." [15:37:58] CFisch_WMDE: I hope you already read my mail about the RevisionSlider Project. [15:37:59] Lucas_WMDE: and the property that serves as the target for canonicalUriProperty setting is just {"type":"string"} (as expected I think) [15:38:58] dbs: hm, okay, that looks good at least [15:39:04] Niridya: yup, I think thats it! [15:39:10] Gopa_: Yeah I was in the process of responding already but got distracted by a lot of things today ^^' [15:39:49] CFisch_WMDE: It's ok ;) [15:40:43] Lucas_WMDE: re: optional service registration, one common method is to have a null service which implements the same interface but doesn't do anything [15:41:21] I have some patches i was wondering if i could get code review on ( https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/CategoryTree/+/444019/ https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/444757/ ) [15:41:42] dbs: wait, I’m not sure if *export* of those canonical thingies *to RDF* is also in 1.30… lemme check again [15:41:52] bawolff: =o [15:41:53] * bawolff is not sure if that is an appropiate thing to ask for at this meeting [15:43:06] bawolff: did you test the CategoryTree one? [15:43:14] bawolff: normally I would say it's not ....but depending on the size of the patch and if your asking as a volunteer it might be a yes ^^ [15:43:41] bawolff: and of cause you have to ask nicely! [15:44:11] dbs: nope :( [15:44:23] commit ad332804b1fea069043d14d0195f6fe2ed5a6f4b is apparently only in REL1_31, not in REL1_30 [15:44:41] Lucas_WMDE: we need to backport something for Wikibase? :D [15:45:02] addshore: or create a 1_31 Docker container? [15:45:22] addshore: umm i think i did. Its been a while since i submitted it [15:45:50] Niridya: I'm going to get my user page translated and then I'll be able to provide you with an example to copy :) [15:46:05] Lucas_WMDE: there is a ticket for that! [15:46:17] tgr: that means a lot of callers would have to check “instance of DummyWhatever” instead of “=== null” :/ [15:46:23] Lucas_WMDE: ah that's fantastic to know, at least [15:46:35] Lucas_WMDE: https://phabricator.wikimedia.org/T197212 [15:46:38] CFisch_WMDE: feel free to tell me to go away :) Sometimes it can be difficult to find people for certain areas of the code base [15:46:51] addshore thank you a lot ! (and sorry if i didn't answered quickly) [15:46:55] bawolff: No worries ;-) [15:46:59] why do callers need to know how the service is implemented? [15:46:59] Niridya: np :) [15:47:31] the categorytree one is definitely work related. I just have no idea who is maintaining category tree now a days [15:47:41] they don’t need to know how it’s implemented, they need to know if it’s available or not [15:47:47] and act differently if it’s not available [15:47:48] * dbs is now subscribed to https://phabricator.wikimedia.org/T197212 [15:48:05] in ways that a simple dummy implementation couldn’t provide [15:48:12] at least not without a lot of refactoring [15:48:14] bawolff: Maybe we should invent a we review each others patches IRC session! [15:48:56] Yeah, i think that would be cool :) [15:49:17] Niridya: infact, we are just going to do it on your page! :D (but the translated page stuff will only work on meta, not through the global user page magic) [15:49:55] Only on meta ? That's strange because LangSwitch works cross-wiki [15:50:05] CFisch_WMDE: bawolff I think maybe that was discussed at one point, or maybe there is already such a time [15:50:08] wasn't the code review hour canceled on the basis of merging it into this event? [15:50:25] tgr: ^^ that might be what I was thinking of! [15:50:33] It was more cancelled as a failure [15:50:40] =[ [15:52:18] Lucas_WMDE: oh wow, that commit was in 2016! Thanks for tracking that down [15:52:55] Niridya: so shall we try to mark your page as translatable? :) [15:52:57] or not bother [15:53:11] my biased opinion what happened: 1) most of the people involved did not have +2. 2) it was on an obscure irc channel and not well advertised 3) people could sign up patches beforehand...which turned into people other than the code author signing up these ten thousand line rewrite everything butrotted patches because they wanted to see cool feature X it promised [15:53:55] 7 mins left! [15:53:58] I dont think irc code review hour can really work for anything but small patches [15:54:15] addshore you can't only translate the template (if you can he will be translated on others users pages) ? [15:54:21] bawolff: I agree [15:54:40] that should be a basic rule for such a session [15:55:01] well, the template is already translated, but in order to take advantage of that translated table it looks as if you have to also translate the page the template is used on [15:55:17] dbs: it was only merged only a year later [15:55:22] for the +2 problem one could orchestrate - like here - that you have people with these rights present, that also feel responsible [15:55:22] that confused me at first as well :D [15:55:25] CFisch_WMDE: bawolff strict patch requirements, must be mergable, no -1s, less than 100 lines? etc :) [15:55:31] “surely a 2016 commit must be in 1.30?” [15:55:36] s/butrotted/bitrotted (thats a funny typo) [15:55:45] 5 mins left! [15:55:50] addshore It's quite strange, there are lots of more complicated templates where the translation work (for example Igen on Commons) [15:56:00] addshore but you can try if you want ;) [15:56:15] Lucas_WMDE: the service could always have an isAvailable() or supports(...) flag. It's ugly but the nice solution would probably be to refactor the code so it does not need to know what the service does or if it does anything at all [15:56:17] addshore: i would add to that must have been rebased in last 2 weeks [15:56:31] bawolff: yes :) or rebased during the week of the meeting [15:56:32] Niridya, have you seen this page? https://meta.wikimedia.org/wiki/Global_user_pages#Translation [15:56:53] everything translation related is a mess [15:57:01] bawolff: basically CI and gerrit both have to say yes :) [15:57:11] 3 mins left! [15:57:41] addshore no I didn't read this page, it's interessant [15:58:15] returning null for a service is problematic inasmuch as it forces all calling code with the service being null (so if your class stores the service in the constructor and calls it a hundred times all hundred places need a null check now) [15:58:50] Niridya, I think that's the only option for translation of GlobalUserPage. Because it only pulls the root page, not the subpages. [15:59:20] BUTTROTTED. [15:59:32] i will try in a short time, but can you locally translate Template:User info ? [15:59:42] Sorry. [16:00:22] tgr: that doesn’t sound too different from 100 ->supports() calls to me… [16:00:30] Times up! [16:00:33] \o/ [16:00:49] Huge thanks Lucas_WMDE++ [16:00:54] and we currently do those null checks, we’re just not using the service registry yet [16:00:56] :D Thanks Lucas_WMDE ! [16:00:58] Thanks for this awesome Technical Advice IRC Meeting! [16:01:04] thanks everyone! [16:01:05] Very busy one! [16:01:07] thanks tgr for your comments :r [16:01:09] * :) [16:01:10] it's not. Ideally code would be written in a way where that's not needed. [16:01:14] Oh yeah! [16:01:19] addshore you arrive at something ? [16:02:34] Niridya, I think using the #switch will work both at meta and at other SUL wikis. It also prevents the active translators from being distracted by less important work (ie. we don't want editors spending dozens of hours translating random userpages ;) [16:02:48] (of course it is a lot easier to make sweeping theoretical claims like that than dealing with an actual use case...) [16:03:03] and if we used the system it would just appear in their usual task queue system [16:03:57] ciao [16:04:13] (Niridya, Note: Addshore and I are disappearing to find food now. Good luck!) [16:04:24] quiddity that's a good idea to don't distract translators ;) [16:04:30] \o [16:04:58] good luck addshore ! [16:05:02] tgr: do you want to summarize that for Discourse or should I do it? [16:05:09] I think it would be useful to record the answer there [19:24:20] HI, I talk with user Seadick which opened task T199930. I saw to he earlier opened T124417. Can you check, is there problem with databases? [19:24:24] T124417: Connection to Wikimedia projects slow/timing out for some users - https://phabricator.wikimedia.org/T124417 [19:24:24] T199930: I can't edit and can't view preview - https://phabricator.wikimedia.org/T199930 [23:24:22] I'm getting a weird error when trying to delete something [23:24:49] eh, I reloaded it a third time and it worked [23:27:03] jonkolbert: in the future, please copy the error message when reporting it, so we can investigate :) [23:27:53] MatmaRex: Yeah, my bad. I accidentally closed out the window and tried to reproduce but it worked the last time around [23:28:00] sorry about that :P [23:28:06] :)