Add a metric for a number of category order changes. |
||||||
Issue descriptionWe want to see that the order "settles down" with time, so we need a metric, which would show how many times the order changes.
,
Feb 3 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7496904fab33e2c38f8c6ec6cfad27ebd158c1f4 commit 7496904fab33e2c38f8c6ec6cfad27ebd158c1f4 Author: vitaliii <vitaliii@chromium.org> Date: Fri Feb 03 09:27:41 2017 [NTP::SectionOrder] Track category index after it has been moved up. 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} [modify] https://crrev.com/7496904fab33e2c38f8c6ec6cfad27ebd158c1f4/components/ntp_snippets/category_rankers/click_based_category_ranker.cc [modify] https://crrev.com/7496904fab33e2c38f8c6ec6cfad27ebd158c1f4/components/ntp_snippets/category_rankers/click_based_category_ranker_unittest.cc [modify] https://crrev.com/7496904fab33e2c38f8c6ec6cfad27ebd158c1f4/components/ntp_snippets/content_suggestions_metrics.cc [modify] https://crrev.com/7496904fab33e2c38f8c6ec6cfad27ebd158c1f4/components/ntp_snippets/content_suggestions_metrics.h [modify] https://crrev.com/7496904fab33e2c38f8c6ec6cfad27ebd158c1f4/tools/metrics/histograms/histograms.xml
,
Feb 3 2017
Requesting to merge the CL from the comment #2 to M57.
,
Feb 3 2017
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
,
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
,
Feb 7 2017
,
Feb 7 2017
I verified, we get the data from Canary already.
,
Feb 7 2017
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 |
||||||
Comment 1 by vitaliii@chromium.org
, Feb 1 2017