[07:15:39] (03CR) 10Legoktm: [C: 032] Api: Cleanup videoinfo implementation [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/283872 (owner: 10TheDJ) [07:30:43] (03Merged) 10jenkins-bot: Api: Cleanup videoinfo implementation [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/283872 (owner: 10TheDJ) [08:50:58] 06Multimedia, 10Possible-Tech-Projects, 10Wikimedia-General-or-Unknown: Implement International Image Interoperability Framework (IIIF) prototype service on Wikimedia labs - https://phabricator.wikimedia.org/T89552#2213479 (10GAllegre) Thanks a lot! [09:27:41] 06Multimedia, 06Commons, 06Editing-Department, 10MediaWiki-File-management, and 4 others: Allow uploading of 3D files to Wikimedia Commons - https://phabricator.wikimedia.org/T3790#2213678 (10Qgil) >>! In T3790#2176135, @Gilles wrote: > Backend thumbnail renderer: https://github.com/gi11es/3d2png > Extensi... [13:29:48] 06Multimedia, 06Commons, 10MediaWiki-Uploading, 10Possible-Tech-Projects, 07Community-Wishlist-Survey: Add support for KML/KMZ filetype - https://phabricator.wikimedia.org/T28059#2214018 (10Qgil) Hi @putnik, did you work on this task at the Hackathon? [14:07:31] 06Multimedia, 06Commons, 06Operations, 10Traffic: Commons API fails (413 error) to upload file within 100MB threshold - https://phabricator.wikimedia.org/T86436#2214064 (10Slowking4) this is a problem for uploading books with IA moving away from dejavu the bloated jpegs will be larger important copyright... [15:04:20] (03PS1) 10Bartosz Dziewoński: Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 [15:04:22] (03PS1) 10Bartosz Dziewoński: Simplify some error logging code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283982 [15:04:24] (03PS1) 10Bartosz Dziewoński: mw.UploadWizardUploadInterface: Remove unused method [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283983 [15:04:26] (03PS1) 10Bartosz Dziewoński: Use 'csrf' token name rather than 'edit' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283984 [15:07:08] (03CR) 10jenkins-bot: [V: 04-1] Use 'csrf' token name rather than 'edit' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283984 (owner: 10Bartosz Dziewoński) [15:07:27] (03CR) 10jenkins-bot: [V: 04-1] Simplify some error logging code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283982 (owner: 10Bartosz Dziewoński) [15:07:44] (03CR) 10jenkins-bot: [V: 04-1] Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 (owner: 10Bartosz Dziewoński) [15:09:39] (03CR) 10jenkins-bot: [V: 04-1] mw.UploadWizardUploadInterface: Remove unused method [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283983 (owner: 10Bartosz Dziewoński) [15:10:00] eh [15:10:01] 06Multimedia, 10UploadWizard: UploadWizard doesn't handle errors caused by upload rate limiting - https://phabricator.wikimedia.org/T132920#2214156 (10matmarex) [15:32:17] 06Multimedia, 06Commons, 06Operations, 10Traffic: Commons API fails (413 error) to upload file within 100MB threshold - https://phabricator.wikimedia.org/T86436#2214250 (10Slowking4) ok i have uploaded this 69.29 MB volume using metadata from IA uploader and uploading using chunked uploads. could someone p... [16:00:40] o/ [16:09:16] Hey prtksxna [16:10:26] (03CR) 10Anomie: "See also T89971." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/283872 (owner: 10TheDJ) [16:12:29] MarkTraceur: ruecarTkraM [16:14:34] prtksxna: Indeed. [16:37:23] 06Multimedia, 10UploadWizard: UploadWizard doesn't handle errors caused by upload rate limiting - https://phabricator.wikimedia.org/T132920#2214474 (10matmarex) [17:08:52] (03CR) 10Brion VIBBER: [C: 04-2] "Bitrates get pretty high to maintain quality in Theora at 720p; I don't think it's worth producing these." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/283868 (owner: 10Paladox) [17:13:44] 06Multimedia, 10ImageTweaks, 10Security-Reviews, 06Security-Team: Security review of the ImageTweaks extension ahead of production deployment - https://phabricator.wikimedia.org/T123594#2214629 (10dpatrick) a:03dpatrick [17:26:39] 06Multimedia, 10ImageTweaks, 10Security-Reviews, 06Security-Team: Security review of the ImageTweaks extension ahead of production deployment - https://phabricator.wikimedia.org/T123594#2214687 (10dpatrick) == General Observations == Excellent developer documentation in client-side code. == Issues == No i... [18:01:00] (03CR) 10MarkTraceur: [C: 032] mw.DestinationChecker: Check for non-exact extension matches too [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283005 (https://phabricator.wikimedia.org/T132356) (owner: 10Bartosz Dziewoński) [18:01:54] (03PS2) 10MarkTraceur: Use 'disclaimerpage' localisation message rather than literal 'Project:General disclaimer' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283759 (owner: 10Bartosz Dziewoński) [18:02:00] (03CR) 10MarkTraceur: [C: 032] Use 'disclaimerpage' localisation message rather than literal 'Project:General disclaimer' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283759 (owner: 10Bartosz Dziewoński) [18:03:26] (03Merged) 10jenkins-bot: mw.DestinationChecker: Check for non-exact extension matches too [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283005 (https://phabricator.wikimedia.org/T132356) (owner: 10Bartosz Dziewoński) [18:05:05] (03Merged) 10jenkins-bot: Use 'disclaimerpage' localisation message rather than literal 'Project:General disclaimer' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283759 (owner: 10Bartosz Dziewoński) [18:30:44] (03CR) 10MarkTraceur: [C: 032] Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 (owner: 10Bartosz Dziewoński) [18:32:55] (03CR) 10jenkins-bot: [V: 04-1] Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 (owner: 10Bartosz Dziewoński) [18:33:37] Jenkins wants you to remove even more code. :) [18:33:46] yeah, i'll fix later [18:34:24] (03CR) 10MarkTraceur: [C: 032] Simplify some error logging code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283982 (owner: 10Bartosz Dziewoński) [18:35:59] (03PS2) 10Bartosz Dziewoński: mw.UploadWizardUploadInterface: Remove unused method [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283983 [18:36:01] (03PS2) 10Bartosz Dziewoński: Simplify some error logging code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283982 [18:36:03] (03PS2) 10Bartosz Dziewoński: Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 [18:36:05] (03PS2) 10Bartosz Dziewoński: Use 'csrf' token name rather than 'edit' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283984 [18:36:10] (03CR) 10jenkins-bot: [V: 04-1] Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 (owner: 10Bartosz Dziewoński) [18:36:12] (03CR) 10jenkins-bot: [V: 04-1] Simplify some error logging code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283982 (owner: 10Bartosz Dziewoński) [18:36:54] (03CR) 10MarkTraceur: [C: 032] Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 (owner: 10Bartosz Dziewoński) [18:37:11] (03PS1) 10TheDJ: [DO NOT MERGE] alpha 1.1.0 of ogv.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/284016 [18:38:37] (03CR) 10jenkins-bot: [V: 04-1] Simplify some error logging code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283982 (owner: 10Bartosz Dziewoński) [18:38:40] (03CR) 10MarkTraceur: [C: 032] mw.UploadWizardUploadInterface: Remove unused method [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283983 (owner: 10Bartosz Dziewoński) [18:39:06] (03CR) 10MarkTraceur: [C: 032] Use 'csrf' token name rather than 'edit' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283984 (owner: 10Bartosz Dziewoński) [18:39:39] (03Merged) 10jenkins-bot: Remove some more old cruft related to forms and iframes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283981 (owner: 10Bartosz Dziewoński) [18:39:41] (03Merged) 10jenkins-bot: Simplify some error logging code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283982 (owner: 10Bartosz Dziewoński) [18:42:25] (03Merged) 10jenkins-bot: mw.UploadWizardUploadInterface: Remove unused method [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283983 (owner: 10Bartosz Dziewoński) [18:42:55] (03CR) 10jenkins-bot: [V: 04-1] Use 'csrf' token name rather than 'edit' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283984 (owner: 10Bartosz Dziewoński) [18:47:12] (03CR) 10Paladox: [C: 031] "We should also update the videojs-ogv plugin too please." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/284016 (owner: 10TheDJ) [20:10:41] 06Multimedia, 06Commons, 10Wikimedia-Site-requests: Convert Commons filter 140 to a $wgRateLimits configuration - https://phabricator.wikimedia.org/T132930#2215442 (10matmarex) [20:26:32] (03CR) 10Bartosz Dziewoński: [C: 032] "Ugh, I guess the ConcurrentQueue tests are flappy after all :(" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283984 (owner: 10Bartosz Dziewoński) [20:30:24] (03Merged) 10jenkins-bot: Use 'csrf' token name rather than 'edit' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/283984 (owner: 10Bartosz Dziewoński) [20:38:57] (03CR) 10Jforrester: [C: 031] Use $.makeCollapsible instead of custom stuff [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/280717 (https://phabricator.wikimedia.org/T131202) (owner: 10Bartosz Dziewoński) [20:48:46] 06Multimedia, 10UploadWizard: UploadWizard reports uploads blocked by AbuseFilter as 'verification-error' ("This file might be corrupt, or have the wrong extension") - https://phabricator.wikimedia.org/T132866#2215617 (10matmarex) a:03matmarex [20:57:19] 06Multimedia, 10MediaWiki-Uploading: UploadWizard rejects all uploads with false error message after a certain number of uploads - https://phabricator.wikimedia.org/T121896#2215645 (10matmarex) I found out that there is a throttle, sneakily implemented as an AbuseFilter filter ([[https://commons.wikimedia.org/... [20:57:29] 06Multimedia, 10UploadWizard: UploadWizard doesn't handle errors caused by upload rate limiting - https://phabricator.wikimedia.org/T132920#2214156 (10matmarex) [20:57:32] 06Multimedia, 06Commons, 10Wikimedia-Site-requests: Convert Commons filter 140 to a $wgRateLimits configuration - https://phabricator.wikimedia.org/T132930#2214457 (10matmarex) [20:57:33] 06Multimedia, 10MediaWiki-Uploading: UploadWizard rejects all uploads with false error message after a certain number of uploads - https://phabricator.wikimedia.org/T121896#2215652 (10matmarex) [21:03:04] 06Multimedia, 10UploadWizard: UploadWizard doesn't handle errors caused by upload rate limiting - https://phabricator.wikimedia.org/T132920#2215672 (10matmarex) a:03matmarex [21:06:22] 06Multimedia, 10MediaWiki-Uploading: Create check hook before really uploading file with infos about description page and file - https://phabricator.wikimedia.org/T89302#2215688 (10matmarex) [21:06:25] 06Multimedia, 06Commons, 10MediaWiki-Uploading, 10MediaWiki-extensions-AbuseFilter: AbuseFilter should not check upload stash or at least not log it - https://phabricator.wikimedia.org/T87381#2215687 (10matmarex) [21:10:16] 06Multimedia, 06Commons, 10MediaWiki-Uploading, 10MediaWiki-extensions-AbuseFilter: AbuseFilter should not check upload stash or at least not log it - https://phabricator.wikimedia.org/T87381#990162 (10matmarex) Note that it would still be sometimes useful to check uploads going into stash, and not only wh... [21:14:24] 06Multimedia, 06Commons, 10MediaWiki-Uploading, 10MediaWiki-extensions-AbuseFilter: ABF is ignoring wikitext during upload - https://phabricator.wikimedia.org/T89252#2215720 (10matmarex) [21:18:28] 06Multimedia, 06Commons, 10MediaWiki-Uploading, 10MediaWiki-extensions-AbuseFilter: ABF is ignoring wikitext during upload - https://phabricator.wikimedia.org/T89252#2215745 (10matmarex) p:05High>03Normal a:03matmarex I'd like to work on this at some point. No promises as to when it happens (I have a... [21:21:48] brion: so should volume be working now ? [21:22:24] brion: cause in my test both volume and mute seem to be broken now :( Which can definetly be my fault. [21:28:11] brion: both Chrome and Safari [21:39:51] hmmm [21:40:00] thedj: link? [21:40:11] local [21:40:53] mute's still working for me at https://brionv.com/misc/ogv.js/demo2/, should be the latest commit from last night / this morning / whenever i updated it [21:41:13] i can set the property, i get the event. but the volume stays the same [21:41:26] maybe i compiled something incorrectly [21:41:44] make sure the old ogv.js isn't cached? [21:41:54] hm, no wait, if you get the event .... wtf? [21:42:02] ah [21:42:08] thedj: you may have to rerun npm install to update audio-feeder [21:42:16] ah [21:42:34] otherwise it'll be setting a null-op property on the old AudioFeeder class :D [21:43:35] i think that might be it [21:56:29] br [21:56:50] yup, that was the problem [22:01:09] :D [22:04:58] brion: will publish my updated tech module for videojs in an hour or so [22:11:03] brion: found a small bug [22:16:48] 06Multimedia, 10Analytics, 06Commons, 10Wikidata, and 3 others: Allow tabular datasets on Commons (or some similar central repository) (CSV, TSV, JSON, XML) - https://phabricator.wikimedia.org/T120452#2215951 (10Legoktm) [22:21:20] Backing out of the standup today...no major updates, have to run some errands now [22:57:22] Boo. [22:59:39] I don't have much of an update either, could skip [23:00:48] MatmaRex, brion: You joining? [23:01:53] uh [23:02:05] James_F: i'm still running that npm install. it's taking 100% CPU. so not really [23:02:16] Wait, really? [23:02:16] i have nothing to report. did some boring UW maintenance. [23:02:18] yes [23:02:26] really [23:02:27] Sounds like an npm issue on Windows maybe? [23:02:29] it's ridiculous [23:02:39] sounds like we have a jillion of useless dependencies [23:02:45] Not /that/ many. [23:02:50] *that* many. [23:02:57] It takes me about four minutes to re-install from scratch. [23:03:01] also, npm is a shit package manager. [23:03:10] Plausibly. [23:04:00] OK, it's over now. [23:04:07] Update: Everyone's not got much to report. EOM. [23:12:30] brion: https://www.npmjs.com/package/videojs-ogvjs [23:13:27] brion: that version should be ogvjs 1.1.0 compatible (though i did not include the alpha version into it) [23:14:46] woot [23:15:00] James_F: dammit I need to update my calendar alarm [23:15:14] I wlays get a ping 10 minutes before, then get distracted for 25 minutes [23:16:41] story of my life. [23:16:52] :) [23:17:09] but I usually notice at 4 minutes late [23:17:26] yeah sometimes I remember in time but y'all are pretty good about closing out quickly :) [23:21:22] thedj: seeking seems pretty flaky, i'll have to check that out [23:22:54] brion: in the new version ? [23:23:21] thedj: in the demo link, haven't tried locally yet [23:23:31] https://cdn.rawgit.com/hartman/videojs-ogvjs/v1.1.0/example.html [23:24:02] oh, the example isn't running ogv.js 1.1.0 [23:24:20] it's just compatible with it, i'm not distributing it yet [23:24:42] well seek code hasn't changed much yet, so i'm betting that's still broken [23:25:52] there are nights that I dream about Folgers coffee [23:26:22] that's been my OGV test movie ever since mdale uploaded it :) [23:26:50] James_F: whoooo it finished [23:27:38] in my vlc days tarzan.ogm was my test movie. i still know every line of it. [23:27:41] http://streams.videolan.org/streams/ogm/tarzan.ogm [23:28:08] probably the source file doesn't have a skeleton index and it's doing the bisect seek, and something's messing up the state [23:28:17] it kinda looks like it's playing back during bisection o_O [23:28:22] heh [23:28:42] brion: looking back for the previous iframe or something ? [23:29:38] thedj: while first implementing a/v sync I played https://commons.wikimedia.org/wiki/File%3ASneak_Preview_-_Wikipedia_VisualEditor.webm a bajillion times [23:29:46] brion: :-) [23:29:49] MatmaRex: Yay? [23:30:58] thedj: basically; if there's no keyframe index available, you have to search in the ogg stream by just popping into semi-random places in the stream, reading until you sync up to a packet, and checking its timestamp. then you do the same thing again looking for the specific timestamp of the previous keyframe [23:31:06] it's the most inefficient thing EVER over a network [23:31:30] *and* I've implemented it fairly naively in terms of throwing away cached data [23:32:08] yeah, there is a reason why skeleton indexes were added to ogg :) [23:32:14] thedj: it might run better with the transcode - https://upload.wikimedia.org/wikipedia/commons/transcoded/a/ae/Folgers_commercial.ogg/Folgers_commercial.ogg.360p.ogv [23:32:23] which should have the index [23:35:37] brion: seems a bit better indeed [23:36:21] in any case I need to make sure it doesn't explode horribly from scrubbing in the file [23:37:07] i'm betting the scrubbing fires off a currentTime = or fastSeek() continuously, while my demo's controls and our old kaltura player only fire it when you release iirc [23:40:16] 06Multimedia, 06Commons, 10MediaWiki-Database, 10MediaWiki-File-management: image.img_bits should be consistent - https://phabricator.wikimedia.org/T132986#2216285 (10Bawolff) Maybe we should reevaluate if we should store it at all. Its incorrect for many formats and basically not used. [23:40:50] thedj: Wednesday I fly out to berlin so my schedule'll be even wackier than usual; what's the biggest remaining thing you need from me Tuesday? [23:42:43] i'll probably try to finish up the src/load/play cycle [23:42:49] since we need that for source switching [23:42:56] and try to figure out that seek bug [23:43:12] and otherwise keep cleaning up the packaging :) [23:47:14] yeah src/load/play cycle is my biggest item i think [23:51:14] brion: nite from this side of the world [23:52:03] dream of folgers coffee [23:59:17] brion: ps, are ogv-version and/or ogv-support supposed to be in dist ? because they aren't for me. [23:59:59] Hmm they should be