[01:29:42] (03PS5) 10Catrope: Only make hidenondamaging available if damaging is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/310475 [01:29:45] (03PS3) 10Catrope: Refactor and simplify changeslist/contribs queries a bit [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311652 [01:29:48] (03PS2) 10Catrope: Use aliases with "damaging" in them so we can add other ones (e.g. "goodfaith") [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312164 [01:29:51] (03PS2) 10Catrope: Only pull in damaging scores when damaging model is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312165 [01:29:54] (03PS2) 10Catrope: [WIP] Add goodfaith filtering [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312166 (https://phabricator.wikimedia.org/T137966) [01:29:57] (03PS2) 10Catrope: Clean up === false comparisons [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311651 [01:30:36] (03CR) 10jenkins-bot: [V: 04-1] Only make hidenondamaging available if damaging is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/310475 (owner: 10Catrope) [01:30:47] (03CR) 10jenkins-bot: [V: 04-1] Use aliases with "damaging" in them so we can add other ones (e.g. "goodfaith") [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312164 (owner: 10Catrope) [01:30:53] (03CR) 10jenkins-bot: [V: 04-1] Only pull in damaging scores when damaging model is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312165 (owner: 10Catrope) [01:31:02] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Add goodfaith filtering [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312166 (https://phabricator.wikimedia.org/T137966) (owner: 10Catrope) [01:31:37] (03CR) 10jenkins-bot: [V: 04-1] Refactor and simplify changeslist/contribs queries a bit [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311652 (owner: 10Catrope) [01:31:45] (03PS6) 10Catrope: Only make hidenondamaging available if damaging is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/310475 [01:33:13] (03PS4) 10Catrope: Refactor and simplify changeslist/contribs queries a bit [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311652 [01:33:21] (03PS3) 10Catrope: Use aliases with "damaging" in them so we can add other ones (e.g. "goodfaith") [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312164 [01:34:15] (03CR) 10jenkins-bot: [V: 04-1] Refactor and simplify changeslist/contribs queries a bit [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311652 (owner: 10Catrope) [01:34:18] (03CR) 10jenkins-bot: [V: 04-1] Use aliases with "damaging" in them so we can add other ones (e.g. "goodfaith") [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312164 (owner: 10Catrope) [01:35:16] (03PS3) 10Catrope: Only pull in damaging scores when damaging model is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312165 [01:36:26] (03CR) 10jenkins-bot: [V: 04-1] Only pull in damaging scores when damaging model is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312165 (owner: 10Catrope) [01:41:12] (03PS5) 10Catrope: Refactor and simplify changeslist/contribs queries a bit [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311652 [01:41:15] (03PS4) 10Catrope: Use aliases with "damaging" in them so we can add other ones (e.g. "goodfaith") [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312164 [01:41:18] (03PS4) 10Catrope: Only pull in damaging scores when damaging model is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312165 [01:41:21] (03PS3) 10Catrope: [WIP] Add goodfaith filtering [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312166 (https://phabricator.wikimedia.org/T137966) [01:42:28] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Add goodfaith filtering [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312166 (https://phabricator.wikimedia.org/T137966) (owner: 10Catrope) [01:46:44] (03PS7) 10Catrope: Only make hidenondamaging available if damaging is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/310475 [01:46:47] (03PS6) 10Catrope: Refactor and simplify changeslist/contribs queries a bit [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311652 [01:46:50] (03PS5) 10Catrope: Use aliases with "damaging" in them so we can add other ones (e.g. "goodfaith") [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312164 [01:46:53] (03PS5) 10Catrope: Only pull in damaging scores when damaging model is enabled [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312165 [10:18:05] 06Revision-Scoring-As-A-Service, 10Edit-Review-Improvements, 10rsaas-editquality, 03Collab-Team-Q1-July-Sep-2016: Research how to present ORES scores to users in a way that is understandable and meets their reviewing goals - https://phabricator.wikimedia.org/T146333#2664201 (10Trizek-WMF) Ready for a page... [15:00:57] (03CR) 10Ladsgroup: "Hey, It seems you misunderstood the point of this maintenance script. We just added some unique constraints to ores_classification table b" [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312286 (https://phabricator.wikimedia.org/T145503) (owner: 10Ladsgroup) [16:04:44] 10Revision-Scoring-As-A-Service-Backlog, 10MediaWiki-extensions-ORES: Introduce ORES rvprop - https://phabricator.wikimedia.org/T143614#2664477 (10Ladsgroup) >>! In T143614#2652771, @Anomie wrote: > ** How many would be sane to fetch in one API request? We'd need to choose a limit that won't take too long to e... [16:08:45] 10Revision-Scoring-As-A-Service-Backlog, 10MediaWiki-extensions-ORES: Introduce rcshow=oresreview and similar ones - https://phabricator.wikimedia.org/T143616#2664479 (10Ladsgroup) For the sake of consistency with the GUI, I think we should follow #2. Number of revisions not stored in the db is negligible for... [22:31:05] (03PS1) 10Ladsgroup: Made PHPStorm happy on includes/Hooks.php [extensions/ORES] (wmf/1.28.0-wmf.20) - 10https://gerrit.wikimedia.org/r/312707 [22:33:09] (03Abandoned) 10Ladsgroup: Made PHPStorm happy on includes/Hooks.php [extensions/ORES] (wmf/1.28.0-wmf.20) - 10https://gerrit.wikimedia.org/r/312707 (owner: 10Ladsgroup) [22:33:16] (03PS1) 10Ladsgroup: Made PHPStorm happy on includes/Hooks.php [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312708 [22:42:26] (03PS2) 10Ladsgroup: Made PHPStorm happy on includes/Hooks.php [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312708 [22:49:21] (03PS3) 10Ladsgroup: Made PHPStorm happy on includes/Hooks.php [extensions/ORES] - 10https://gerrit.wikimedia.org/r/312708 [23:22:40] (03CR) 10Ladsgroup: [C: 032] "Usually changing ! to false is not good idea because of Boolean conversion (https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP#B" [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311651 (owner: 10Catrope) [23:24:03] (03Merged) 10jenkins-bot: Clean up === false comparisons [extensions/ORES] - 10https://gerrit.wikimedia.org/r/311651 (owner: 10Catrope) [23:56:44] 06Revision-Scoring-As-A-Service, 10MediaWiki-extensions-ORES, 15User-Ladsgroup: Extensive CI tests for ORES extension - https://phabricator.wikimedia.org/T146560#2664966 (10Ladsgroup)