[00:04:48] (03PS9) 10Mooeypoo: Implement SortedList in Echo notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/249503 [00:05:51] (03CR) 10jenkins-bot: [V: 04-1] Implement SortedList in Echo notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/249503 (owner: 10Mooeypoo) [00:07:30] 6Collaboration-Team-Backlog, 10Flow: Using ctrl-z (browser undo) after previewing is not possible - https://phabricator.wikimedia.org/T117817#1784263 (10Quiddity) 3NEW [00:07:57] 3Collaboration-Team-Current, 10Echo, 10MediaWiki-Internationalization, 10MediaWiki-extensions-CLDR, 7Wikimedia-log-errors: wgUserLanguage in JS should be valid for API (Exception: Invalid language code "nl_nl") - https://phabricator.wikimedia.org/T116890#1784272 (10Etonkovidova) Checked in beta - using u... [00:18:54] matt_flaschen, re: links in topic titles - are we implementing just wikilinks, or also external links? [00:19:04] (i assume the former) [00:19:07] Correct [00:19:10] ty [00:19:21] quiddity, it will be the exact same as edit summaries. [00:28:46] (03PS1) 10Mattflaschen: Fix regression in edit post and edit topic summary due to OO UI changes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251175 (https://phabricator.wikimedia.org/T117698) [00:30:37] ^ mooeypoo, that should fix prod. It seems there is another issue with description locally which I'll look at next, doesn't affect prod for me. [00:31:35] 6Collaboration-Team-Backlog, 10Flow: Broken width-expanding toggle on Topic pages - https://phabricator.wikimedia.org/T117823#1784357 (10Quiddity) 3NEW [00:35:34] (03CR) 10Catrope: [C: 032] Fix regression in edit post and edit topic summary due to OO UI changes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251175 (https://phabricator.wikimedia.org/T117698) (owner: 10Mattflaschen) [00:35:43] matt_flaschen: Well done, good catch [00:35:58] stephanebisson: urrrgh not more UBNs :( [00:36:58] stephanebisson found it, but thanks. :) [00:37:47] Oh OK [00:37:52] Good job stephanebisson then [00:38:49] (03Merged) 10jenkins-bot: Fix regression in edit post and edit topic summary due to OO UI changes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251175 (https://phabricator.wikimedia.org/T117698) (owner: 10Mattflaschen) [00:39:40] thanks the BT [00:44:08] RoanKattouw, want me to try to sneak it in the SWAT? Or we could ask greg-g for a window after. [00:45:00] Sure, try that [00:45:05] Or morning-SWAT it tomorrow [00:49:23] 6Collaboration-Team-Backlog, 10Flow: "Talk" tab stays red - https://phabricator.wikimedia.org/T115201#1784398 (10Quiddity) Possibly related to the problems/fixes seen in {T78102} ? @matthiasmullie possibly you can give brief advice? [00:50:01] matt_flaschen: same issue is happening on the undo page (coming from history) in firefox [00:50:45] Okay, will do that now. Like I mentioned, I think there is also a header issue that hasn't hit prod yet. [00:53:47] It's wmf5-only, right? [01:06:31] 6Collaboration-Team-Backlog, 10Flow: When a page is manually converted to use Flow, the page-tabs turns blue immediately even if the page is empty - https://phabricator.wikimedia.org/T117828#1784441 (10Quiddity) 3NEW [01:20:42] (03PS1) 10Mattflaschen: Fix regression in edit post and edit topic summary due to OO UI changes [extensions/Flow] (wmf/1.27.0-wmf.5) - 10https://gerrit.wikimedia.org/r/251183 (https://phabricator.wikimedia.org/T117698) [01:30:48] 6Collaboration-Team-Backlog, 10Flow: During preview of a Flow post all refs show as [0] - https://phabricator.wikimedia.org/T117832#1784496 (10Quiddity) 3NEW [01:37:06] 6Collaboration-Team-Backlog, 10Flow, 6Language-Engineering, 10Parsoid, and 4 others: Provide an API flag to suppress auto-generated - https://phabricator.wikimedia.org/T113331#1784505 (10santhosh) @ssastry, CX needs the in the output and at this point we don't have any usescas... [01:43:12] Oh, I put it up for morning SWAT. [01:44:27] RoanKattouw, yeah, that's all https://gerrit.wikimedia.org/r/#/c/249187/ is in. [01:52:10] 6Collaboration-Team-Backlog, 10Flow: When a page is manually converted to use Flow, the page-tabs turns blue immediately even if the page is empty - https://phabricator.wikimedia.org/T117828#1784552 (10Mattflaschen) We no longer have magic globals (wgFlowOccupyPages) for this, because we wanted to do it the co... [02:14:15] 3Collaboration-Team-Current, 10Flow: undo (coming from history) fails in Firefox - https://phabricator.wikimedia.org/T117835#1784584 (10Mattflaschen) 3NEW a:3Mattflaschen [02:17:54] 3Collaboration-Team-Current, 10Flow: Fix en:Wikipedia_talk:Flow/Developer_test_page - https://phabricator.wikimedia.org/T117812#1784596 (10Mattflaschen) a:3Mattflaschen [02:30:05] 6Collaboration-Team-Backlog, 10Flow, 7Easy, 3Google-Code-In-2015: History tab should show as soon as board is created, before refresh - https://phabricator.wikimedia.org/T117837#1784626 (10Mattflaschen) [02:53:38] 3Collaboration-Team-Current, 10Flow: undo (coming from history) fails in Firefox - https://phabricator.wikimedia.org/T117835#1784646 (10Mattflaschen) [02:54:35] (03PS1) 10Mattflaschen: Fix undo form for OO UI selection changes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251191 (https://phabricator.wikimedia.org/T117835) [02:55:39] 6Collaboration-Team-Backlog, 10Flow: Create a system to see last active conversations on a wiki - https://phabricator.wikimedia.org/T117222#1784657 (10Spage) A few random thoughts on this. I believe the "ugly" UUIDs of topics and posts are actually in chronological order, since they start with a timestamp. So... [02:56:41] Can't reproduce the header problem anymore. If anyone else sees it, let me know. [02:59:13] See you all tomorrow. [03:01:10] ttfn! [03:07:38] 6Collaboration-Team-Backlog, 10Flow, 6Language-Engineering, 10Parsoid, and 4 others: Provide an API flag to suppress auto-generated - https://phabricator.wikimedia.org/T113331#1784660 (10ssastry) 5Open>3declined From IRC conversations, it seems that Flow doesn't need this either. Given... [03:40:16] 6Collaboration-Team-Backlog, 10Flow, 6Language-Engineering, 10Parsoid, and 3 others: Provide an API flag to suppress auto-generated - https://phabricator.wikimedia.org/T113331#1784669 (10Arlolra) [03:58:21] quiddity: if only I was a fluent gadget programmer, https://www.mediawiki.org/wiki/Topic:Srraiyf15fb22qil awaits :) [04:39:34] 6Collaboration-Team-Backlog, 10Flow: When a page is manually converted to use Flow, the page-tabs turns blue immediately even if the page is empty - https://phabricator.wikimedia.org/T117828#1784710 (10Spage) >>! In T117828#1784552, @Mattflaschen wrote: > > ... But in order to mark a page Flow it has to exist... [04:49:58] 3Collaboration-Team-Current, 10Echo, 5Patch-For-Review: Log notification impressions and clicks again - https://phabricator.wikimedia.org/T114833#1784714 (10Mooeypoo) It was blocked because we were waiting for some data to make sure we have the correct logging information. If that is approved, we can close... [09:29:50] 6Collaboration-Team-Backlog, 10Flow: Moving topics between boards - https://phabricator.wikimedia.org/T88140#1784963 (10Pginer-WMF) [09:29:51] 6Collaboration-Team-Backlog, 10Flow: Spike: Autocomplete for moving topics between Flow boards (Translatewiki) - https://phabricator.wikimedia.org/T88616#1784962 (10Pginer-WMF) [09:31:34] 6Collaboration-Team-Backlog, 10Flow: Spike: Autocomplete for moving topics between Flow boards (Translatewiki) - https://phabricator.wikimedia.org/T88616#1016228 (10Pginer-WMF) [10:38:31] 3Collaboration-Team-Current, 10Flow: Freeze LQT on pt.wikibooks - https://phabricator.wikimedia.org/T114278#1785113 (10He7d3r) Just checking: Is this the step which will make the link [[https://pt.wikibooks.org/wiki/Special:NewMessages|New messages (none)]] to be removed from the personal links at the top of t... [11:44:41] 6Collaboration-Team-Backlog, 10Flow, 10Browser-Tests-Infrastructure, 5Patch-For-Review: Fix or delete failing Flow browsertests Jenkins jobs - https://phabricator.wikimedia.org/T94153#1785226 (10zeljkofilipin) a:5zeljkofilipin>3None [11:45:36] 6Collaboration-Team-Backlog, 10Flow, 10Browser-Tests-Infrastructure: Fix or delete failing Flow browsertests Jenkins jobs - https://phabricator.wikimedia.org/T94153#1169822 (10zeljkofilipin) [11:52:37] 6Collaboration-Team-Backlog, 10Echo, 7WorkType-NewFunctionality: Invite users to enable Cross-wiki Notifications beta feature - https://phabricator.wikimedia.org/T117669#1785251 (10Pginer-WMF) > Correct. This does give us an easy opportunity to promote the BetaFeature more aggressively later, should we wish... [12:24:57] 6Collaboration-Team-Backlog, 10Echo, 7WorkType-NewFunctionality: Invite users to enable Cross-wiki Notifications beta feature - https://phabricator.wikimedia.org/T117669#1785324 (10Pginer-WMF) [12:24:58] 6Collaboration-Team-Backlog, 10Echo: Present cross-wiki notifications as a beta feature to users - https://phabricator.wikimedia.org/T114237#1785325 (10Pginer-WMF) [14:31:43] (03CR) 10Sbisson: [C: 032] Fix undo form for OO UI selection changes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251191 (https://phabricator.wikimedia.org/T117835) (owner: 10Mattflaschen) [14:35:09] (03Merged) 10jenkins-bot: Fix undo form for OO UI selection changes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251191 (https://phabricator.wikimedia.org/T117835) (owner: 10Mattflaschen) [14:47:13] 6Collaboration-Team-Backlog, 10Flow: Visual editor's "wikitext markup detected" message needs to be tweaked for Flow - https://phabricator.wikimedia.org/T107100#1785469 (10Trizek-WMF) [[ https://fr.wikipedia.org/wiki/Sujet:Ss0gcm7rnwahc405 | Got a +1 from fr ]] about this feature. [14:56:27] Morning [15:07:03] 3Collaboration-Team-Current, 10Flow, 7Regression: Broken width-expanding toggle on Topic pages - https://phabricator.wikimedia.org/T117823#1785495 (10Mattflaschen) p:5Triage>3High [15:15:33] (03PS1) 10Mattflaschen: Fix undo form for OO UI selection changes [extensions/Flow] (wmf/1.27.0-wmf.5) - 10https://gerrit.wikimedia.org/r/251246 (https://phabricator.wikimedia.org/T117835) [15:21:28] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Convert LQT -> Flow on pt.wikibooks - https://phabricator.wikimedia.org/T113682#1785555 (10Mattflaschen) Until it's 100% complete, we should not move any archive pages unless they have `{{#useliquidthreads:0}}`. I'll be looking into the duplic... [15:23:38] 6Collaboration-Team-Backlog, 10Flow: Can't connect to Parsoid locally - https://phabricator.wikimedia.org/T117156#1785574 (10SBisson) [15:53:08] (03CR) 10Siebrand: [C: 031] "i18n/L10n reviewed." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/243066 (https://phabricator.wikimedia.org/T59153) (owner: 10Mattflaschen) [15:56:03] (03CR) 10Siebrand: "i18n/L10n reviewed." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/247017 (owner: 10Mooeypoo) [16:07:47] (03CR) 10Thcipriani: [C: 032] "SWAT" [extensions/Flow] (wmf/1.27.0-wmf.5) - 10https://gerrit.wikimedia.org/r/251183 (https://phabricator.wikimedia.org/T117698) (owner: 10Mattflaschen) [16:10:50] (03Merged) 10jenkins-bot: Fix regression in edit post and edit topic summary due to OO UI changes [extensions/Flow] (wmf/1.27.0-wmf.5) - 10https://gerrit.wikimedia.org/r/251183 (https://phabricator.wikimedia.org/T117698) (owner: 10Mattflaschen) [16:30:57] matt_flaschen: are we supposed to stop showing Flow notifications related to moderated content? [16:33:05] stephanebisson, I can't imagine we'd want to show it for deleted or suppressed content. Hidden probably not, but should confirm with quiddity. [16:33:48] stephanebisson, locked should still show (e.g. if there are 3 replies to a topic, then it gets locked, you may still want to read them even though you can't reply without unlocking it). [16:34:13] matt_flaschen: that's what I thought. I'll redo my tests before saying it's completely broken ;) [16:34:33] yeah, lock is almost not moderated [16:34:44] Yeah, just mentioned that for completeness. [16:35:14] stephanebisson, it probably is broken. Core deletion is/was until recently too ([No page]), so not surprised. [16:35:59] I'm surprised we haven't heard about it more with the spam incident [16:36:51] matt_flaschen: also, is it ok for a normal user to see deleted topic titles in the deletion log? [16:37:18] while they show as empty string in the history [16:38:15] stephanebisson, I'm aware of that. The history rendering really needs to be changed. IIRC it's to spec but the spec was "okay, let's do this for now". I don't consider them restricted, though. [16:38:44] stephanebisson, normal page titles are not restricted after deletion (you can know the page once existed at that title), and if we ever implement "Topic:Friendly name" as the real page name we'll probably have to keep that aspect of core. [16:39:02] So i consider it public even though we're not consistent about it. [16:39:38] so suppression is the way to erase all traces of it [16:40:08] Yeah, even admins can't see the content (including topic title) then. [16:48:43] (03PS1) 10Mattflaschen: Historical UID: Change counter to start at any possible value [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251265 [16:55:46] (03CR) 10Mattflaschen: [C: 04-2] "Should be removed by Matthias and/or Erik, just to make sure there is not some reason for this." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251265 (owner: 10Mattflaschen) [16:57:11] ebernhardson, could you look at https://gerrit.wikimedia.org/r/251265 ? It's about the historical UID generator. Looked at it because of the collisions we are having. I don't think it will help that much, but I don't see why counter starts at max 256 (but I could be wrong). [16:59:50] matt_flaschen: i don't remember why it starts at 256 either, best guess is i did it wrong :( [17:01:36] ebernhardson, no problem, I just wanted to make sure there wasn't some elaborate reason that isn't covered in the first four years of CS. [17:01:45] ;) [17:04:35] :) [17:10:48] (03CR) 10Mattflaschen: "Erik couldn't recall there being a reason for this, so this can be reviewed." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251265 (owner: 10Mattflaschen) [17:11:29] (03CR) 10EBernhardson: [C: 031] "seems sane. I might use a regular class constant, but that's just bike shedding :)" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251265 (owner: 10Mattflaschen) [17:11:46] ebernhardson, I don't think that's going to make that much difference though. I could re-run it with just that, but I think I'm going to add actual retry logic in the importer so it catches duplicate ID errors and tries again. [17:12:04] matt_flaschen: hmm, as long as memory usage isn't an issue should be reasonable [17:12:28] I found one other apparent bug (https://gerrit.wikimedia.org/r/#/c/251256/), but that shouldn't make any actual difference either (unless we take a *really* long time converting ptwikibooks ;) ). [17:12:56] heh [17:13:01] ebernhardson, we're already having memory leaks, but I don't think this will make it much worse. I'll probably cap the retries anyway, probably to 3. I'm thinking one reason might be bots posting in close succession. [17:14:06] matt_flaschen: that could do it, Assuming those get imported in order could also use a counter for the milliseconds used in the time as well [17:14:26] so instead of mt_rand(0, 999) use $otherCount++ % 1000 [17:14:55] ebernhardson, yeah, that's a good idea. [17:15:13] I was thinking that if two events happen in the same second (quite likely, especially but not only for bots) we only have milliseconds and counter to distinguish. [17:15:52] Which is about 20 bits (should be plenty with retries), but random milliseconds could indeed be making it worse. Not sure. [17:22:23] ebernhardson, there's actually no reason posts with the same second would be imported in order, though. Since it goes in page order, not time order. [17:22:33] So I think I'll keep the random milliseconds and just do retries. [17:23:08] hmm, yea [17:36:40] I'm seeing "Error: Unknown dependency: ext.gather.watchstar" in my js console all over the prod sites. Known issue? [17:37:13] oh "gather" is mobile web huh? ?me wanders over there [17:39:39] (03CR) 10Matthias Mullie: [C: 032] Historical UID: Change counter to start at any possible value [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251265 (owner: 10Mattflaschen) [17:41:02] (03CR) 1020after4: [C: 032] Fix undo form for OO UI selection changes [extensions/Flow] (wmf/1.27.0-wmf.5) - 10https://gerrit.wikimedia.org/r/251246 (https://phabricator.wikimedia.org/T117835) (owner: 10Mattflaschen) [17:42:36] (03Merged) 10jenkins-bot: Historical UID: Change counter to start at any possible value [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251265 (owner: 10Mattflaschen) [17:44:57] (03Merged) 10jenkins-bot: Fix undo form for OO UI selection changes [extensions/Flow] (wmf/1.27.0-wmf.5) - 10https://gerrit.wikimedia.org/r/251246 (https://phabricator.wikimedia.org/T117835) (owner: 10Mattflaschen) [17:49:33] ebernhardson, but there's still 42 bits that should be random between every call (counter and node ID). [17:49:42] It seems weird to have this many collisions. [17:52:03] Morning, mooeypoo [18:03:57] skierpage, yes! I wish I could, too. [18:05:25] ? [18:05:43] Scrollback? [18:06:48] matt_flaschen, , yup, from last night. ( quiddity: if only I was a fluent gadget programmer, https://www.mediawiki.org/wiki/Topic:Srraiyf15fb22qil awaits :)) [18:08:30] quiddity, I've been meaning to finally set up a bouncer. [18:08:40] Anyone have any tips? [18:08:48] Looks like we don't have an official one yet. [18:09:00] I could use IRCCloud, but open to suggestions. [18:10:45] 3Collaboration-Team-Current, 10Flow, 7Database, 5Patch-For-Review, 7WorkType-NewFunctionality: Deploy artificial primary key reference change to all production wikis simultaneously - https://phabricator.wikimedia.org/T117785#1786029 (10jcrespo) [18:10:46] 3Collaboration-Team-Current, 10Flow, 7Database, 5Patch-For-Review, 7WorkType-NewFunctionality: Add artificial primary key to flow_wiki_ref and flow_ext_ref - https://phabricator.wikimedia.org/T109676#1786030 (10jcrespo) [18:10:47] 3Collaboration-Team-Current, 10Flow, 7Blocked-by-schema-change, 7Database, and 2 others: Add ref_id to flow_wiki_ref and flow_ext_ref (DBA phase 1) - https://phabricator.wikimedia.org/T117783#1786027 (10jcrespo) 5Open>3Resolved Deployed to both x1 and office wiki. Final results: ``` MariaDB [flowdb]>... [18:10:53] matt_flaschen, there are docs for getting an IRCCloud account at https://office.wikimedia.org/wiki/Office_IT/IRCCloud - I haven't tried that myself though. [18:11:09] skierpage, you basically wrote the entire thing. :) You can use OO.getProp as a substitute for jq to handle the nesting. [18:11:43] I'm sure you can write it, but I will if you insist. ;) [18:12:06] And roots is better, though if you do toconly it may not matter. [18:15:51] (03CR) 10Jcrespo: [C: 031] "I am ok with the code (but I have not tested it myself). Everithings seems ok and I do NOT see race condition (like UUIDs being transforme" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/238393 (https://phabricator.wikimedia.org/T109676) (owner: 10Matthias Mullie) [18:15:53] matt_flaschen: I recommend using quassel [18:16:11] mornin [18:16:20] Works nicely across all my devices. Only downside is that you can't pick your preferred IRC client like with ZNC [18:16:37] (03CR) 10Jcrespo: "Forgot to say that I have already applied the first patch on live servers: T117783" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/238393 (https://phabricator.wikimedia.org/T109676) (owner: 10Matthias Mullie) [18:16:47] stephanebisson, matt_flaschen, I think 'hidden' content should also be removed from notifications. It's the equivalent of "revert", which is how non-admins help to moderate inappropriate content. (But I'm asking Benoit for his thoughts, as I'm unsure). [18:17:17] legoktm, so you self-host quassel-core (is that the bouncer part)? [18:17:36] quiddity: maybe https://phabricator.wikimedia.org/T93673 shouldn't be in the freezer... [18:17:57] legoktm do you have a few minutes to talk about Gadgets 2.0? [18:18:13] quiddity: especially for suppressed content [18:18:25] 3Collaboration-Team-Current, 10Flow, 7Database, 5Patch-For-Review, 7WorkType-NewFunctionality: Run maintenance/FlowPopulateRefId.php on all production wikis - https://phabricator.wikimedia.org/T117786#1786059 (10jcrespo) If possible, and I am online, ping me when this happens to check the lag. [18:18:54] legoktm we want to work on the Gadgets namespaces in the next sprint, so we'd like to check with you about what needs to get done [18:19:08] stephanebisson, agreed. Let's ask at standup. [18:19:26] legoktm, also, I really like Konversation. [18:20:09] 6Collaboration-Team-Backlog, 3Collaboration-Team-Current, 10Echo, 10Flow: Remove Notifications about posts/topics that have been Moderated - https://phabricator.wikimedia.org/T93673#1786062 (10Mattflaschen) [18:20:23] dannyh: yeah, I have a few minutes before standup starts [18:20:26] 3Collaboration-Team-Current, 10Echo, 10Flow: Remove Notifications about posts/topics that have been Moderated - https://phabricator.wikimedia.org/T93673#1143233 (10Mattflaschen) [18:20:40] legoktm can you come to Hangout comm-tech? [18:20:52] legoktm I'm in there w Kaldari [18:21:05] * legoktm finds headphones [18:21:54] (03CR) 10Mattflaschen: [C: 032] "Phase 1 is deployed. I reviewed the code earlier." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/238393 (https://phabricator.wikimedia.org/T109676) (owner: 10Matthias Mullie) [18:23:33] 3Collaboration-Team-Current, 10Flow, 7Database, 5Patch-For-Review, 7WorkType-NewFunctionality: Deploy artificial primary key reference change to all production wikis simultaneously - https://phabricator.wikimedia.org/T117785#1786133 (10Mattflaschen) Blocker is done. I +2'ed this. Let's deploy it everyw... [18:23:59] (03CR) 10Mattflaschen: "Phase 1 being just the DB part" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/238393 (https://phabricator.wikimedia.org/T109676) (owner: 10Matthias Mullie) [18:24:21] matt_flaschen: Yeah, I run quasselcore on a VPS, and then have quassel client on my laptops, and quasseldroid on my phone. [18:24:37] I used to use ZNC, but it didn't work very well when my internet connection kept dropping out [18:26:02] (03Merged) 10jenkins-bot: Add artificial primary key to flow_wiki_ref and flow_ext_ref [extensions/Flow] - 10https://gerrit.wikimedia.org/r/238393 (https://phabricator.wikimedia.org/T109676) (owner: 10Matthias Mullie) [18:26:19] http://blog.legoktm.com/2014/11/23/quassel.html [18:29:23] Thanks, will think about it. [18:29:38] Actually, 52 bits (32 bit node ID, 10 bit counter, and ~10 bit ms). [18:33:52] 6Collaboration-Team-Backlog, 10Echo, 5Patch-For-Review: Support global account rights change notification in Echo - https://phabricator.wikimedia.org/T53541#1786225 (10matej_suchanek) [18:52:13] 6Collaboration-Team-Backlog, 10Flow, 7Easy: Trigger topic loading (infinite scroll) before the very end of the list is reached (ToC v2) - https://phabricator.wikimedia.org/T78573#1786311 (10Pginer-WMF) [18:54:36] 6Collaboration-Team-Backlog, 10Flow, 7Easy: Edit description link should be grey by default, not blue - https://phabricator.wikimedia.org/T112872#1786321 (10Pginer-WMF) [18:55:43] (03PS10) 10Mooeypoo: Implement SortedList in Echo notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/249503 [18:55:46] (03PS12) 10Mooeypoo: Add an APIHandler to Echo notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/247021 [18:55:50] (03PS8) 10Mooeypoo: Allow notification model and widget to show combined notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/249031 [18:55:52] (03PS7) 10Mooeypoo: Initialize notifications model outside of the widget [extensions/Echo] - 10https://gerrit.wikimedia.org/r/249008 [18:57:52] 6Collaboration-Team-Backlog, 10Flow, 7Easy: Better distinguish rich text (VE) from code (wikitext) modes in Flow - https://phabricator.wikimedia.org/T116567#1786332 (10Pginer-WMF) [19:01:50] matt_flaschen: I get the patch and refresh Talk:Flow_QA: https://phabricator.wikimedia.org/P2280 [19:02:06] (03CR) 10Mattflaschen: "Roan was able to reproduce this." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/245621 (https://phabricator.wikimedia.org/T112230) (owner: 10Mattflaschen) [19:09:45] 6Collaboration-Team-Backlog, 10Echo, 10UI-Standardization, 5Patch-For-Review: Echo notification icons should be properly themed in MonoBook - https://phabricator.wikimedia.org/T112290#1786346 (10Catrope) >>! In T112290#1781832, @Esanders wrote: > | Before | After | > |{F2914227,size=full} |{F2914222,size=f... [19:09:48] 6Collaboration-Team-Backlog, 10Echo, 10UI-Standardization, 5Patch-For-Review: Echo notification icons should be properly themed in MonoBook - https://phabricator.wikimedia.org/T112290#1786347 (10Catrope) >>! In T112290#1781832, @Esanders wrote: > | Before | After | > |{F2914227,size=full} |{F2914222,size=f... [19:11:06] (03CR) 10Catrope: [C: 04-1] "This change makes "unseen notifications" and "seen notifications" almost indistinguishable. The only indication is the font color of the n" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/250969 (https://phabricator.wikimedia.org/T112290) (owner: 10Esanders) [19:22:06] (03PS11) 10Mooeypoo: Implement SortedList in Echo notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/249503 [19:24:08] (03PS1) 10Mooeypoo: [wip] Allow notifications to be marked as unread [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251309 [19:25:25] (03PS2) 10Mooeypoo: [wip] Allow notifications to be marked as unread [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251309 [19:36:51] (03CR) 10Catrope: "Works for preloadtitle but not for preload, for some reason. I don't know why, and I'm not very familiar with this code." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/250959 (https://phabricator.wikimedia.org/T107637) (owner: 10Bartosz Dziewoński) [19:38:43] (03CR) 10Catrope: [C: 032] Use flow- prefix for the CSS classes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/248988 (owner: 10Mattflaschen) [19:39:35] (03CR) 10Catrope: [C: 032] Add Estonian translation for flow topic namespace [extensions/Flow] - 10https://gerrit.wikimedia.org/r/250829 (owner: 10Gerrit Patch Uploader) [19:42:38] (03PS1) 10Mooeypoo: Make 'x' button in notifications bigger [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251315 [19:43:18] RoanKattouw_away, legoktm there was a bug about this and I can't find it... any of you remember where it is or did I imagine it? ^^ [19:43:24] (03Merged) 10jenkins-bot: Use flow- prefix for the CSS classes [extensions/Flow] - 10https://gerrit.wikimedia.org/r/248988 (owner: 10Mattflaschen) [19:43:29] * legoktm finds [19:43:49] mooeypoo: https://phabricator.wikimedia.org/T112217 [19:44:03] ah! thanks! I don't know why I didn't find it' [19:44:26] (03PS2) 10Mooeypoo: Make 'x' button in notifications bigger [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251315 (https://phabricator.wikimedia.org/T112217) [19:44:45] 6Collaboration-Team-Backlog, 10Echo, 7Accessibility, 5Patch-For-Review: Mark as read 'x' button is very small - https://phabricator.wikimedia.org/T112217#1786450 (10Mooeypoo) a:3Mooeypoo [19:45:08] 3Collaboration-Team-Current, 10Echo, 7Accessibility, 5Patch-For-Review: Mark as read 'x' button is very small - https://phabricator.wikimedia.org/T112217#1629127 (10Mooeypoo) [19:45:28] (03Merged) 10jenkins-bot: Add Estonian translation for flow topic namespace [extensions/Flow] - 10https://gerrit.wikimedia.org/r/250829 (owner: 10Gerrit Patch Uploader) [19:52:50] (03CR) 10Catrope: [C: 032] Make 'x' button in notifications bigger [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251315 (https://phabricator.wikimedia.org/T112217) (owner: 10Mooeypoo) [19:53:52] (03CR) 10Catrope: [C: 032] Add an APIHandler to Echo notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/247021 (owner: 10Mooeypoo) [19:55:45] (03Merged) 10jenkins-bot: Make 'x' button in notifications bigger [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251315 (https://phabricator.wikimedia.org/T112217) (owner: 10Mooeypoo) [19:57:46] (03Merged) 10jenkins-bot: Add an APIHandler to Echo notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/247021 (owner: 10Mooeypoo) [19:59:03] 6Collaboration-Team-Backlog, 10Thanks, 7Design: Echo's "thanked you for your edit" should also link to thanker user page in email notification - https://phabricator.wikimedia.org/T49511#1786522 (10Etonkovidova) Currently, in an email notification for changes on a non-Flow page, there will be a link, e.g. ```... [20:00:19] matt_flaschen: Re UUIDs: perhaps we are using a very poor source of randomness, or always seeding it the same, or something? Perhaps terbium's randomness source sucks or isn't being used? [20:00:32] 3Collaboration-Team-Current, 10Echo, 10Thanks, 5Patch-For-Review, 5WMF-deploy-2015-11-03_(1.27.0-wmf.5): Write presentation models for notifications in Thanks - https://phabricator.wikimedia.org/T116847#1786525 (10Etonkovidova) Checked in betalabs - regression testing Notifications on both kinds of pages... [20:00:42] RoanKattouw, yeah, I'm going to run a test to see how many we get before collisions on average, vs. expected from birthday paradox. [20:00:56] Cause if you're using 32 bits of randomness then your collision probability should be related to 2^16 [20:00:59] mt_rand is not cryptographically secure, but I don't know how "good" it is. [20:01:03] because of the birthday paradox [20:01:09] Right [20:01:25] The what now? [20:01:38] mooeypoo, ooh, it's pretty fun. [20:01:39] Why is that familiar... [20:01:49] I have a feeling I heard about that paradox before [20:02:00] mooeypoo, how many people have to be in a room for there to be a 50% chance of a birthday collision. [20:02:01] RoanKattouw, matt_flaschen fwiw the image on the left here is generated with rand() in a libc that has a horrible implementation, and the one on the right is php's mt_rand implementation: ttp://tinypic.com/view.php?pic=vwtppl&s=5#.Vju1mnUViko [20:02:04] The Answer May Surprise You! TM [20:02:15] i don't know how to rate good vs bad, but at least its not as bad as some things :) [20:02:24] how you you define a birthday collision? same day? [20:02:31] mooeypoo, yeah, not same year. [20:03:19] mooeypoo: In the context of random numbers, the birthday paradox says that if you generate a bunch of random numbers between 0 and 2^n , you expect duplicates to be likely when you generate approx 2^(n/2) numbers [20:03:22] which is sqrt(2^n) [20:03:35] Naively, you'd expect this to occur at (2^n)/2 which is 2^(n-1) [20:03:36] I have a few comments about this, but there's food. I'll be back after I eat. [20:03:37] bbiab [20:10:03] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Determine cause of historical UIDs colliding - https://phabricator.wikimedia.org/T117907#1786561 (10Mattflaschen) 3NEW a:3Catrope [20:10:12] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Determine cause of historical UIDs colliding - https://phabricator.wikimedia.org/T117907#1786561 (10Mattflaschen) a:5Catrope>3Mattflaschen [20:10:56] Getting lunch [20:34:09] 3Collaboration-Team-Current, 10Flow: Validate which Flow moves are allowed? - https://phabricator.wikimedia.org/T96304#1786611 (10Quiddity) p:5Normal>3Triage [20:37:09] stephanebisson: regarding https://phabricator.wikimedia.org/T117698 Edit post and Summarize fail in firefox - you saw it only locally? [20:37:26] 3Collaboration-Team-Current, 10Flow: Validate which Flow moves are allowed? - https://phabricator.wikimedia.org/T96304#1786624 (10Quiddity) I've overhauled the description, unset the priority, and moved to 'current' workboard primarily to re-triage the difficulty. Needs team discussion, on the available option... [20:38:10] etonkovidova: it was on master yesterday, I don't remember if I checked on beta but it must have been there, I don't think it was on mw.org [20:38:55] stephanebisson: ok - thx [20:38:58] etonkovidova, stephanebisson, the problem was occurring on mediawiki yesterday, but it works fine today. [20:39:15] (i.e. the page wouldn't finish loading, when I clicked "edit post") [20:39:25] (I assume that''s the problem you're discussing) [20:40:58] quiddity: edit post was navigating to a new page, as opposed to editing in place, and that page wouldn't finish loading [20:41:10] quiddity: that FF - argh... now I cannot load HIstory page, another day there were 'request is too long' stuff - just randomly and not reproducible [20:41:36] etonkovidova: which history page can't you load? [20:42:19] stephanebisson: http://en.wikipedia.beta.wmflabs.org/w/index.php?title=Talk:ET24&action=history - now it seems to be ok - was some api error [21:00:32] 3Collaboration-Team-Current, 10Flow, 5Patch-For-Review, 5WMF-deploy-2015-11-03_(1.27.0-wmf.5), 5WMF-deploy-2015-11-10_(1.27.0-wmf.6): Edit post and Summarize fail in firefox - https://phabricator.wikimedia.org/T117698#1786697 (10Etonkovidova) Checked in betalabs. [21:21:14] (03PS1) 10Sbisson: Presentation model for new-topic notification [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251411 (https://phabricator.wikimedia.org/T116845) [21:25:29] (03PS1) 10Sbisson: Flow post-reply presentation model [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251412 (https://phabricator.wikimedia.org/T116845) [21:26:29] (03CR) 10jenkins-bot: [V: 04-1] Presentation model for new-topic notification [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251411 (https://phabricator.wikimedia.org/T116845) (owner: 10Sbisson) [21:29:27] (03PS2) 10Sbisson: Presentation model for new-topic notification [extensions/Flow] - 10https://gerrit.wikimedia.org/r/251411 (https://phabricator.wikimedia.org/T116845) [21:30:53] 3Collaboration-Team-Current, 10Echo, 10Flow, 5Patch-For-Review: Write presentation models for notifications in Flow - https://phabricator.wikimedia.org/T116845#1786806 (10SBisson) [21:31:50] (03PS5) 10Sbisson: Add browser tests for adding/removing categories [extensions/Flow] - 10https://gerrit.wikimedia.org/r/246794 (owner: 10Mooeypoo) [21:41:26] etonkovidova, it was on MediaWiki.org. [21:41:42] matt_flaschen: ok [21:48:27] (03PS1) 10Mattflaschen: Add artificial primary key to flow_wiki_ref and flow_ext_ref [extensions/Flow] (wmf/1.27.0-wmf.4) - 10https://gerrit.wikimedia.org/r/251415 (https://phabricator.wikimedia.org/T109676) [21:48:53] (03PS1) 10Mattflaschen: Add artificial primary key to flow_wiki_ref and flow_ext_ref [extensions/Flow] (wmf/1.27.0-wmf.5) - 10https://gerrit.wikimedia.org/r/251416 (https://phabricator.wikimedia.org/T109676) [21:49:19] (03CR) 10Sbisson: [C: 032] Add browser tests for adding/removing categories [extensions/Flow] - 10https://gerrit.wikimedia.org/r/246794 (owner: 10Mooeypoo) [21:50:39] (03CR) 10jenkins-bot: [V: 04-1] Add artificial primary key to flow_wiki_ref and flow_ext_ref [extensions/Flow] (wmf/1.27.0-wmf.4) - 10https://gerrit.wikimedia.org/r/251415 (https://phabricator.wikimedia.org/T109676) (owner: 10Mattflaschen) [21:50:48] (03CR) 10Sbisson: [C: 032] Refactor CAPTCHA support into data model and widget [extensions/Flow] - 10https://gerrit.wikimedia.org/r/234423 (https://phabricator.wikimedia.org/T109000) (owner: 10Mattflaschen) [21:52:27] 6Collaboration-Team-Backlog, 10Echo, 10MediaWiki-skins-Metrolook: Echo looks wrong with metrolook installed - https://phabricator.wikimedia.org/T117921#1786886 (10Paladox) 3NEW [21:53:24] (03Merged) 10jenkins-bot: Add browser tests for adding/removing categories [extensions/Flow] - 10https://gerrit.wikimedia.org/r/246794 (owner: 10Mooeypoo) [21:54:20] (03Merged) 10jenkins-bot: Refactor CAPTCHA support into data model and widget [extensions/Flow] - 10https://gerrit.wikimedia.org/r/234423 (https://phabricator.wikimedia.org/T109000) (owner: 10Mattflaschen) [21:55:01] 6Collaboration-Team-Backlog, 10Echo, 10MediaWiki-skins-Metrolook: Echo looks wrong with metrolook installed - https://phabricator.wikimedia.org/T117921#1786905 (10Paladox) @Ashley can I add you to this task please. [21:55:06] 6Collaboration-Team-Backlog, 10Echo, 10MediaWiki-skins-Metrolook: Echo looks wrong with metrolook installed - https://phabricator.wikimedia.org/T117921#1786907 (10Paladox) [21:57:42] (03CR) 10Mattflaschen: "recheck" [extensions/Flow] (wmf/1.27.0-wmf.4) - 10https://gerrit.wikimedia.org/r/251415 (https://phabricator.wikimedia.org/T109676) (owner: 10Mattflaschen) [21:58:46] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Run FlowCreateTemplates.php on pt.wikibooks - https://phabricator.wikimedia.org/T114541#1786914 (10Etonkovidova) The templates from FlowCreateTemplates.php are present on https://pt.wikibooks.org/w/index.php?title=Especial%3ATodas+as+p%C3%A1gin... [21:59:25] Thanks for reviewing, stephanebisson. [22:00:09] matt_flaschen: I reviewed it a while ago. I wanted to give other people a chance to look at it... but that time has expired ;) [22:00:52] matt_flaschen: same thing for "link in topic title" [22:00:54] 6Collaboration-Team-Backlog, 10Echo, 10MediaWiki-skins-Metrolook: Echo looks wrong with metrolook installed - https://phabricator.wikimedia.org/T117921#1786922 (10Mattflaschen) p:5Triage>3Lowest Collaboration team won't be working on this, but patches will be accepted. [22:04:00] 6Collaboration-Team-Backlog, 10Echo, 10MediaWiki-skins-Metrolook: Echo looks wrong with metrolook installed - https://phabricator.wikimedia.org/T117921#1786935 (10Paladox) Found problem. But still need help on how to remove that bell type icon so only 0 is shown. [22:04:02] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Determine cause of historical UIDs colliding - https://phabricator.wikimedia.org/T117907#1786936 (10Mattflaschen) [22:04:07] matt_flaschen: I see that "links in topic title" in still in Dev on the board but the patch has been there and reviewed for a while, are we waiting on something or we just need someone to +2? [22:06:48] stephanebisson, no, I just forgot to move it. My bad. [22:09:16] ok, I'll ask matthias to review it on monday, I'm not comfortable merging it right now [22:10:38] 3Collaboration-Team-Current, 10Flow, 7Database, 5Patch-For-Review, 7WorkType-NewFunctionality: Deploy artificial primary key reference change to all production wikis simultaneously - https://phabricator.wikimedia.org/T117785#1786963 (10Mattflaschen) Scheduled for SWAT: https://wikitech.wikimedia.org/wiki... [22:14:12] 6Collaboration-Team-Backlog, 10Echo: Echo text is way too big - https://phabricator.wikimedia.org/T117922#1786975 (10Paladox) 3NEW [22:15:13] stephanebisson, sounds good. We don't want to merge it Monday, though. [22:15:32] sure, Tuesday pm [22:16:38] 6Collaboration-Team-Backlog, 10Echo: Make echo colourful - https://phabricator.wikimedia.org/T117923#1786995 (10Paladox) 3NEW [22:22:37] 6Collaboration-Team-Backlog, 10Echo: Echo text is way too big - https://phabricator.wikimedia.org/T117922#1787029 (10Paladox) {F2920203} Browser is ie on windows 10. [22:23:00] matt_flaschen: Did you guys end up figuring out the DB error problem [22:23:02] ? [22:23:06] stephanebisson: Thanks for taking on some CR [22:24:32] 6Collaboration-Team-Backlog, 10Echo: Echo text is way too big - https://phabricator.wikimedia.org/T117922#1787032 (10Paladox) Partially because of css .mw-echo-ui-notificationBadgeButtonPopupWidget .oo-ui-popupWidget but not sure what other css is doing it. [22:25:19] RoanKattouw: please take a look at my comment on https://phabricator.wikimedia.org/T116890 - most likely it's fine... [22:27:18] RoanKattouw, the UID collisions? I'm working on it. [22:27:28] 3Collaboration-Team-Current, 10Echo, 10MediaWiki-Internationalization, 10MediaWiki-extensions-CLDR, 7Wikimedia-log-errors: wgUserLanguage in JS should be valid for API (Exception: Invalid language code "nl_nl") - https://phabricator.wikimedia.org/T116890#1787033 (10Catrope) >>! In T116890#1784272, @Etonk... [22:27:49] matt_flaschen: No I mean the ") git review -d matt's patch 2) F5 3) DB error 4) cry" problem [22:27:49] RoanKattouw, or history pagination? [22:27:52] Yeah [22:28:00] Jinx. Yeah, I'll get back to that after LQT. [22:28:11] OK [22:28:33] Because Stephane and you seemed to be saying it's almost ready to merge and I was like "wait isn't there a DB error" [22:31:01] RoanKattouw: the "links in topic title" patch is the one that's ready to merge [22:32:23] Oooh right and that's not the one with the DB error [22:41:11] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Determine cause of historical UIDs colliding - https://phabricator.wikimedia.org/T117907#1787058 (10Mattflaschen) If there are 52 free bits, there should be 2^52 possibilities. Assume those are random (not entirely true since one of them start... [23:08:46] did OO.ui.LookupElement disappear? https://www.mediawiki.org/wiki/OOjs_UI/Elements/Lookup [23:09:56] jdlrobson, no, doesn't look like it. [23:10:07] jdlrobson, it's a mixin you know. [23:10:28] so i'm guessing this equates to [BREAKING CHANGE] Remove aliases for OO.ui.mixins (C. Scott Ananian) ? [23:10:43] It's not clear from the email what i'm supposed to do instead though [23:10:48] jdlrobson, oh, yeah. [23:10:50] Just put: [23:11:02] OO.ui.mixin.LookupElement [23:11:03] instead of: [23:11:05] OO.ui.LookupElement [23:11:11] ahh [23:11:14] okay [23:12:01] great that works! please consider putting upgrade notes in the future release emails [23:12:18] i'm not too oojs ui savvy yet to follow all these things [23:12:46] jdlrobson: It was in the release e-mail sent in June. [23:12:55] jdlrobson: When the old name was deprecated. [23:13:07] JamesF i saw that please read above [23:13:13] what i'm saying is there was no path for how to upgrade [23:13:17] just that a breaking change was made [23:14:03] just a suggested improvement for future release notes (i appreciate you sending them) [23:14:54] We did a search and fix before I sent the announcement. [23:14:59] Did we miss one? Sorry. [23:17:06] James_F: it was a gadget [23:17:25] James_F, appreciate you doing these search/replaces. Gadgets are searchable with mwgrep, BTW. [23:17:29] Ah. Yeah, that would do it. [23:17:31] But I understand it's more work to update those. [23:17:34] matt_flaschen: I don't have access. [23:17:41] so nothing major, just wanted to know how to fix it - and going forward as our community uses it more we'll have to be more careful how we advertise these breaking changes [23:17:46] [2015-11-05 22:53:14] Fatal error: Call to undefined function wfTimestamp() at /srv/mediawiki/php-1.27.0-wmf.5/extensions/Flow/maintenance/collision_script_T117907.php on line 7 [23:18:18] ori, yeah, I am testing something with mwscript, and I was missing boilerplate. Still trying to get it working. It's not a real script (doesn't touch DB or anything), just testing UID generation. [23:18:27] cool [23:18:32] just making sure you're aware [23:18:37] Yep [23:32:01] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Determine cause of historical UIDs colliding - https://phabricator.wikimedia.org/T117907#1787195 (10Mattflaschen) The following script is running on terbium. So far it's at 653000. {P2284} [23:33:58] jdlrobson: Yeah, sorry. [23:34:26] No worries this time, just suggesting improvements for future. [23:42:43] legoktm, mooeypoo, does the code in https://phabricator.wikimedia.org/T117907 look right to you? [23:44:29] what are you trying to test? how long it takes for a collision? [23:45:30] 3Collaboration-Team-Current, 10Flow, 7WorkType-NewFunctionality: Determine cause of historical UIDs colliding - https://phabricator.wikimedia.org/T117907#1787255 (10Mattflaschen) I stopped it at 1300000. If it's not generating duplicate UIDs, something may be causing it to actually try to insert the same en... [23:45:55] legoktm, yeah. [23:46:08] hmm [23:46:11] yes [23:46:12] legoktm: disabling Preferences->Notification->Flow will "wipe" all previous messages from Messages flyout-'There are no notifications' is displayed, but all Alerts are in place [23:46:23] legoktm: is it OK/ [23:46:28] legoktm: ? [23:46:31] matt_flaschen, yeah it looks good to me [23:46:36] etonkovidova, do they actually get wiped, or just not displayed? [23:46:38] etonkovidova: yes, that's expected [23:46:44] er [23:46:47] If you turn it back on do they re-appear? [23:46:50] right, they shouldn't be wiped from the db [23:46:50] matt_flaschen, i wonder if it'll break after 24 uuids [23:46:52] just hidden [23:46:52] :D [23:47:08] legoktm: matt_flaschen: they are not displayed ... hmm... looks strange though [23:47:14] mooeypoo, I don't get it, is that a Jack Bauer joke? :) [23:47:23] oh, sorry, 23 [23:47:32] no, that's the 50% cutoff for the Birthday Problem [23:47:54] mooeypoo, oh yeah. Did you guess before Roan gave it away? :) [23:48:10] I did not guess, no, but I remembered part of the problem [23:48:15] I was just confused about the use of "Paradox" [23:48:28] it's not a paradox. The universe did not implode. [23:48:37] mooeypoo, yeah, it's not really a paradox, but then again nothing is with consistent assumptions. ;) [23:48:53] Wikipedia calls it the "birthday problem". [23:49:07] matt_flaschen, given that most discrete logic starts with "Assume...." that would make all of it not a paradox [23:49:22] yeah i just saw that [23:49:38] Though they are mentioning "In probability theory, the birthday problem or birthday paradox[1]..." [23:49:40] mooeypoo, yeah, except if your assumptions conflict I think you can still have paradoxes. [23:50:16] Assume time is linear. Assume time travel is possible. <-- no conflicts. Assume you go back in time to kill your grandfather.... [23:50:38] Though one *could* make the point that the first two assumptions do conflict (which is what physicists do, really... and so does "Back to the Future" series) [23:51:06] but yes, it's the same problem I have with the so called "Twin Paradox" [23:51:10] it's not a paradox. [23:51:59] mooeypoo, it is if you grandfather is immortal and your twin is two years older than you *before* you start going to space. [23:52:45] Highlander: Messing up special relativity since 1592. [23:53:06] You know, my first instinct was to object to your second statement, but then my brain started considering hibernation while pregnant scenarios, and that actually can be valid. [23:53:20] Though probably extremely inconvenient [23:54:16] I will resist the obvious joke, and instead just be impressed you came up with that solution to my twin paradox-paradox so quickly. [23:55:10] If anyone can think of anything on the UID thing, let me know. [23:55:33] legoktm, so, I'm implementing "mark unread" in the API. I already implemented markUnread in UserNotificationGateway for the DB, but now I'm trying to do that in NotifUser.php... but I'm looking at markRead() function, and I am not sure if I should reverse some of the things in it ... after it asks the DB for updating the read status it deletes the records from "echo_target_page" and updates notification count in cache... [23:55:37] I can update the cache [23:55:58] but what do I do with the echo_target_page, and do I need to add to those recogrds instead of deleting in the "markUnRead" case ? [23:56:14] matt_flaschen, UID Hibernation ? [23:56:16] :D [23:56:57] ... actually, crap, that is not a bad idea. If the UUID is exactly the same and you need to find a new one, wait a nano second and regenerate it -- it will be different based on timestamps. [23:57:10] Hence....UID hibernation. [23:57:24] mooeypoo, except we don't actually use the current timestamp for this path. :( [23:57:33] oh. nevermind then. [23:57:35] We use the original second-level timestamp, then randomize the millisecond [23:57:45] Hm. [23:57:51] Can we randomize until it's unique? [23:58:03] if you find a non unique one, generate a new one and repeat the test [23:58:13] That was what I was thinking originally. [23:58:24] But I'm actually now skeptical that the generation itself is a problem. [23:58:35] Because it doesn't seem to generate collisions easily. [23:58:57] I'm thinking there could be some other bug, e.g. double-insertions in some edge case, though i don't know why it would only be coming up now. [23:59:14] hm [23:59:43] Still, the regeneration should fix it and point to trends