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

Issue 687505 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Feature

Blocking:
issue 683901



Sign in to add a comment

Add a metric for a number of category order changes.

Project Member Reported by vitaliii@chromium.org, Feb 1 2017

Issue description

We want to see that the order "settles down" with time, so we need a metric, which would show how many times the order changes. 
 
Labels: zine-17-01-30
Labels: -M-58 M-57 Merge-Request-57
Requesting to merge the CL from the comment #2 to M57.
Project Member

Comment 4 by sheriffbot@chromium.org, Feb 3 2017

Labels: -Merge-Request-57 Hotlist-Merge-Approved Merge-Approved-57
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by sheriffbot@chromium.org, Feb 6 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: zine-17-02-06
Status: Fixed (was: Started)
Status: Verified (was: Fixed)
I verified, we get the data from Canary already.
Project Member

Comment 8 by bugdroid1@chromium.org, Feb 7 2017

Labels: -merge-approved-57 merge-merged-2987
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/59022d2e77967435d18cbe426f18793151d1eb9d

commit 59022d2e77967435d18cbe426f18793151d1eb9d
Author: Vitalii Iarko <vitaliii@chromium.org>
Date: Tue Feb 07 14:06:22 2017

Merge "[NTP::SectionOrder] Track category index after it has been ..."

This is a merge of https://codereview.chromium.org/2668063004 into M57
(branch 2987).

There were no conflicts, it has been tested on a device + unittests.

Original description:

Add a new metric to track category index after it has been moved up due
to a click. This is done in order to check how stable to order is and
whether it is becoming more stable.

BUG= 687505 

Review-Url: https://codereview.chromium.org/2668063004
Cr-Commit-Position: refs/heads/master@{#447968}
(cherry picked from commit 7496904fab33e2c38f8c6ec6cfad27ebd158c1f4)

Review-Url: https://codereview.chromium.org/2679133002 .
Cr-Commit-Position: refs/branch-heads/2987@{#360}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}

[modify] https://crrev.com/59022d2e77967435d18cbe426f18793151d1eb9d/components/ntp_snippets/category_rankers/click_based_category_ranker.cc
[modify] https://crrev.com/59022d2e77967435d18cbe426f18793151d1eb9d/components/ntp_snippets/category_rankers/click_based_category_ranker_unittest.cc
[modify] https://crrev.com/59022d2e77967435d18cbe426f18793151d1eb9d/components/ntp_snippets/content_suggestions_metrics.cc
[modify] https://crrev.com/59022d2e77967435d18cbe426f18793151d1eb9d/components/ntp_snippets/content_suggestions_metrics.h
[modify] https://crrev.com/59022d2e77967435d18cbe426f18793151d1eb9d/tools/metrics/histograms/histograms.xml

Sign in to add a comment