[00:14:13] (03PS1) 10Robmoen: Add missing messages to RL module [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/120968 [01:30:59] (03PS3) 10Aarcos: [WIP] Phase 1: Download tab [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 [01:31:37] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Phase 1: Download tab [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos) [01:34:02] (03PS4) 10Aarcos: [WIP] Phase 1: Download tab [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 [01:34:43] (03PS1) 10Gergő Tisza: Store event handler proxy for size change [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120979 [01:35:18] (03CR) 10jenkins-bot: [V: 04-1] Store event handler proxy for size change [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120979 (owner: 10Gergő Tisza) [01:37:38] (03CR) 10Aarcos: "Just an update on what I am doing. Still lots to go but feedback on the user experience and general structure of the code are welcome." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos) [01:47:30] (03PS2) 10Gergő Tisza: Store event handler proxy for size change [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120979 [10:25:49] (03CR) 10Gilles: Makes the size information (WxH) in Embed lighter (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120531 (owner: 10Gilles) [10:33:01] (03PS6) 10Gilles: Makes the size information (WxH) in Embed lighter [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120531 [10:47:44] (03CR) 10Gilles: [C: 04-1] "Still waiting for https://gerrit.wikimedia.org/r/#/c/120685/" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120531 (owner: 10Gilles) [11:03:32] (03CR) 10Gilles: [C: 032] Load MMV scripts for everyone; handle hashes [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120718 (owner: 10MarkTraceur) [11:04:09] (03Merged) 10jenkins-bot: Load MMV scripts for everyone; handle hashes [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120718 (owner: 10MarkTraceur) [11:07:28] (03CR) 10Gilles: [C: 032] Store event handler proxy for size change [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120979 (owner: 10Gergő Tisza) [11:08:07] (03Merged) 10jenkins-bot: Store event handler proxy for size change [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120979 (owner: 10Gergő Tisza) [11:12:02] (03CR) 10Gilles: [C: 04-1] Sanitize embed HTML (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120953 (owner: 10Gergő Tisza) [11:12:44] (03CR) 10Gilles: [C: 032] Add missing messages to RL module [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/120968 (owner: 10Robmoen) [11:12:54] (03Merged) 10jenkins-bot: Add missing messages to RL module [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/120968 (owner: 10Robmoen) [11:20:43] (03CR) 10Siebrand: [C: 031] "i18n/L10n reviewed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120531 (owner: 10Gilles) [11:33:17] (03CR) 10Siebrand: "i18n/L10n reviewed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos) [11:33:22] (03CR) 10Siebrand: [C: 031] [WIP] Phase 1: Download tab [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos) [11:48:21] (03PS1) 10Siebrand: Migrate to JSON i18n [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/121028 [12:15:38] (03CR) 10Gilles: [C: 04-1] "Some feedback on the user experience with Firefox:" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos) [12:44:57] (03CR) 10Gilles: "Some comments about the code" (037 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos) [13:02:33] (03CR) 10Gilles: [C: 04-1] Add more information to embed HTML (032 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120709 (owner: 10Gergő Tisza) [13:17:25] Hmm, after writing that all out, perhaps bug 32387 really is not an "easy" bug [13:20:13] *sigh* its really hard to find easy multimedia bugs [13:35:14] (03PS1) 10Gilles: Add explanatory one-liners to share & embed [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121052 [13:49:08] (03CR) 10Siebrand: [C: 031] "i18n/L10n reviewed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121052 (owner: 10Gilles) [14:01:02] (03CR) 10Raimond Spekking: [C: 032] Migrate to JSON i18n [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/121028 (owner: 10Siebrand) [14:01:08] (03Merged) 10jenkins-bot: Migrate to JSON i18n [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/121028 (owner: 10Siebrand) [14:02:57] (03PS1) 10Amire80: Use the new way of specifying beta screenshots [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121057 [14:22:33] (03PS1) 10Gilles: Select html embed by default for logged-out users [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121064 [14:50:39] (03PS1) 10Gilles: Increase the contrast of the share&embed input/textarea [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121074 [16:56:07] (03CR) 10Aarcos: [C: 031] "LGTM, just a cosmetic issue, merge at your convenience providing no other comments." (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121052 (owner: 10Gilles) [17:06:17] (03CR) 10Aarcos: [C: 04-1] Select html embed by default for logged-out users (032 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121064 (owner: 10Gilles) [17:31:36] (03CR) 10Gilles: Select html embed by default for logged-out users (032 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121064 (owner: 10Gilles) [18:11:12] marktraceur: can you review the changes on https://gerrit.wikimedia.org/r/#/c/119981/ ? fabrice asked us to deploy it this sprint [18:11:18] -.- [18:12:11] tgr: $wgDefaultUserOptions['multimediaviewer-enable'] = true; // I thought this was already the case based on the pref definition? [18:12:32] Oh, you moved it. [18:12:35] yes, but that had issues [18:12:43] see the linked bugs [18:13:19] Hrm [18:13:22] um, the links might be hidden somwhere in the PS1 comments... [18:13:46] https://bugzilla.wikimedia.org/show_bug.cgi?id=62946 [18:13:49] https://bugzilla.wikimedia.org/show_bug.cgi?id=63033 [18:15:43] Yeah [18:16:45] Well, I believe you anyway [18:17:09] (03CR) 10MarkTraceur: [C: 031] "Seems good to me, ta." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/119981 (owner: 10MarkTraceur) [18:17:34] i.e. changes you made seem fine; I can +2 if you prefer [18:18:12] thx [18:26:21] (03CR) 10Gergő Tisza: [C: 032] Add user preference for non-beta disabling [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/119981 (owner: 10MarkTraceur) [18:27:28] (03Merged) 10jenkins-bot: Add user preference for non-beta disabling [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/119981 (owner: 10MarkTraceur) [18:49:40] Hi umang13 [19:32:27] Hi [19:32:45] Hello umang13 [19:33:57] umang13: Sorry I don't have any easy bugs for you. People went and fixed them all :s [19:34:09] The bug I was working on last night got a patch today by dav. I should probably move on? [19:34:59] Yeah I think so [19:36:11] generally the first person to submit a patch (unless something is really wrong with it) is the person who gets their patch merged [19:39:53] Sorry, could not recieve previous messages(if any), some problem with freenode I guess [19:40:09] [17:38] Yeah I think so [19:40:11] [17:39] generally the first person to submit a patch (unless something is really wrong with it) is the person who gets their patch merged [19:40:49] ok, so any new bugs (even non-multimedia ) you would suggest? [19:40:49] umang13: The web irc interface is really bad. I reccomend downloading a "real" irc program (I personally use chatzilla, which is a firefox extension) [19:41:08] ah, will do that :) [19:41:35] The bugs on this list are supposed to be straightforward https://bugzilla.wikimedia.org/buglist.cgi?keywords=easy&resolution=---&list_id=292978 [19:42:46] Sure, and you suggest that my proposal should be more detailled? [19:43:45] I personally think it would be good to have some information on what files of these types exist [19:44:10] It would help justify the use case, although that's a minor concern [19:44:51] Ok, I will look into it. Apart from this any other suggestions from your side? [19:45:02] The biggest issue with your proposal is lack of mentors. Projects with two mentors are considered before other projects [19:45:54] yeah, i was coming to that. As of yet I have talked to Bryan Davis. But he is primary mentor for some other project and thus can only take secondary mentorship here [19:46:01] I talked about this to Quim [19:46:15] So the best thing you could do would be to try to attract more mentors (somehow, I don't really got any good advice on how to accomplish that), and try and get a a patch merged into mediawiki [19:46:37] he said not to worry as this was a project featured by the community itself. However, since monday is the final date, now this concerns me [19:47:25] Quim, is the ultimate authority on gsoc, so he knows what he is talking about [19:47:50] But, still there needs to be a willing mentor?? [19:48:28] ultimately yes [19:48:52] And further, if I need to check the microtasks, they need to be given to me by someone [19:49:05] the tasks you gave were unfortunately taken [19:49:19] So, who will assign me micro tasks? [19:50:01] umang13: How about https://bugzilla.wikimedia.org/show_bug.cgi?id=60562 [19:50:41] has this not been assigned to micheal dale? [19:50:46] Does that sound interesting to you. If so I'll mark it assigned to you so nobody steals it [19:51:09] Oh for historical reasons, everything TMH is assigned to micheal dale, you can ignore that for TimedMediaHandler bugs [19:51:33] So that task is indeed open [19:51:53] Oh ok thanks. I at the moment do not have much idea what it is about. I will have to look into it [19:52:03] will do so [19:52:18] Just a second, I'll see if I can find an example [19:52:53] sure. and one more thing, if it is not informal, have you taken mentorship for any project in Gsoc? [19:57:44] umang13: Sorry, I froze my browser trying to load the example ;) [19:58:02] you got my last message?? [19:58:26] umang13: https://commons.wikimedia.org/wiki/File:2013-11-22_Ein_Entz%C3%BCcklop%C3%A4discher_Abend.webm if you click on the "Error in transcode on 2014-02-15T02:06:28" it will show a lot of text (or outright freeze your browser) [19:58:41] It should chop out the middle, because showing that much text is just annoying [19:59:22] Sorry, but I'm not being a mentor in GSOC this year [20:00:06] Ah, so basically you want to shorten this error message? [20:00:53] Yeah, it should say something like the first 1000 bytes, and the last 1000 bytes, but if its longer then 2000 bytes, the middle should be cut out [20:01:16] ok, yeah I guess this should be easy. [20:02:41] Ok, assigned to you [20:03:20] Thanks [20:04:17] As a hint, you're going to want to look at WebVideoTranscode/WebVideoTranscodeJob.php in TimedMediaHandler extension [20:04:58] thanks again! [20:05:34] * marktraceur works on core docs with Krinkle [20:05:44] No problem, if you run into any trouble don't hesitate to ask [20:05:54] Sure :) [20:06:04] marktraceur: documentation! that's just crazy talk [20:08:55] bawolff: I must be new here [20:09:24] marktraceur: next you're going to want clean interfaces between logically separate code. Where will it end? [20:13:44] Crazy talk [20:46:19] *sigh*, why are there 6736 queued video transcodes, but 0 running transcodes :S [21:04:10] (03CR) 10Aaron Schulz: [C: 032] Set X-Content-Duration header on ogg transcodes. [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/120185 (owner: 10Brian Wolff) [21:04:38] (03Merged) 10jenkins-bot: Set X-Content-Duration header on ogg transcodes. [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/120185 (owner: 10Brian Wolff) [22:37:00] where do I dump my database from for testing purposes? [22:37:32] Generally people just create a new wiki for testing purposes [22:37:55] They don't generally import a database, as that takes forever for any medium-large sized db [22:38:04] and usually is not as useful as it sounds [22:38:16] So, how do I reproduce this bug to test it? [22:38:52] You can upload the file to your wiki (Just download the original file, then upload at Special:Upload) [22:39:15] Or you can make timed media handler call a shell script that outputs a lot of text instead of calling ffmpeg2theora [22:41:13] That is by setting $wgFFmpeg2theoraLocation in LocalSettings.php to some shell script