New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 674851 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Feature

Blocking:
issue 646841



Sign in to add a comment

Implement new dynamic category ranker

Project Member Reported by vitaliii@chromium.org, Dec 16 2016

Issue description

We need to implement a new category ranker, which would provider personalized order.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 22 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9f8aa5f8fc0acdbb5f473687f547edcc342df687

commit 9f8aa5f8fc0acdbb5f473687f547edcc342df687
Author: vitaliii <vitaliii@chromium.org>
Date: Thu Dec 22 09:26:35 2016

[NTP::SectionOrder] First version of click based category ranker.

This CL adds an implementation of CategoryRanker based on clicks.

BUG= 674851 

Review-Url: https://codereview.chromium.org/2585263002
Cr-Commit-Position: refs/heads/master@{#440369}

[modify] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/chrome/browser/prefs/browser_prefs.cc
[modify] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/BUILD.gn
[add] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/category_rankers/click_based_category_ranker.cc
[add] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/category_rankers/click_based_category_ranker.h
[add] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/category_rankers/click_based_category_ranker_unittest.cc
[modify] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/category_rankers/constant_category_ranker.cc
[modify] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/category_rankers/constant_category_ranker.h
[modify] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/pref_names.cc
[modify] https://crrev.com/9f8aa5f8fc0acdbb5f473687f547edcc342df687/components/ntp_snippets/pref_names.h

Labels: zine-16-12-19
Status: Fixed (was: Assigned)
There are multiple issues with this implementation (see issue 646841 for the list), but all of them will be addressed as separate bugs.

Sign in to add a comment