[03:18:22] (03CR) 10Yaron Koren: "I still think the commit message needs to change more. The handling of multiple values is not really what this patch changes - that's just" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [03:49:26] (03PS7) 10Sahajsk: Enable multiple HOLDS clauses in a single query. [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 [08:58:53] (03CR) 10Matěj Suchánek: [C: 04-1] [FEAT] Add support for Lexicographical data (032 comments) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/461372 (https://phabricator.wikimedia.org/T189321) (owner: 10Rua) [10:18:20] (03PS1) 10Xqt: [IMPR] use float for statistics [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598227 [10:29:01] (03CR) 10Xqt: "Is this still an active branch?" [pywikibot/core] - 10https://gerrit.wikimedia.org/r/595987 (owner: 10Andre Engels) [10:29:22] (03CR) 10Xqt: "Is this still an active branch?" [pywikibot/core] - 10https://gerrit.wikimedia.org/r/595988 (owner: 10Andre Engels) [10:59:10] 10Pywikibot: Make Pywikibot daemon logs unexecutable - https://phabricator.wikimedia.org/T253472 (10Tacsipacsi) [11:28:25] (03CR) 10Xqt: "Please deactivate style checks as I proposed by my uploaded patch." (0313 comments) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/597350 (owner: 10Andre Engels) [12:21:28] (03CR) 10Huji: [C: 03+2] [IMPR] Check for missing generator after setup() call [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598157 (owner: 10Xqt) [12:23:41] (03Merged) 10jenkins-bot: [IMPR] Check for missing generator after setup() call [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598157 (owner: 10Xqt) [12:25:58] (03PS1) 10Xqt: [IMPR] Make Pywikibot daemon logs unexecutable [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) [12:27:09] 10Pywikibot, 10Patch-For-Review: Make Pywikibot daemon logs unexecutable - https://phabricator.wikimedia.org/T253472 (10Xqt) a:03Xqt [12:27:26] 10Pywikibot, 10Patch-For-Review: Make Pywikibot daemon logs unexecutable - https://phabricator.wikimedia.org/T253472 (10Xqt) p:05Triage→03Low [13:39:22] (03CR) 10Yaron Koren: "I still think the commit message needs to change more. I mean, this patch changes the whole underlying SQL for HOLDS queries. Of course, t" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [13:51:19] (03PS28) 10Xqt: [script] port speedy_delete.py to core [pywikibot/core] - 10https://gerrit.wikimedia.org/r/148646 (https://phabricator.wikimedia.org/T66880) (owner: 10Ricordisamoa) [13:51:27] (03CR) 10jerkins-bot: [V: 04-1] [script] port speedy_delete.py to core [pywikibot/core] - 10https://gerrit.wikimedia.org/r/148646 (https://phabricator.wikimedia.org/T66880) (owner: 10Ricordisamoa) [13:51:45] (03CR) 10Xqt: "recheck" [pywikibot/core] - 10https://gerrit.wikimedia.org/r/148646 (https://phabricator.wikimedia.org/T66880) (owner: 10Ricordisamoa) [13:53:26] (03PS29) 10Xqt: [script] port speedy_delete.py to core [pywikibot/core] - 10https://gerrit.wikimedia.org/r/148646 (https://phabricator.wikimedia.org/T66880) (owner: 10Ricordisamoa) [13:55:03] (03PS20) 10Xqt: Porting parser_function_count.py from compat to core/scripts [pywikibot/core] - 10https://gerrit.wikimedia.org/r/181724 (https://phabricator.wikimedia.org/T66878) (owner: 10Prianka) [13:59:13] (03PS14) 10Xqt: [IMPR] Derive RedirectRobot from MultipleSitesBot [pywikibot/core] - 10https://gerrit.wikimedia.org/r/456998 (https://phabricator.wikimedia.org/T211586) [14:01:07] (03PS8) 10Sahajsk: Enable multiple HOLDS clauses in a single query. [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 [14:03:26] (03CR) 10Sahajsk: "> Patch Set 7:" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [14:07:04] (03PS1) 10Xqt: [doc] Update ROADMAP.rst [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598255 [14:09:58] (03CR) 10Yaron Koren: "It doesn't need to be longer (the current commit message might be too long already), just different. For example, it's not true that this " [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [14:19:09] (03PS9) 10Sahajsk: Change in SQL querying of HOLDS clause [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 [14:24:01] 10Pywikibot: Massive image upload with pywikibot - https://phabricator.wikimedia.org/T216015 (10Xqt) > upload of images by means of pywikibot is a task that takes several seconds per image (I assume this is a constraint set by any kind of policy) This is the usual put_throttle delay for each write operation (he... [14:25:02] 10Pywikibot: Massive image upload with pywikibot - https://phabricator.wikimedia.org/T216015 (10Xqt) 05Open→03Invalid [14:28:26] (03CR) 10Yaron Koren: "Better! But still not ideal. Sorry to keep pushing back - though coming up with good commit messages is important, so hopefully you're lea" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [14:37:42] (03CR) 10Sahajsk: "> Patch Set 9:" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [14:53:46] (03CR) 10Yaron Koren: "Yes. I'm not saying that's the best commit message, but it's at least more informative." [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [15:00:49] (03PS10) 10Sahajsk: Change in SQL querying of HOLDS clause [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 [15:13:43] (03CR) 10Yaron Koren: "You still haven't changed the main commit message... the top sentence is the most important one, because that's the one that shows up in c" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [15:24:37] (03CR) 10Tacsipacsi: "Thanks for the fast action! I have one minor suggestion." (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) (owner: 10Xqt) [16:23:25] (03CR) 10Xqt: [IMPR] Make Pywikibot daemon logs unexecutable (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) (owner: 10Xqt) [16:59:48] (03CR) 10Tacsipacsi: [IMPR] Make Pywikibot daemon logs unexecutable (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) (owner: 10Xqt) [18:19:41] (03PS1) 10Xqt: [IMPR] Throttle requests after ratelimits exceeded [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598269 (https://phabricator.wikimedia.org/T253180) [18:20:14] 10Pywikibot, 10Patch-For-Review: touch.py does not throttle for rate limits correctly - https://phabricator.wikimedia.org/T253180 (10Xqt) a:03Xqt [18:21:47] (03CR) 10jerkins-bot: [V: 04-1] [IMPR] Throttle requests after ratelimits exceeded [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598269 (https://phabricator.wikimedia.org/T253180) (owner: 10Xqt) [18:22:01] 10Pywikibot, 10Patch-For-Review: "ratelimited" API error does not throttle for rate limits - https://phabricator.wikimedia.org/T253180 (10Xqt) [18:23:13] (03PS2) 10Xqt: [IMPR] Throttle requests after ratelimits exceeded [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598269 (https://phabricator.wikimedia.org/T253180) [18:28:48] (03PS2) 10Xqt: [IMPR] Make Pywikibot daemon logs unexecutable [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) [18:30:23] (03CR) 10Xqt: [IMPR] Make Pywikibot daemon logs unexecutable (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) (owner: 10Xqt) [19:01:40] (03CR) 10Huji: [C: 03+2] [cleanup] warnfile.py wasn't ported to core [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598180 (owner: 10Xqt) [19:01:47] (03CR) 10jerkins-bot: [V: 04-1] [cleanup] warnfile.py wasn't ported to core [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598180 (owner: 10Xqt) [19:06:51] (03CR) 10Huji: [C: 03+2] [IMPR] Enable tuple for regex parameter in RegexFilter [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598199 (owner: 10Xqt) [19:09:15] (03Merged) 10jenkins-bot: [IMPR] Enable tuple for regex parameter in RegexFilter [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598199 (owner: 10Xqt) [19:29:26] 10Pywikibot, 10Patch-For-Review: "ratelimited" API error does not throttle for rate limits - https://phabricator.wikimedia.org/T253180 (10JJMC89) Re 598269 PS2: The patch iterates a specific set of groups and calculates the delay based on the first one hit. It should iterate the groups in the response and cal... [19:29:51] (03CR) 10JJMC89: [C: 04-1] [IMPR] Throttle requests after ratelimits exceeded (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598269 (https://phabricator.wikimedia.org/T253180) (owner: 10Xqt) [20:43:39] (03CR) 10Xqt: "recheck" [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598180 (owner: 10Xqt) [20:49:13] 10Pywikibot, 10Patch-For-Review: "ratelimited" API error does not throttle for rate limits - https://phabricator.wikimedia.org/T253180 (10Xqt) Many thanks for this response sample [21:00:02] (03CR) 10Tacsipacsi: [C: 03+1] [IMPR] Make Pywikibot daemon logs unexecutable (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) (owner: 10Xqt) [21:05:06] (03CR) 10Xqt: [C: 03+2] [IMPR] Make Pywikibot daemon logs unexecutable [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) (owner: 10Xqt) [21:05:30] 10Pywikibot, 10Patch-For-Review: Make Pywikibot daemon logs unexecutable - https://phabricator.wikimedia.org/T253472 (10Xqt) 05Open→03Resolved [21:07:17] (03Merged) 10jenkins-bot: [IMPR] Make Pywikibot daemon logs unexecutable [pywikibot/core] - 10https://gerrit.wikimedia.org/r/598238 (https://phabricator.wikimedia.org/T253472) (owner: 10Xqt) [23:47:31] (03PS11) 10Sahajsk: Use of SQL subqueries for every HOLDS occurrence [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 [23:49:25] (03CR) 10Sahajsk: "> Patch Set 10:" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk) [23:57:24] (03CR) 10Yaron Koren: "Oh! Sorry for the confusion. The commit message is looking better now! Although now there's the opposite problem, where it doesn't mention" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/597267 (owner: 10Sahajsk)