[00:03:55] marktraceur: hey, so i'm working on https://phabricator.wikimedia.org/T97426 . switching it to OOUI's date input widget should be easy, but i noticed that all the other fields are still plain input fields (or jQuery UI thingies). it would look a bit meh [00:04:09] marktraceur: so my question is, is your https://gerrit.wikimedia.org/r/#/c/212842/ as very WIP as it says? :P [00:06:23] hmm, i guess it is. [00:22:50] oh gods i have wildly underestimated how hairy this code is. [01:39:11] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10OOjs-UI: Use OOUI datepicker input widget in UploadWizard - https://phabricator.wikimedia.org/T97426#1497547 (10matmarex) This seems to actually be impossible, as the date field in UploadWizard is actually a freeform text input field, with a loosely attache... [01:46:52] 6Multimedia, 10MediaWiki-Uploading, 5MW-1.26-release, 5WMF-deploy-2015-08-04_(1.26wmf17): mw.Upload should have a shortcut for getting the imageinfo for the uploaded file - https://phabricator.wikimedia.org/T107460#1497554 (10matmarex) 5Open>3Resolved [05:04:46] Yeah, it is. [05:04:51] But he's not home anymore [05:06:35] marktraceur: Keeping the error messages simple for now, will amend [05:06:48] marktraceur: Writing doc strings and pushing to gerrit :D [08:11:24] 6Multimedia, 10MediaWiki-extensions-PdfHandler: PDF warning popup misaligned - https://phabricator.wikimedia.org/T107554#1497852 (10Gilles) 3NEW [08:11:31] 6Multimedia, 10MediaWiki-extensions-PdfHandler, 7Easy: PDF warning popup misaligned - https://phabricator.wikimedia.org/T107554#1497859 (10Gilles) [08:34:28] (03CR) 10Gergő Tisza: [C: 032] Track image load time with statsv [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/226556 (https://phabricator.wikimedia.org/T105681) (owner: 10Gilles) [08:35:33] (03Merged) 10jenkins-bot: Track image load time with statsv [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/226556 (https://phabricator.wikimedia.org/T105681) (owner: 10Gilles) [08:49:25] (03PS1) 10Gilles: Track image load time with statsv [extensions/MultimediaViewer] (wmf/1.26wmf16) - 10https://gerrit.wikimedia.org/r/228218 (https://phabricator.wikimedia.org/T105681) [08:49:39] (03CR) 10Gilles: [C: 032] Track image load time with statsv [extensions/MultimediaViewer] (wmf/1.26wmf16) - 10https://gerrit.wikimedia.org/r/228218 (https://phabricator.wikimedia.org/T105681) (owner: 10Gilles) [08:50:37] (03CR) 10jenkins-bot: [V: 04-1] Track image load time with statsv [extensions/MultimediaViewer] (wmf/1.26wmf16) - 10https://gerrit.wikimedia.org/r/228218 (https://phabricator.wikimedia.org/T105681) (owner: 10Gilles) [08:51:45] (03PS2) 10Gilles: Track image load time with statsv [extensions/MultimediaViewer] (wmf/1.26wmf16) - 10https://gerrit.wikimedia.org/r/228218 (https://phabricator.wikimedia.org/T105681) [08:55:17] (03CR) 10Gilles: [C: 032] Track image load time with statsv [extensions/MultimediaViewer] (wmf/1.26wmf16) - 10https://gerrit.wikimedia.org/r/228218 (https://phabricator.wikimedia.org/T105681) (owner: 10Gilles) [10:49:30] 6Multimedia, 10Continuous-Integration-Infrastructure, 6operations, 5Patch-For-Review: Investigate impact of switching from ffmpeg to libav (ffmpeg is not in Jessie) - https://phabricator.wikimedia.org/T103335#1498115 (10fgiunchedi) >>! In T103335#1481198, @MoritzMuehlenhoff wrote: >>>! In T103335#1480470,... [10:55:23] 6Multimedia, 10Continuous-Integration-Infrastructure, 6operations, 5Patch-For-Review: Investigate impact of switching from ffmpeg to libav (ffmpeg is not in Jessie) - https://phabricator.wikimedia.org/T103335#1498117 (10MoritzMuehlenhoff) > I gave this a try, so far we already backported `x265` and `shine`... [10:57:26] 6Multimedia, 10Continuous-Integration-Infrastructure, 6operations, 5Patch-For-Review: Investigate impact of switching from ffmpeg to libav (ffmpeg is not in Jessie) - https://phabricator.wikimedia.org/T103335#1498118 (10MoritzMuehlenhoff) We could also disable shine, that's only relevant for embedded hardw... [12:40:58] (03PS1) 10Bartosz Dziewoński: Remove some pointless datepicker code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228243 [12:42:37] heh, uploadwizard hardcodes some HTTP links to commons. [12:47:00] (03PS1) 10Bartosz Dziewoński: Use HTTPS for links to Commons and MediaWiki.org [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228244 [13:12:35] MatmaRex: I believe it. [13:13:41] marktraceur: this code has been evolving for so long, it's a wonder it's not self-aware yet. [13:14:18] 6Multimedia, 10MediaWiki-Vagrant, 10MediaWiki-extensions-TimedMediaHandler, 7Upstream: ogv transcodes have broken audio tracks with ffmpeg2theora 0.29 (Trusty upstream) - https://phabricator.wikimedia.org/T69953#1498266 (10fgiunchedi) I tried this on trusty with ffmpeg2theora 0.29 from debian and an update... [13:14:19] i'm currently trying to figure out a way to both display a sane date widget with validation, and allow arbitrary input. i think i'll put in some button to switch to plain text input. [13:16:07] MatmaRex: Sounds fine. [13:16:26] MatmaRex: Are we certain it's *not* self-aware? [13:16:40] Could we be violating the prime directive by refactoring this code is all I want to know [13:31:45] 6Multimedia, 10MediaWiki-Uploading: MediaWiki api doesn't serve EXIF GPSAltitude (and other tags) as decimals - https://phabricator.wikimedia.org/T34410#1498296 (10matmarex) @Bawolff, https://gerrit.wikimedia.org/r/#/c/20288/ / d1f0c5a7d8e25d378b06fb0850727a357b6b39f6 has been merged, is this bug fixed? [13:34:07] 6Multimedia, 10MediaWiki-extensions-UploadWizard: UploadWizard does not let you input 'altitude' when uploading a file (geodata) - https://phabricator.wikimedia.org/T41553#1498298 (10matmarex) [13:34:25] (03PS1) 10Bartosz Dziewoński: Remove unused message 'mwe-upwiz-error-date' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228248 [13:34:27] (03PS1) 10Bartosz Dziewoński: Remove commented-out for ages 'altitude' code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228249 (https://phabricator.wikimedia.org/T41553) [13:34:41] Hah, you're having a lot of fun with this :) [13:34:43] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: UploadWizard does not let you input 'altitude' when uploading a file (geodata) - https://phabricator.wikimedia.org/T41553#1498310 (10matmarex) [13:34:45] 6Multimedia, 10MediaWiki-Uploading: MediaWiki api doesn't serve EXIF GPSAltitude (and other tags) as decimals - https://phabricator.wikimedia.org/T34410#375027 (10matmarex) [13:35:11] marktraceur: i'm not going to be refactoring code which does not actually work. [13:35:29] feel like merging some of that? [13:37:14] marktraceur: when we're importing the date from Flickr or EXIF, we currently also import the time, and put it into the date field. is this desirable, or can i ax it? [13:45:25] 6Multimedia, 10Continuous-Integration-Infrastructure, 6operations, 5Patch-For-Review: Investigate impact of switching from ffmpeg to libav (ffmpeg is not in Jessie) - https://phabricator.wikimedia.org/T103335#1498323 (10fgiunchedi) >>! In T103335#1498117, @MoritzMuehlenhoff wrote: >> I gave this a try, so... [13:48:24] (03PS1) 10Bartosz Dziewoński: Never import the time of creation from anywhere, only the date [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228250 [13:49:25] preparing more work for you guys [13:50:18] more money and more people [13:50:53] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 6operations: Support VP9 in TMH (Unable to decode) - https://phabricator.wikimedia.org/T55863#1498329 (10fgiunchedi) with `ffmpeg2theora` 0.29 from debian and an updated `ffmpeg` the video stream is recognized but I'm getting a segfault: ``` filippo@fil... [13:56:51] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Update avconv to support opus in ogg files - https://phabricator.wikimedia.org/T53313#1498348 (10fgiunchedi) with an updated `ffmpeg` from debian this seems to work: ``` filippo@filippo-test-trusty:~$ ffmpeg -y -i Sound_of_the_bells_of_Sveta_Nedelya_in_So... [13:58:07] 6Multimedia, 10Labs-Vagrant, 10MediaWiki-Vagrant, 10MediaWiki-extensions-TimedMediaHandler, 10Wikimedia-Video: Bad bitrate/quality for WebM two-pass encoding with avconv in vagrant - https://phabricator.wikimedia.org/T106612#1498350 (10fgiunchedi) what's the respective ffmpeg/avconv commandline we can us... [14:05:01] MatmaRex: I'll review all of them today [14:05:21] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 6operations: Support VP9 in TMH (Unable to decode) - https://phabricator.wikimedia.org/T55863#1498365 (10MoritzMuehlenhoff) The same segfault happens on when converting Snowdonia_by_drone.webm in Debian unstable, so this isn't limited to the backport, b... [14:09:06] MatmaRex: I think attempting to find the time is not necessary [14:09:12] Shoot. [15:26:43] MatmaRex: is getting a working oojsui date control on your todo list? I saw your comments on Brad's patch and wondered when he might be unblocked for the rewrite of ApiSandbox [15:27:12] bd808: He's working on it now I think. [15:27:23] awesomesauce [15:27:31] MatmaRex: You missed me saying "time is not important for the information date field" when you logged out [15:27:43] Do we need to get you a bouncer? :) [15:28:32] bd808: there is one, just recently merged, but currently in MW under mw.widgets namespace, not in OOUI proper. [15:28:45] marktraceur: oh, thanks. something disconnected me. [15:29:02] marktraceur: I can park stashbot in here to give you a searchable backscroll :) -- https://stashbot.wmflabs.org/#/dashboard/elasticsearch/default [15:29:05] marktraceur: getting a bouncer would mean that IRC messages would turn into yet another TODO list, so… i dunno [15:30:40] MatmaRex: thanks for the update. Brad is going to be taking a well earned vacation for a couple of weeks so it sounds like things may be in good shape for him by the time he gets back [15:31:01] * bd808 manages to act like a manager [15:37:25] MatmaRex: Life is one big TODO list. [15:42:45] 6Multimedia, 6Commons, 10Wikimedia-Media-storage, 7Tracking: Thumbnail/imagescaler (tracking) - https://phabricator.wikimedia.org/T43371#1498601 (10Aklapper) [15:43:23] marktraceur, James_F|Away - there's a relevant-to-yr-interests thread at "[Wikimedia-l] Media handling is difficult" [15:43:55] Ugh foundation-l [15:45:17] quiddity: Yeah, sounds like all of those things are on our radar except maybe the mobile app [15:45:28] But only because I know there are bigger fish to fry at the moment [15:45:45] * quiddity nods [15:51:46] (03CR) 10MarkTraceur: [C: 032] Remove some pointless datepicker code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228243 (owner: 10Bartosz Dziewoński) [15:52:40] (03Merged) 10jenkins-bot: Remove some pointless datepicker code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228243 (owner: 10Bartosz Dziewoński) [15:53:04] (03CR) 10MarkTraceur: [C: 032] "It was only ever meant for use on Commons, I think, so linking to the cluster makes sense. But we can make things Better™." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228244 (owner: 10Bartosz Dziewoński) [15:53:25] (03CR) 10MarkTraceur: [C: 032] Remove unused message 'mwe-upwiz-error-date' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228248 (owner: 10Bartosz Dziewoński) [15:53:56] (03CR) 10MarkTraceur: [C: 032] "etc." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228249 (https://phabricator.wikimedia.org/T41553) (owner: 10Bartosz Dziewoński) [15:54:03] (03Merged) 10jenkins-bot: Use HTTPS for links to Commons and MediaWiki.org [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228244 (owner: 10Bartosz Dziewoński) [15:54:44] (03CR) 10MarkTraceur: [C: 032] Never import the time of creation from anywhere, only the date [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228250 (owner: 10Bartosz Dziewoński) [15:54:55] (03Merged) 10jenkins-bot: Remove unused message 'mwe-upwiz-error-date' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228248 (owner: 10Bartosz Dziewoński) [15:54:58] MatmaRex just wanted to up his number of patches per month. :P [15:56:21] i hate big refactors [15:57:42] (03Merged) 10jenkins-bot: Remove commented-out for ages 'altitude' code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228249 (https://phabricator.wikimedia.org/T41553) (owner: 10Bartosz Dziewoński) [15:57:44] (03Merged) 10jenkins-bot: Never import the time of creation from anywhere, only the date [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228250 (owner: 10Bartosz Dziewoński) [15:58:10] Who doesn't... [15:59:07] i wonder how many lines of UW code i can manage to delete in one day [15:59:17] I wonder what the record for that is. [15:59:31] I'm sure I hold it but I wouldn't mind yielding the title with some copious code review :) [16:04:38] marktraceur: hmm, looks like we could chuck out the entire jquery.pubsub library by using oojs instead. [16:04:52] MatmaRex: I've tried...so hard [16:05:01] :o [16:05:03] But every single time there was too much cruft left laying around for it to make sense [16:05:09] Let me find you one of those patches. [16:05:35] https://gerrit.wikimedia.org/r/146903 [16:05:59] Abandoned, dunno why, but it was definitely painful [16:07:26] hm. okay, will think about it later [16:08:47] (03PS1) 10Bartosz Dziewoński: jquery.morphCrossfade: Do not set 'overflow: hidden' permanently [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228295 [16:08:49] (03PS1) 10Bartosz Dziewoński: [WIP] Use mw.widgets.DateInputWidget [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228296 (https://phabricator.wikimedia.org/T97426) [17:25:57] marktraceur: are UploadWizard's browser tests functional, and should they be kept up to date when i change stuff around? [17:58:25] MatmaRex: Yes! I don't use them for trivial stuff but they're good for when you change user interface stuff [18:08:01] aight [18:51:18] (03PS2) 10Bartosz Dziewoński: Use mw.widgets.DateInputWidget instead of jQuery UI datepicker [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228296 (https://phabricator.wikimedia.org/T97426) [18:51:20] (03PS1) 10Bartosz Dziewoński: Remove no-op method #prefillSource [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228322 [18:54:31] (03PS3) 10Bartosz Dziewoński: Use mw.widgets.DateInputWidget instead of jQuery UI datepicker [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228296 (https://phabricator.wikimedia.org/T97426) [18:55:30] marktraceur: i think that's ready. no idea how it interacts with the browser tests, though. [18:56:34] 6Multimedia, 10MediaWiki-File-management: Include at least some EXIF metadata in resized pictures - https://phabricator.wikimedia.org/T20871#1499283 (10Tgr) [18:56:37] 6Multimedia, 10MediaWiki-Uploading, 7Testme: Image author, description, and copyright data saved in EXIF fields - https://phabricator.wikimedia.org/T5361#1499284 (10Tgr) [18:57:38] 6Multimedia, 10MediaWiki-Uploading, 7Testme: Image author, description, and copyright data saved in EXIF fields - https://phabricator.wikimedia.org/T5361#1499289 (10Tgr) T95217 is the audio version of this task. [19:07:44] (03PS1) 10Bartosz Dziewoński: Use OO.ui.ProgressBarWidget instead of jQuery UI progressbar [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228328 [19:17:10] 6Multimedia, 10MediaWiki-Uploading, 7Testme: Embed image author, description, and copyright data in file metadata fields - https://phabricator.wikimedia.org/T5361#1499329 (10Tgr) [19:18:03] 6Multimedia, 10MediaWiki-File-management: Include at least some EXIF metadata in resized pictures - https://phabricator.wikimedia.org/T20871#1499338 (10Tgr) [19:21:40] 6Multimedia, 10MediaWiki-File-management, 7Testme: Embed image author, description, and copyright data in file metadata fields - https://phabricator.wikimedia.org/T5361#1499354 (10Tgr) [19:22:34] 6Multimedia, 10MediaWiki-File-management, 7Testme: Embed image author, description, and copyright data in file metadata fields - https://phabricator.wikimedia.org/T5361#77258 (10Tgr) [19:30:05] (03PS2) 10Bartosz Dziewoński: Use OO.ui.ProgressBarWidget instead of jQuery UI progressbar [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228328 [19:34:36] (03PS1) 10Bartosz Dziewoński: Remove some dead commented-out code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228346 [20:12:29] (03PS1) 10Bartosz Dziewoński: mw.UploadWizardUpload: Replace some pubsub usage, use promises [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/228400 [20:16:20] You're a brave man. [20:16:36] MatmaRex: I thought for sure that I couldn't get rid of parts, that I had to do all of it at once [20:17:16] marktraceur: looks like we were using it for two mostly distinct things [20:17:27] marktraceur: this was the easier one to disentangle, though [20:17:50] i can't even trace where the pubs and subs for the second one come from yet [20:18:27] anyway, i think this is a good cliffhanger to end the week on. ;) [20:19:10] :-) [20:20:06] 6Multimedia, 10MediaWiki-Uploading: MediaWiki api doesn't serve EXIF GPSAltitude (and other tags) as decimals - https://phabricator.wikimedia.org/T34410#1499471 (10Bawolff) Honestly, I didn't really think its a bug in the first place. Exif tags that are "rationals" according to the spec (with exception to lat... [20:22:05] Which one did you do first, MatmaRex? [20:22:10] The thumbnail one is the pain in the ass I think [20:22:17] I don't remember what the other stupid bloody thing was [20:22:35] marktraceur: Thumnails. [20:22:37] marktraceur: the two were the thumbnail one, and an inner-thumbnail-magic one. i did the inner-thumbnail-magic in that patch. [20:22:45] Oh. [20:23:02] * James_F stops trying to be helpful. :-) [20:23:51] OK. [20:24:01] MatmaRex: I'll see what I can see. [20:39:37] 6Multimedia, 10MediaWiki-Unit-tests, 10MediaWiki-Uploading, 7Regression: mediawiki.api.upload.test doesn't tear down spurious