[09:20:06] 6Multimedia, 10MediaWiki-File-management: Thumbnailer generates ghost layers with random simple-paged TIFF files - https://phabricator.wikimedia.org/T91063#1091602 (10BeatEstermann) Further examples of this issue can be found here: https://commons.wikimedia.org/w/index.php?title=Special%3AListFiles&user=Kunstm... [09:57:58] 6Multimedia, 10MediaWiki-File-management: Thumbnailer generates ghost layers with random simple-paged TIFF files - https://phabricator.wikimedia.org/T91063#1091700 (10Nemo_bis) A thumbnail extracted from that "Photoshop Data: , 9076 bytes"? ``` $ tiffinfo LIBTIFF, Version 4.0.3 Copyright (c) 1988-199... [10:37:31] 6Multimedia, 10MediaWiki-Uploading, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: SVG upload should have more specific error (warning) message when blocking - https://phabricator.wikimedia.org/T85924#1091748 (10Darkdragon09) Hi, Can someone review this patch. The patch needs to be reviewed for th... [10:55:22] (03CR) 10Gilles: [C: 032] Fix typo in CorsLogger [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/194008 (https://phabricator.wikimedia.org/T507) (owner: 10Gergő Tisza) [10:56:11] (03Merged) 10jenkins-bot: Fix typo in CorsLogger [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/194008 (https://phabricator.wikimedia.org/T507) (owner: 10Gergő Tisza) [11:02:37] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Displaying the stashed file as a thumbnail in UploadWizard fails with an 503 - https://phabricator.wikimedia.org/T90599#1091789 (10Gilles) It does go to the scalers internally through the regular web reques... [12:23:50] 6Multimedia, 10MediaWiki-Uploading, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: SVG upload should have more specific error (warning) message when blocking - https://phabricator.wikimedia.org/T85924#1091878 (10Aklapper) (Please don't ping after two hours as every message (in Phab and in Gerrit)... [13:33:23] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 7Browser-test-bug: Beta Cluster: "error while storing the file in the stash.' - https://phabricator.wikimedia.org/T75229#1091977 (10yuvipanda) [14:02:12] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-UploadWizard, 10Wikimedia-Media-storage, 7Tracking: Uploadstash errors (tracking) - https://phabricator.wikimedia.org/T85568#1092035 (10Gilles) [14:02:13] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 7Browser-test-bug: Beta Cluster: "error while storing the file in the stash.' - https://phabricator.wikimedia.org/T75229#1092033 (10Gilles) 5Open>3Resolved a:3Gilles [14:56:22] (03PS3) 10Gilles: Improve transcode job insertion deduping [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) [14:57:04] (03CR) 10Gilles: Improve transcode job insertion deduping (032 comments) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) (owner: 10Gilles) [15:42:45] 6Multimedia, 10OTRS, 10Wikimedia-Media-storage, 7Epic: Provide a clickthrough workflow for file copyright email permissions - https://phabricator.wikimedia.org/T88620#1092250 (10Nemo_bis) > Email is somewhat understood as a legal tool for creating contracts, and has some value as an audit trail. Some valu... [15:53:31] 6Multimedia, 10MediaWiki-Uploading, 5Patch-For-Review: [REGRESSION] i18n fallback uses message from i18n file before message from db. - https://phabricator.wikimedia.org/T57473#1092275 (10Nemo_bis) >>! In T57473#1037355, @Rillke wrote: > [[ https://commons.wikimedia.org/wiki/Commons:Administrators%27_noticeb... [15:58:05] 6Multimedia, 10MediaWiki-extensions-Sentry, 10MediaWiki-extensions-UploadWizard, 3Roadmap: Deploy Sentry to production, configured to log only UploadWizard - https://phabricator.wikimedia.org/T91649#1092287 (10Gilles) 3NEW a:3Tgr [15:59:21] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Roadmap: Refactor UploadWizard to make step controllers truly independent - https://phabricator.wikimedia.org/T91650#1092297 (10Gilles) 3NEW a:3MarkTraceur [16:00:43] * James_F sighs at https://www.mediawiki.org/w/index.php?title=Wikimedia_Engineering/2014-15_Goals&diff=0&oldid=1425681 [16:00:59] why? [16:01:21] Merging the Multimedia team into core before you've done all the things we've been waiting for for years is unfun. :-( [16:02:03] Unless you think "API-callable, embeddable UploadWizard so VisualEditor can use it" is going to be delivered by core. [16:02:18] Or structured data for searching images in arbitrary languages. [16:02:35] could be, gergo and mark will be on the API sub-team [16:02:50] Hmm. [16:02:54] structured data needs a much larger team to take off, though [16:03:00] I've been told "could be" for three years. [16:03:33] True, but killing the team and subsuming into a team with a much much much larger area to work on doesn't make it more likely. [16:04:50] it's one of those things that needs a proper organizational drive and staffing [16:05:12] piling that on us on top of keeping 15 odd extensions alive is inadequate [16:06:24] and we don't even have a PM, it would have been a death march to embark on that now [16:07:00] Sure. [16:07:07] I'm not asking a team of three to take it on. [16:07:31] I'm asking for the team to exist so I can keep lobbying for it to get the damn resources to do what I need from it, though. :-) [16:07:38] * James_F grumbles. [16:08:06] we need a structured data team more than we need a multimedia team, imho [16:08:45] a video team if that's a priority, etc. [16:09:06] multimedia is too broad and there's almost no expertise crossing over from one project/extension to the next [16:11:12] it might make sense if it was 3 or 4 times bigger and you essentially had sub-teams handling these topics [16:11:46] but right now the context switching is ridiculous, some extensions we're responsible for I've never pulled from git [16:12:51] James_F: Realistically it's a team of one right now, I'm the only one working on refactoring UW, the others have different priorities [16:12:56] Which makes sense [16:13:06] But it means very little chance of a big change like embeddable UW [16:13:18] And that wasn't even the goal, because I knew we wouldn't get there [16:13:22] :-( [16:13:54] It's been the #1 request from the Editing team of Platform, then Multimedia, and now again Platform for three years. [16:14:42] to be fair the work we're doing on uw right now will make that easier [16:15:05] Yeah, we couldn't do it without making the system work in a sane way. [16:15:10] it's still a long-term investment on UW to clean it up, fix it, test it [16:15:46] and gergo's work will benefit all JS projects [16:16:01] * James_F nods. [16:16:14] I'm not saying you three aren't working on important things. [16:16:29] It's just hugely frustrating that our requests seem to always go nowhere. :-( [16:16:39] tough choices to be made when we're vastly understaffed compared to our ambitions [16:18:11] Except there doesn't appear to be anyone making this choice explicitly. [16:18:21] No-one has ever said "no, we're not doing that" to me. [16:19:03] I've never been asked to help prioritise areas of work. [16:20:28] well, all we said is structured data is too big for us to take on, then the idea of disbanding us came up [16:20:38] embedded uw wasn't a topic that was discussed [16:20:43] Why not? [16:20:58] wasn't on erik's mind nor ours, I guess? [16:21:02] * James_F sighs. [16:21:15] but if you bring it up to erik, it could be something we do in Q4, further delaying our disbanding [16:21:17] Want me to go to Erik and demand you get 10 engineers so I get embedded UW? [16:21:29] no, just ask for that and delay our next assignment [16:21:33] That might not go very well for you. :P [16:21:51] hard to say if we can fit it in Q4 alone though [16:22:04] Well, yeah, it doesn't easily fit in a quarter. [16:22:10] That's why it keeps getting delayed, it seems. [16:22:17] And now it's just been shunted off the cliff instead. [16:23:47] James_F: I'll likely still be working on refactoring UploadWizard, slowly but surely. [16:23:47] well, you can always argue that we threw 4 quarters at media viewer, so budgeting 2 quarters for this given our team size isn't a ridiculous request [16:24:25] gi11es: I can argue, but in a world where everything ships in a quarter or dies, it probably won't gain traction. :-( [16:24:40] cf. VE, Flow, and Media Viewer. [16:25:29] marktraceur: We've shipped something new to actual users every quarter for 6 quarters now… [16:25:37] Ah, neat. [16:25:46] marktraceur: Flow has actual users too, albeit fewer so far. [16:26:04] Wait, no, 7 quarters. [16:26:07] Whatever. [16:26:44] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Roadmap: Significantly increase UploadWizard test coverage - https://phabricator.wikimedia.org/T91651#1092357 (10Gilles) 3NEW a:3Gilles [16:28:17] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Roadmap: Address technical issues responsible for funnel drop-offs in UploadWizard - https://phabricator.wikimedia.org/T91652#1092375 (10Gilles) 3NEW [16:28:36] Ooh, fancy PMing footwork here. [16:28:48] It being an odd-numbered day and all. :) [16:29:00] PO, I'm not graced with the full title [16:29:41] Aw. [16:29:47] gi11es: "Graced" is a polite way to put it. :-) [16:30:03] "Burdened"? But in the most positive sense. [16:30:12] As in "My what a lovely burden" [16:31:15] (03PS4) 10Gilles: Move makeTransitioner to the step objects [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194168 (https://phabricator.wikimedia.org/T91258) (owner: 10MarkTraceur) [16:31:55] I tried to find the Frasier clip that would explain the reference, but failed. [16:32:57] marktraceur: OK. [16:33:31] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Roadmap, 7Technical-Debt: Refactor UploadWizard to make step controllers truly independent - https://phabricator.wikimedia.org/T91650#1092420 (10Aklapper) [16:35:29] speaking of those roadmap tasks marktraceur, feel free to link them to the actual workable/worked ones as blocking tasks [16:36:06] same for the sentry ones tgr|away [16:36:08] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 7Tracking: Video subtitle support (tracking) - https://phabricator.wikimedia.org/T66031#1092430 (10Aklapper) [16:36:14] gi11es: Will do. [16:36:26] I'll leave it up to you to decide if that's useful phabricator tidying or not :) [16:36:49] gi11es: Not sure yet, but next time I'm dealing with those tasks, I will probably do it out of compulsive need for order [16:37:16] Hm, these file info messages seem very lego-y [16:37:52] Oh, no, they're just...different [16:37:58] "special". [16:40:14] :o [16:40:22] legoktm: Not you. [16:40:30] :( [16:40:36] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Failure for NASA option of browser test on integration server - https://phabricator.wikimedia.org/T90890#1092441 (10Gilles) [16:40:37] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Browser tests no longer work on Chrome - https://phabricator.wikimedia.org/T86770#1092442 (10Gilles) [16:40:38] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Don't use the preferences page to update preferences for each test. Only do that in the test dedicated to preferences - https://phabricator.wikimedia.org/T87462#1092444 (10Gilles) [16:40:39] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Browser test for flickr import - https://phabricator.wikimedia.org/T86120#1092440 (10Gilles) [16:40:40] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Quality-Assurance, 5Patch-For-Review: UploadWizard browser test for chunked upload - https://phabricator.wikimedia.org/T89289#1092443 (10Gilles) [16:40:42] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Default license in preferences not respected for multi-file uploads going through the per-file license picking step - https://phabricator.wikimedia.org/T89717#1092445 (10Gilles) [16:40:43] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Roadmap: Significantly increase UploadWizard test coverage - https://phabricator.wikimedia.org/T91651#1092439 (10Gilles) [16:40:45] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Disambiguate features in browser tests - https://phabricator.wikimedia.org/T90696#1092446 (10Gilles) [16:40:46] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Browser test for "upload more files" - https://phabricator.wikimedia.org/T86119#1092448 (10Gilles) [16:40:48] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Browser test for "copy file metadata" - https://phabricator.wikimedia.org/T89356#1092447 (10Gilles) [16:40:53] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Roadmap: Significantly increase UploadWizard test coverage - https://phabricator.wikimedia.org/T91651#1092357 (10Gilles) [16:40:54] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Browser test for "upload more files" - https://phabricator.wikimedia.org/T86119#1092449 (10Gilles) 5Open>3Resolved [16:41:16] legoktm: I think in this case it's a good thing, because these messages are a bit weird. [16:44:56] pg damn it [16:47:52] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1092522 (10MarkTraceur) @Pginer-WMF your mockup is on a PDF that only has one page - on multiple-paged PDFs, the page count comes after th... [16:55:39] (03CR) 10Gilles: [C: 032] Move makeTransitioner to the step objects [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194168 (https://phabricator.wikimedia.org/T91258) (owner: 10MarkTraceur) [17:00:12] (03PS9) 10Gilles: Move updateFileCounts to the step controllers [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/184739 (https://phabricator.wikimedia.org/T86679) (owner: 10MarkTraceur) [17:00:49] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Move newUpload to the Upload step controller - https://phabricator.wikimedia.org/T86679#973962 (10Gilles) [17:01:23] (03Merged) 10jenkins-bot: Move makeTransitioner to the step objects [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194168 (https://phabricator.wikimedia.org/T91258) (owner: 10MarkTraceur) [17:03:28] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1092564 (10MarkTraceur) @Pginer-WMF sorry to hit you with concerns after the fact, but you also have, in the mockup, only part of the seco... [17:06:52] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1092577 (10MarkTraceur) And...what is that link supposed to be to? [17:23:47] (03CR) 10Gilles: [C: 032] Move updateFileCounts to the step controllers [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/184739 (https://phabricator.wikimedia.org/T86679) (owner: 10MarkTraceur) [17:24:01] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Roadmap, 7Technical-Debt: Refactor UploadWizard to make step controllers truly independent - https://phabricator.wikimedia.org/T91650#1092632 (10Eloquence) @Gilles, thanks for the roadmap edits! I am removing this one and excluding refactoring/tech debt fr... [17:24:09] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 7Technical-Debt: Refactor UploadWizard to make step controllers truly independent - https://phabricator.wikimedia.org/T91650#1092633 (10Eloquence) [17:26:16] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Make the transports mostly independent, and add tests for them - https://phabricator.wikimedia.org/T85629#1092659 (10Gilles) All the changesets found here have been merged. Is there more work left to do on... [17:29:50] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1092668 (10Pginer-WMF) I think the same approach should work on multi-page PDFs, but it is a good catch, we can check how it looks in case... [17:30:08] (03Merged) 10jenkins-bot: Move updateFileCounts to the step controllers [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/184739 (https://phabricator.wikimedia.org/T86679) (owner: 10MarkTraceur) [18:03:04] Ooh. [18:04:27] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 7Blocked-on-Editing, 5Patch-For-Review: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#1092773 (10RobLa-WMF) [18:05:20] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 7Blocked-on-Editing, 5Patch-For-Review: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#942722 (10RobLa-WMF) The people that have the expertise to review this a... [18:12:27] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Displaying the stashed file as a thumbnail in UploadWizard fails with an 503 - https://phabricator.wikimedia.org/T90599#1092800 (10Ragesoss) I tried it again with that testwiki debug extension yesterday. G... [18:13:38] gi11es: Do you know if that magic up-arrow trick is a less mixin in core, or do I need to copy/paste it somehow [18:27:17] gi11es, this is a judgment call but I'll remove test coverage related tasks from the roadmap for now as well -- and we'll just keep the Sentry & funnel optimization goals, because those have the biggest dev/user impact [18:28:19] 6Multimedia, 6Editing, 10MediaWiki-Watchlist, 10MediaWiki-extensions-GlobalUsage, and 2 others: Show edits made on WikimediaCommons on the Watchlist - https://phabricator.wikimedia.org/T91192#1092901 (10Jdforrester-WMF) [18:28:20] 6Multimedia, 6Editing, 10Possible-Tech-Projects, 10Wikimedia-General-or-Unknown, 7Epic: Allow uploading of 3D files to Wikimedia Commons - https://phabricator.wikimedia.org/T3790#1092907 (10Jdforrester-WMF) [18:34:05] 6Multimedia, 10MediaWiki-extensions-UploadWizard: Significantly increase UploadWizard test coverage - https://phabricator.wikimedia.org/T91651#1092969 (10Eloquence) [18:47:51] (03CR) 10Aaron Schulz: [C: 031] Improve transcode job insertion deduping (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) (owner: 10Gilles) [18:56:25] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 6Security, 7Design, 5Patch-For-Review: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1093067 (10MarkTraceur) Here's what I've got so far. {F56124} [20:07:42] marktraceur: what up-arrow trick? [20:29:15] 6Multimedia, 10Wikipedia-App-iOS-App: Package OGVKit for CocoaPods (iOS Ogg Vorbis/Theora playback library) - https://phabricator.wikimedia.org/T91689#1093387 (10brion) 3NEW a:3brion [20:29:43] 6Multimedia, 10Wikipedia-App-iOS-App: Package OGVKit for CocoaPods (iOS Ogg Vorbis/Theora playback library) - https://phabricator.wikimedia.org/T91689#1093398 (10brion) [20:29:53] 6Multimedia, 10Wikipedia-App-iOS-App: Package OGVKit for CocoaPods (iOS Ogg Vorbis/Theora playback library) - https://phabricator.wikimedia.org/T91689#1093387 (10brion) [20:36:41] gi11es: For the dialogs, never mind, I got it figured out. [20:36:49] It was easier than I remembered. [21:02:32] 6Multimedia, 10MediaWiki-extensions-Sentry, 10MediaWiki-extensions-UploadWizard, 3Roadmap: Deploy Sentry (JavaScript error logging) to production, configured to log only UploadWizard - https://phabricator.wikimedia.org/T91649#1093514 (10Eloquence) [22:41:18] (03PS1) 10Krinkle: Update JSDuck config to standard jsduck.json [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/194720 [22:43:18] (03PS2) 10Krinkle: Update JSDuck config to standard jsduck.json [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/194720 [23:27:16] (03PS1) 10Chad: Profile command line scripts the new way [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194743 [23:31:02] (03PS2) 10Chad: Profile command line scripts the new way [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194743