In ClickBasedCategoryRanker make dismissed category penalty a variational parameter. |
||
Issue descriptionCurrently it is hardcoded to 2 (i.e. the dismissed category jumps 2 positions down). However, jkrcal@ raised a concern that this may be too dramatic and, thus, it would be nice to control it.
,
Jan 5 2017
,
Jan 17 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ecbf607e7aced540428687d67a9991d97d115e6e commit ecbf607e7aced540428687d67a9991d97d115e6e Author: vitaliii <vitaliii@chromium.org> Date: Tue Jan 17 06:59:11 2017 [NTP::SectionOrder] Make dismissed category penalty a variation param. Make dismissed category penalty a variation param instead of simply hardcoding the value. Handle penalty equal to 0 correctly (ignore OnCategoryDismissed call). We may want to change the value or to disable it completely (by setting to 0) later. BUG= 677919 Review-Url: https://codereview.chromium.org/2612773003 Cr-Commit-Position: refs/heads/master@{#443996} [modify] https://crrev.com/ecbf607e7aced540428687d67a9991d97d115e6e/components/ntp_snippets/category_rankers/click_based_category_ranker.cc [modify] https://crrev.com/ecbf607e7aced540428687d67a9991d97d115e6e/components/ntp_snippets/category_rankers/click_based_category_ranker_unittest.cc
,
Jan 17 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by vitaliii@chromium.org
, Jan 5 2017Owner: vitaliii@chromium.org
Status: Started (was: Available)