[00:00:08] anyway, as I said, we actually measured this [00:00:15] https://upload.wikimedia.org/wikipedia/commons/thumb/1/1f/Stora_Kronan.jpeg/429px-Stora_Kronan.jpeg was generated half an hour ago [00:00:17] as opposed to arguing about it :) [00:00:28] So it was already pre-generated when you viewed it [00:00:33] tgr: Okay, if your measurements don't show that Media Viewer is slower, than the measurements are wrong. :-) [00:00:34] MediaViewer was about three times faster in the tests [00:00:50] bawolff: So it's faster... [00:00:51] So any slowness there is not from lack of thumb pre-generation (although it could be in other cases) [00:00:52] bawolff: That was my point. [00:00:54] Carmela: not impossible, measuring is hard [00:01:05] bawolff: The 800px version came from MV. [00:01:08] oh, whoops [00:01:11] let me try that again [00:01:28] And it's a larger image, because apparently screen size correlates to bandwidth(???). [00:01:47] Carmela: The media viewer one was generated 5 hours ago [00:01:57] Age: 20015 [00:02:37] Then you're probably dealing with throughput. [00:02:54] Assuming a larger monitor has a faster connection is... err... interesting. [00:03:34] Hmm, you're right. 1.2 seconds vs 0.3 seconds [00:03:54] However, if the goal is to view a bigger sized file, surely you want to view the bigger one? [00:03:58] Probably an imperceptible difference. ;-) [00:04:14] I'm still unclear on what the goals are. [00:04:27] actually, that's a bigger difference then I would expect, coming from varnish cache on both of them [00:04:35] I assume a goal was not to make the site feel slower. [00:05:07] bawolff: try to measure the full loading time from when you click on the image, though [00:05:17] Perhaps a slower site increases user retention metrics. ;-) [00:05:21] for a file page that tends to be multiple seconds [00:06:02] then you navigate back to the article, again several seconds... [00:06:36] At some point the user learns to stop clicking images. [00:06:40] Because it's just a slow headache. [00:07:00] Arguably, user preception would be that MV slower in that case. 1.2 seconds of loading where user could read nothing vs possibly more seconds, but smaller time between when page changes and image shows up [00:07:39] Well if you're going to argue editor retention, the read only nature of media viewer is probably a stronger argument [00:08:21] The user retention bit was a joke. [00:09:14] tgr: Well on a cached hit (viewing both the media viewer view, and the image page view previously), Media viewer does seem ancedotally slower [00:09:42] * bawolff wonders if there's a button somewhere in chrome dev toolbar to kill cache [00:10:07] in the config menu [00:13:43] yeah, sorry. Anecdotally I'm going to have to agree with Carmela that file description page feels faster. Although both feel like they have acceptable performance to me [00:50:45] As far as I can tell, file description pages are loading smaller images that are more likely to be cached. [00:50:55] It doesn't seem surprising that file description pages are faster. [00:51:39] cached --> pre-generated [00:51:48] The image scalers can be slow. [01:43:01] 3MediaWiki extensions / 3MultimediaViewer: Author and source fields are sometimes not emptied - 10https://bugzilla.wikimedia.org/67014 (10Tisza Gergő) 3NEW p:3Unprio s:3normal a:3None Author and source fields are sometimes not emptied. When this happens on prev/next, and the new image has no author/s... [01:46:14] 3MediaWiki extensions / 3MultimediaViewer: Author and source fields are sometimes not emptied - 10https://bugzilla.wikimedia.org/67014#c1 (10Tisza Gergő) The steps to reproduce are pretty strange - see this comment for details: https://www.mediawiki.org/w/index.php?title=Talk%3AMultimedia%2FAbout_Media_Viewe... [01:48:03] fabriceflorin: 1126/336743 ~ 33% [01:48:09] 0.33% I mean [01:48:24] Thanks, I will update the spreadsheet. :) [01:50:31] Carmela: on my machine, a random Commons file page takes ~3.5s to load [01:50:44] of that, 600ms is the actual thumbnail [01:51:03] so there is a lot to win by cutting that out [01:54:08] OTOH when you hit the scalers and the image is large, that will be horribly slow indeed [01:54:29] that will be fixed by pre-rendering thumbnails, which is in the works [01:56:15] tgr: What happens if you disable JS? :-) [01:56:34] with the page load times you mean? [01:56:36] 3.5s sounds high, but Commons is notoriously loose with site-wide scripts and gadgets. [01:56:39] Yeah. [01:58:26] yeah, a lot faster [01:58:36] still dominated by the page, though [01:58:56] this time, 1.5s, of that 200 ms is the thumbnail, 600ms is the HTML file [02:37:47] (03CR) 10Gergő Tisza: [C: 04-1] "It feels a bit off that the text is selected when you use the HTML/plain buttons, but not selected initially. Also, you shouldn't believe " [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/139295 (owner: 10MarkTraceur) [07:47:42] (03PS3) 10Inchikutty: [WIP] Adds OSM parameter to coordinate template [extensions/UploadWizard] (osm) - 10https://gerrit.wikimedia.org/r/141122 [07:47:48] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Adds OSM parameter to coordinate template [extensions/UploadWizard] (osm) - 10https://gerrit.wikimedia.org/r/141122 (owner: 10Inchikutty) [14:38:59] 3MediaWiki / 3Uploading: JSON File upload not working: "File ending .json does not fit MIME type (text/plain)" - 10https://bugzilla.wikimedia.org/66036#c2 (10sreutenauer) 5UNCO>3RESO/FIX Hello, No, it dit not list "text/plain json" I just added it and now it works fine. [15:07:44] 3MediaWiki / 3Uploading: JSON File upload not working: "File ending .json does not fit MIME type (text/plain)" - 10https://bugzilla.wikimedia.org/66036#c3 (10Andre Klapper) 5RESO/FIX>3RESO/INV Glad it works for you! Resetting this ticket to INVALID as no code change was required. [15:49:14] 3MediaWiki / 3Uploading: text like formats (json,csv,etc) misdetected as text/plain by mw - 10https://bugzilla.wikimedia.org/66036#c4 (10Bawolff (Brian Wolff)) 5RESO/INV>3REOP This is technically the wrong solution. We should fix mime type detection for plain text like formats. [16:44:04] (03PS4) 10Raimond Spekking: Support Creative Commons cc-by-(sa-)4.0 [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/138183 [16:51:08] (03PS2) 10Zfilipin: Using new Cucumber browser tags [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141453 [16:51:20] (03PS3) 10Zfilipin: Using new Cucumber browser tags [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141453 [16:51:53] (03CR) 10Zfilipin: [C: 032] Using new Cucumber browser tags [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141453 (owner: 10Zfilipin) [16:52:22] (03Merged) 10jenkins-bot: Using new Cucumber browser tags [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141453 (owner: 10Zfilipin) [17:23:45] 3MediaWiki extensions / 3UploadWizard: Chunked upload fails with internal_api_error_UploadStashFileNotFoundException - 10https://bugzilla.wikimedia.org/36587#c78 (10Greg Grossmeier) a:5Gilles Dubuc>3None Gilles: I'm resetting assignee for now. Should the priority be lowered as well (there hasn't been any... [17:24:45] (03CR) 10Ori.livneh: [C: 032] "Thanks!" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141582 (https://bugzilla.wikimedia.org/66938) (owner: 10Brian Wolff) [17:25:11] (03Merged) 10jenkins-bot: Make unit tests be skipped if no network. [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141582 (https://bugzilla.wikimedia.org/66938) (owner: 10Brian Wolff) [17:28:45] 3MediaWiki extensions / 3UploadWizard: Template redirects can't be used as custom licenses in UW, for example, {{PD-simple}} - 10https://bugzilla.wikimedia.org/40920#c15 (10Greg Grossmeier) a:5Tisza Gergő>3None (In reply to Nischay Nahata from comment #13) > https://gerrit.wikimedia.org/r/#/c/50755/ Tha... [17:35:14] 3MediaWiki / 3Uploading: text like formats (json,csv,etc) misdetected as text/plain by mw - 10https://bugzilla.wikimedia.org/66036#c5 (10Tisza Gergő) Also, I don't understand French, but that error sounds horribly uninformative. At a minimum, it should point the user to the manual page. [17:37:15] tgr: I don't know, the error sounds informative to me. Its only broken because our mime type detection sucks for json [17:38:06] I wonder what fraction of the people running mediawikis even know what a mime type is [17:39:02] anyway, an error message that's probably related to wrong configuration should contain steps on how to fix the configuration, IMO [17:40:07] Well its not really related to a wrong config. andre's suggestion is to change the config to an wrong state that counteracts the bug [17:40:35] Marking .json files as containing text/plain instead of application/json isn't what I would say is a "right" config [17:40:44] agreed [17:42:44] but you would still have to add "application/json json" to the file manually [17:43:27] or is that only needed for types which are misdetected? [17:43:48] If you added "application/json json" it still wouldn't work [17:43:56] because mediawiki thinks the file is text/plain [17:44:08] well, assuming the bug got fixed [17:44:29] If you removed the line "text/plain txt" it would work, because mediawiki wouldn't know what the proper type for a text file is [17:45:17] If the bug got fixed, we would presumably add "application/json json" to mime.types [17:46:47] tgr: ugh, this is actually probably all my fault, due to 3846d1048766a7a7ba0e57 [17:47:14] 3MediaWiki / 3Uploading: text like formats (json,csv,etc) misdetected as text/plain by mw - 10https://bugzilla.wikimedia.org/66036#c6 (10Bawolff (Brian Wolff)) Originally my fault due to 3846d1048766a7a7ba0e57 [17:48:14] that seems like the sane thing to do [17:48:34] if this is a security feature, it should be whitelist-based [17:48:49] so unknown formats have to be rejected [17:49:40] we just have to get better about setting defaults and making information about how to relax the settings easily available [17:50:20] shouldn't json -> text/plain be part of the default, for the time being? [17:52:21] We shell out to an external program to do mime detection. If that program returns application/json, then doing "text/plain json" would break it. [17:58:03] (03CR) 10Krinkle: "Should be mocked I guess?" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141582 (https://bugzilla.wikimedia.org/66938) (owner: 10Brian Wolff) [17:59:27] (03CR) 10Brian Wolff: "Yes probably. I was mostly interested in getting them not to fail. Having unit tests call out to external services doesn't seem very unit " [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141582 (https://bugzilla.wikimedia.org/66938) (owner: 10Brian Wolff) [18:09:25] bawolff: would it make sense to change mime detection in such a way, that if both "application/json json" and "text/plain json" exist as rules, then either result is accepted? [18:09:44] Actually I think we can do that with mime.info [18:09:52] Sounds like a good idea. We should do that [18:10:05] since we use external programs for mimetype detection and have no control over which type we get, that seems to be the conceptually correct thing [18:14:21] (03CR) 10Gergő Tisza: "These are integration tests (in general, tests extending ApiTestCase will not be unit tests), and as far as I am aware, MediaWiki does not" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141582 (https://bugzilla.wikimedia.org/66938) (owner: 10Brian Wolff) [18:15:03] tgr: Ok, let me test to make sure that works with mime.info, and I'll upload a patch [18:15:16] thanks! [18:21:18] oh wait, I have to do something more complicated (or it thinks .txt files should be json) [19:07:44] 3MediaWiki extensions / 3MultimediaViewer: Tooltips can get stuck when MediaViewer is closed - 10https://bugzilla.wikimedia.org/66895 (10Fabrice Florin) p:5Unprio>3Normal [19:31:45] (03PS1) 10Gergő Tisza: Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 [19:32:22] (03CR) 10jenkins-bot: [V: 04-1] Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [19:33:24] (03PS2) 10Gergő Tisza: Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 [19:33:57] (03CR) 10jenkins-bot: [V: 04-1] Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [19:34:53] (03PS1) 10Gergő Tisza: Test Jenkins tests [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141761 [19:35:26] (03CR) 10jenkins-bot: [V: 04-1] Test Jenkins tests [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141761 (owner: 10Gergő Tisza) [19:35:46] (03Abandoned) 10Gergő Tisza: Test Jenkins tests [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141761 (owner: 10Gergő Tisza) [19:39:15] (03CR) 10Gergő Tisza: [C: 032 V: 032] "Self-merging, trivial fix." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [19:39:49] (03CR) 10jenkins-bot: [V: 04-1] Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [20:20:19] (03PS1) 10Gergő Tisza: Fix failing test [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141815 [20:25:30] (03CR) 10Gergő Tisza: [C: 032] "Self-merge, bogus test error gets in the way of doing other stuff" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141815 (owner: 10Gergő Tisza) [20:26:09] (03Merged) 10jenkins-bot: Fix failing test [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141815 (owner: 10Gergő Tisza) [20:27:39] (03PS3) 10Gergő Tisza: Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 [20:27:41] (03CR) 10jenkins-bot: [V: 04-1] Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [20:30:41] (03CR) 10Gergő Tisza: "$ git review -R" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [20:31:20] (03PS4) 10Gergő Tisza: Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 [20:32:35] (03CR) 10Gergő Tisza: [C: 032] "Fingers crossed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [20:33:09] (03Merged) 10jenkins-bot: Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/141760 (owner: 10Gergő Tisza) [20:33:31] (03PS1) 10Gergő Tisza: Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] (wmf/1.24wmf10) - 10https://gerrit.wikimedia.org/r/141824 [20:34:13] (03CR) 10jenkins-bot: [V: 04-1] Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] (wmf/1.24wmf10) - 10https://gerrit.wikimedia.org/r/141824 (owner: 10Gergő Tisza) [20:34:22] fabriceflorin: you can test aspect ratio issues on beta [20:35:37] tgr: Sure, I would be happy to. I am also forwarding you more feedback from our survey respondents and may ask them to check them on beta as well. [20:36:01] I don't think it's necessary, it was a trivial fix [20:36:12] (03PS1) 10Gergő Tisza: Fix failing test [extensions/MultimediaViewer] (wmf/1.24wmf10) - 10https://gerrit.wikimedia.org/r/141825 [20:36:30] marktraceur: ^^ can I has +2? [20:58:22] tgr: I just tested on beta the same files that caused issues on IE and am pleased to report that the problem has been fixed on my end: http://en.wikipedia.beta.wmflabs.org/wiki/Lightbox_demo#mediaviewer/File:Barack_Obama_with_artistic_gymnastic_McKayla_Maroney_2.jpg [20:58:40] great [21:04:19] tgr: However, I have been encountering another bug on beta in IE with the SVG file reported as an issue by tester Oana. It gives me this error message: “could not load thumbnail data. could not load image from http://upload.beta.wmflabs.org/wikipedia/en/thumb/5/52/Giant_planes_comparison.svg.png' — It seems that a ‘png’ is being added at the end of this file name, along with a reference to ‘/en/‘ instead of Commons, where [21:04:20] it is hosted. Do we have a bug ticket for this already? Or should I file a new one? I have a screenshot. Cannot reproduce on Chrome, only on IE: http://en.wikipedia.beta.wmflabs.org/wiki/Lightbox_demo#mediaviewer/File:Giant_planes_comparison.svg [21:08:21] weird [21:08:25] which IE is that? [21:09:26] (03CR) 10Gergő Tisza: [C: 032] "Jenkins was asleep." [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140890 (owner: 10Gilles) [21:10:45] (03CR) 10Gergő Tisza: [C: 032] "Still is, apparently." [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140890 (owner: 10Gilles) [21:14:09] (03CR) 10Gergő Tisza: [V: 032] "Eh, whatever." [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140890 (owner: 10Gilles) [21:19:02] (03CR) 10Gergő Tisza: "D'oh, didn't notice the dependency." [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140890 (owner: 10Gilles) [21:19:34] (03PS2) 10Gergő Tisza: Display opt-out totals for each wiki on a global graph [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140892 (owner: 10Gilles) [21:22:09] (03PS3) 10Gergő Tisza: Separate the opt-in/opt-out actions into their own tab/graph [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140888 (owner: 10Gilles) [21:23:13] tgr: It was IE 10. I cannot reproduce it every time, but it happened twice. Want me to file a ticket for it with the screenshot? [21:23:27] yes, please [21:24:24] (03PS4) 10Gergő Tisza: Separate the opt-in/opt-out actions into their own tab/graph [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140888 (owner: 10Gilles) [21:28:45] (03PS2) 10Gergő Tisza: Add opt-out and opt-in totals to graph [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140890 (owner: 10Gilles) [21:29:06] (03PS3) 10Gergő Tisza: Display opt-out totals for each wiki on a global graph [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/140892 (owner: 10Gilles) [21:36:34] tgr: I filed this ticket on Mingle: #742 SVG file load error with Png suffix — I made it a ‘Should have’ for now, since I cannot reproduce it easily, even though I experienced it twice already. https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/742 [21:39:23] I seem to recall we had other reports of PNG issues related to SVG files in the feedback, which I can dig up if you like. In the meantime, we also have this possibly related ticket: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/470 [22:40:44] (03PS1) 10Gergő Tisza: Allow multimedia team members to +2 changes to wmf/* branches [extensions/MultimediaViewer] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/141856 [22:40:46] (03CR) 10jenkins-bot: [V: 04-1] Allow multimedia team members to +2 changes to wmf/* branches [extensions/MultimediaViewer] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/141856 (owner: 10Gergő Tisza) [22:41:58] (03CR) 10Gergő Tisza: [C: 032 V: 032] Allow multimedia team members to +2 changes to wmf/* branches [extensions/MultimediaViewer] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/141856 (owner: 10Gergő Tisza) [22:42:00] (03CR) 10jenkins-bot: [V: 04-1] Allow multimedia team members to +2 changes to wmf/* branches [extensions/MultimediaViewer] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/141856 (owner: 10Gergő Tisza) [22:42:27] (03CR) 10Gergő Tisza: [C: 032 V: 032] Allow multimedia team members to +2 changes to wmf/* branches [extensions/MultimediaViewer] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/141856 (owner: 10Gergő Tisza) [22:44:51] (03PS1) 10Gergő Tisza: Revert "Allow multimedia team members to +2 changes to wmf/* branches" [extensions/MultimediaViewer] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/141858 [22:45:00] (03CR) 10Gergő Tisza: [C: 032 V: 032] Revert "Allow multimedia team members to +2 changes to wmf/* branches" [extensions/MultimediaViewer] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/141858 (owner: 10Gergő Tisza) [23:00:59] (03CR) 10MaxSem: [C: 032] Fix failing test [extensions/MultimediaViewer] (wmf/1.24wmf10) - 10https://gerrit.wikimedia.org/r/141825 (owner: 10Gergő Tisza) [23:01:27] (03CR) 10MaxSem: [C: 032] Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] (wmf/1.24wmf10) - 10https://gerrit.wikimedia.org/r/141824 (owner: 10Gergő Tisza) [23:01:35] (03Merged) 10jenkins-bot: Fix failing test [extensions/MultimediaViewer] (wmf/1.24wmf10) - 10https://gerrit.wikimedia.org/r/141825 (owner: 10Gergő Tisza) [23:02:00] (03Merged) 10jenkins-bot: Fix for merge mistake in Ic37b4150288055c3fae8d22919ed7b1249db1f09 [extensions/MultimediaViewer] (wmf/1.24wmf10) - 10https://gerrit.wikimedia.org/r/141824 (owner: 10Gergő Tisza) [23:35:18] (03PS1) 10Reedy: Remove unused ext.uploadWizard.campaign and ext.uploadWizard.campaigns [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141869 [23:37:40] (03CR) 10Ori.livneh: [C: 032] Remove unused ext.uploadWizard.campaign and ext.uploadWizard.campaigns [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141869 (owner: 10Reedy) [23:38:03] (03Merged) 10jenkins-bot: Remove unused ext.uploadWizard.campaign and ext.uploadWizard.campaigns [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141869 (owner: 10Reedy) [23:42:24] fabriceflorin: http://multimedia-metrics.wmflabs.org/dashboards/mmv -> optouts actually work here, there just aren't too many of them [23:46:15] (03PS1) 10Reedy: Remove unused messages [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141871 [23:46:58] (03PS2) 10Reedy: Remove unused messages [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141871 [23:49:50] (03CR) 10Gergő Tisza: "I would be careful with this, the message keys seem to correspond to keys in JSON config blobs which are stored in the wiki. Are you sure " [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141871 (owner: 10Reedy) [23:50:01] tgr: Thanks for the clarification. You are correct, I had not checked all the recent dates. However, it seems strange that only 3 anons disabled the tool yesterday, and 7 anons + 1 registered user the day before. This doesn’t seem to match the numbers I see in your opt-out spreadsheet, which suggests higher numbers of disables: e.g 251 registered users disabled from 6/16 to 6/24, which would be an average of 30 disables/day. Why the [23:50:02] discrepancy? http://ur1.ca/hljcq [23:50:37] (03CR) 10Reedy: "They're not used in PHP, or loaded into RL modules at least" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/141871 (owner: 10Reedy) [23:50:45] those are enwiki numbers, and the logging code is not deployed on enwiki yet [23:52:09] tgr: Got it, thanks so much for this helpful explanation. On that basis, I just accepted ticket #558. Much appreciated!