[07:28:43] (03CR) 10jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522343 (owner: 10L10n-bot) [07:38:21] (03PS1) 10Matthias Mullie: Start writing entities into wb_terms [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522355 (https://phabricator.wikimedia.org/T223792) [07:39:02] (03CR) 10Matthias Mullie: [C: 04-1] "-1 until having figured out whether the wb_terms migration has any impact on this" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522355 (https://phabricator.wikimedia.org/T223792) (owner: 10Matthias Mullie) [07:44:47] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Spike: [Spike 2 days] Plan to access SDC data from Lua - https://phabricator.wikimedia.org/T227056 (10matthiasmullie) [07:49:54] 10Multimedia, 10SDC Engineering, 10SDC General: Backfill terms index - https://phabricator.wikimedia.org/T227847 (10matthiasmullie) [07:56:18] 10Multimedia, 10SDC Engineering, 10SDC General, 10Wikidata: [Spike 1 day] Figure out wb_terms redesign, and what that means for MediaInfo - https://phabricator.wikimedia.org/T227848 (10matthiasmullie) [08:00:27] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Spike: [Spike 2 days] Plan to access SDC data from Lua - https://phabricator.wikimedia.org/T227056 (10matthiasmullie) 05Open→03Resolved As mentioned in T223792: Current Wikibase Lua support seems able to deal with MediaInfo entities already... [08:01:38] (03CR) 10jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522332 (owner: 10L10n-bot) [08:19:29] 10Multimedia, 10Commons, 10Wikimedia-Video: Transcode of a video gets turned upside down - https://phabricator.wikimedia.org/T227850 (10AlexisJazz) [08:51:38] 10Multimedia, 10SDC Engineering, 10SDC General: Backfill terms index - https://phabricator.wikimedia.org/T227847 (10Cparle) I thought the `wb_terms` table was being deprecated ... hmmm, maybe I'm mixing it up with something else [08:54:53] 10Multimedia, 10SDC Engineering, 10SDC General: Backfill terms index - https://phabricator.wikimedia.org/T227847 (10matthiasmullie) Nope, you're not mixing it up with something else; it is indeed undergoing redesign. I've created T227848 to look into that, and what it'd mean in this context. [11:02:59] (03CR) 10Daimona Eaytoy: "recheck" [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/521615 (owner: 10Jforrester) [11:03:40] (03CR) 10Daimona Eaytoy: "recheck" [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/512473 (owner: 10Umherirrender) [11:07:11] 10Multimedia, 10EasyTimeline, 10Graphs, 10Wikimedia-Extension-setup, and 2 others: Transition all use of EasyTimeline to the Graph extension and decommission it from Wikimedia's servers - https://phabricator.wikimedia.org/T137291 (10Milimetric) @Krinkle, I agree, we should make Graph/Graphoid maintainable... [11:42:10] 10Multimedia, 10Commons, 10Wikimedia-Video: Transcode of a video gets turned upside down - https://phabricator.wikimedia.org/T227850 (10AlexisJazz) [11:44:08] 10Multimedia, 10WikibaseMediaInfo, 10Discovery-Search (Current work), 10Multimedia-Current-Work, 10Patch-For-Review: Index captions as description fields not label - https://phabricator.wikimedia.org/T226722 (10Cparle) a:03Cparle [11:46:11] 10Multimedia, 10UploadWizard: [BUG] In UploadWizard's warning to disallowed filenames, "more info" does not link to anything - https://phabricator.wikimedia.org/T227871 (10Roy17) [12:09:24] (03PS1) 10Matthias Mullie: Better guard against missing data [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522431 (https://phabricator.wikimedia.org/T227798) [12:09:27] (03PS1) 10Matthias Mullie: Create base before-/afterEach to contruct test environment [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522432 [12:10:04] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review: "Add a caption" is broken - https://phabricator.wikimedia.org/T227798 (10matthiasmullie) a:03matthiasmullie [12:26:53] (03CR) 10jerkins-bot: [V: 04-1] Create base before-/afterEach to contruct test environment [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522432 (owner: 10Matthias Mullie) [12:35:58] 10Multimedia, 10SDC General, 10CommRel-Specialists-Support (Apr-Jun-2019): Community Relations support for the depicts statement SDC release - https://phabricator.wikimedia.org/T213764 (10Elitre) >>! In T213764#5321540, @Keegan wrote: >>>! In T213764#5321001, @Elitre wrote: >> @keegan Do I understand correct... [12:38:36] 10Multimedia, 10SDC General, 10Wikidata, 10CommRel-Specialists-Support (Jul-Sep-2019): First feature suite for Structured Data on Commons blog post - https://phabricator.wikimedia.org/T205484 (10Elitre) [12:38:57] 10Multimedia, 10SDC General, 10Wikidata, 10CommRel-Specialists-Support (Jul-Sep-2019): First feature suite for Structured Data on Commons blog post - https://phabricator.wikimedia.org/T205484 (10Elitre) IIUC you're getting this started on Wikimedia Space! [12:42:50] (03PS2) 10Matthias Mullie: Create base before-/afterEach to contruct test environment [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522432 [12:58:50] (03CR) 10Matthias Mullie: [C: 03+1] "LGTM - will test & merge on Monday." (032 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/520458 (owner: 10Eric Gardner) [13:03:09] (03CR) 10Matthias Mullie: "Looks good except for that promise.abort." (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/521394 (https://phabricator.wikimedia.org/T227230) (owner: 10Eric Gardner) [13:03:14] (03CR) 10Matthias Mullie: [C: 04-1] Add Wikidata links to statement UI elements [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/521394 (https://phabricator.wikimedia.org/T227230) (owner: 10Eric Gardner) [13:12:42] (03CR) 10Matthias Mullie: [C: 04-1] "I'm not a fan of this amount of clarification/warning/handholding, but 't is what we want, I guess..." (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/521461 (https://phabricator.wikimedia.org/T224249) (owner: 10Cparle) [13:14:31] (03Abandoned) 10Matthias Mullie: build: Update wdio-mocha-framework from 0.6.3 to 0.6.4 [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/502847 (https://phabricator.wikimedia.org/T500935) (owner: 10Eric Gardner) [13:40:36] 10Multimedia, 10Commons, 10Lexicographical data, 10Wikidata, and 2 others: Console error "Unknown module: wikibase.experts.Lexeme" on File pages in Commons - https://phabricator.wikimedia.org/T227504 (10Ladsgroup) The fix was actually very easy. [13:40:57] 10Multimedia, 10Commons, 10Lexicographical data, 10Wikidata, and 2 others: Console error "Unknown module: wikibase.experts.Lexeme" on File pages in Commons - https://phabricator.wikimedia.org/T227504 (10Ladsgroup) [14:08:43] (03PS1) 10Cparle: Index captions as description field rather than label [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522473 (https://phabricator.wikimedia.org/T226722) [14:10:33] (03PS2) 10Cparle: Index captions as description field rather than label [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522473 (https://phabricator.wikimedia.org/T226722) [14:11:06] (03CR) 10Cparle: [C: 03+2] Better guard against missing data [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522431 (https://phabricator.wikimedia.org/T227798) (owner: 10Matthias Mullie) [14:19:32] (03CR) 10jerkins-bot: [V: 04-1] Index captions as description field rather than label [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522473 (https://phabricator.wikimedia.org/T226722) (owner: 10Cparle) [14:32:16] (03CR) 10jerkins-bot: [V: 04-1] Better guard against missing data [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522431 (https://phabricator.wikimedia.org/T227798) (owner: 10Matthias Mullie) [14:32:45] 10Multimedia, 10MediaViewer, 10Patch-For-Review, 10Release-Engineering-Team (Unit & Int & System Tooling), and 2 others: selenium-MultimediaViewer Jenkins job failing with `mediaWiki.eventLog.declareSchema is not a function` - https://phabricator.wikimedia.org/T227771 (10zeljkofilipin) 05Open→03Resolved [14:32:48] 10Multimedia, 10MediaViewer, 10Release-Engineering-Team (Unit & Int & System Tooling), 10Release-Engineering-Team-TODO (201907), and 2 others: selenium-MultimediaViewer Jenkins job failures - https://phabricator.wikimedia.org/T227770 (10zeljkofilipin) [14:33:03] 10Multimedia, 10MediaViewer, 10Release-Engineering-Team (Unit & Int & System Tooling), 10Release-Engineering-Team-TODO (201907), and 2 others: selenium-MultimediaViewer Jenkins job failures - https://phabricator.wikimedia.org/T227770 (10zeljkofilipin) 05Open→03Resolved [14:33:10] (03CR) 10Matthias Mullie: [C: 03+2] Better guard against missing data [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522431 (https://phabricator.wikimedia.org/T227798) (owner: 10Matthias Mullie) [15:17:32] (03Merged) 10jenkins-bot: Better guard against missing data [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522431 (https://phabricator.wikimedia.org/T227798) (owner: 10Matthias Mullie) [15:19:50] (03PS3) 10Cparle: Index captions as description field rather than label [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522473 (https://phabricator.wikimedia.org/T226722) [15:38:20] (03CR) 10jenkins-bot: Better guard against missing data [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522431 (https://phabricator.wikimedia.org/T227798) (owner: 10Matthias Mullie) [15:48:15] 10Multimedia, 10SDC General, 10CommRel-Specialists-Support (Apr-Jun-2019): Community Relations support for the depicts statement SDC release - https://phabricator.wikimedia.org/T213764 (10Keegan) >>! In T213764#5328225, @Elitre wrote: >>>! In T213764#5321540, @Keegan wrote: >>>>! In T213764#5321001, @Elitre... [16:35:03] (03PS1) 10Krinkle: selenium: Remove test for Special:Version [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522515 [18:06:34] 10Multimedia, 10UploadWizard: [BUG] In UploadWizard's warning to disallowed filenames, "more info" does not link to anything - https://phabricator.wikimedia.org/T227871 (10matmarex) I debugged a little bit because I remember this used to work back in the day when I wrote this code. :) The problem is caused b... [18:06:45] (03PS1) 10Bartosz Dziewoński: uw.ValidationMessageElement: Fix error messages with buttons/links [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522523 (https://phabricator.wikimedia.org/T227871) [18:06:47] (03PS1) 10Bartosz Dziewoński: Use parseDom() instead of parse() whenever possible [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522524 [18:07:29] (03PS2) 10Bartosz Dziewoński: Use parseDom() instead of parse() whenever possible [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522524 [19:26:46] (03CR) 10Jforrester: [C: 03+2] uw.ValidationMessageElement: Fix error messages with buttons/links [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522523 (https://phabricator.wikimedia.org/T227871) (owner: 10Bartosz Dziewoński) [19:31:20] (03Merged) 10jenkins-bot: uw.ValidationMessageElement: Fix error messages with buttons/links [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522523 (https://phabricator.wikimedia.org/T227871) (owner: 10Bartosz Dziewoński) [19:33:39] (03CR) 10jenkins-bot: uw.ValidationMessageElement: Fix error messages with buttons/links [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522523 (https://phabricator.wikimedia.org/T227871) (owner: 10Bartosz Dziewoński) [19:46:54] 10Multimedia, 10SDC Engineering, 10SDC General: Backfill terms index - https://phabricator.wikimedia.org/T227847 (10alaa_wmde) Yes it is correct that wb_terms is being redesigned. It is still being used in production as for the moment and there will be announcements for when we might transition to reading fr... [19:50:21] 10Multimedia, 10SDC Engineering, 10SDC General: Backfill terms index - https://phabricator.wikimedia.org/T227847 (10alaa_wmde) @matthiasmullie okay now I have got to {T227848} and understand the context. I'll reply on that task. [20:07:27] (03PS7) 10Eric Gardner: Add Wikidata links to statement UI elements [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/521394 (https://phabricator.wikimedia.org/T227230) [20:08:00] (03CR) 10Eric Gardner: Add Wikidata links to statement UI elements (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/521394 (https://phabricator.wikimedia.org/T227230) (owner: 10Eric Gardner) [20:14:34] 10Multimedia, 10SDC Engineering, 10SDC General, 10Wikidata: [Spike 1 day] Figure out wb_terms redesign, and what that means for MediaInfo - https://phabricator.wikimedia.org/T227848 (10alaa_wmde) > what is going to change and where in the process we are The end goal is that we replace wb_terms completel... [20:15:48] (03PS3) 10Bartosz Dziewoński: Use parseDom() instead of parse() wherever possible [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522524 [20:18:10] James_F: Are you and Matmarex starting a new Multimedia team? And not inviting me? :P [20:22:25] 10Multimedia, 10SDC Engineering, 10SDC General: Backfill terms index - https://phabricator.wikimedia.org/T227847 (10Ladsgroup) @matthiasmullie Let me stop you right there, populating `wb_terms` on commons was intentionally stopped (cc. @jcrespo) (and it should be a clean table) because it's pretty inefficie... [20:30:06] (03CR) 10Eric Gardner: [C: 03+2] "This will make it much easier to actually write (and read) tests going forward, nice work." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522432 (owner: 10Matthias Mullie) [20:31:14] 10Multimedia, 10UploadWizard, 10MW-1.34-notes (1.34.0-wmf.14; 2019-07-16): [BUG] In UploadWizard's warning to disallowed filenames, "more info" does not link to anything - https://phabricator.wikimedia.org/T227871 (10matmarex) 05Open→03Resolved a:03matmarex The fix will be deployed to Wikimedia wikis... [20:31:55] marktraceur: MatmaRex writes good code. I merge code by randoms if I have a spare 10 minutes. ;-) [20:32:02] S'a good combo. [20:32:09] Heh indeed [20:32:35] (03CR) 10Jforrester: [C: 03+2] Use parseDom() instead of parse() wherever possible [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522524 (owner: 10Bartosz Dziewoński) [20:47:30] (03CR) 10Eric Gardner: [C: 03+2] "BTW, Matthias – MobileFrontend is also consolidating some of their node-qunit testing setup to work like an installable library: https://g" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522432 (owner: 10Matthias Mullie) [20:54:10] (03Merged) 10jenkins-bot: Create base before-/afterEach to contruct test environment [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522432 (owner: 10Matthias Mullie) [20:54:12] (03Merged) 10jenkins-bot: Use parseDom() instead of parse() wherever possible [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522524 (owner: 10Bartosz Dziewoński) [20:56:36] (03CR) 10jenkins-bot: Use parseDom() instead of parse() wherever possible [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/522524 (owner: 10Bartosz Dziewoński) [20:59:12] (03CR) 10jenkins-bot: Create base before-/afterEach to contruct test environment [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522432 (owner: 10Matthias Mullie) [21:51:07] (03CR) 10Jforrester: [C: 04-2] "We promised the DBAs that we absolutely would not under any circumstances do this." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522355 (https://phabricator.wikimedia.org/T223792) (owner: 10Matthias Mullie) [22:25:52] (03PS1) 10Eric Gardner: WIP: Add LinkNoticeWidget above Statements Panel [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/522563 (https://phabricator.wikimedia.org/T227230)