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

Issue 644716 link

Starred by 2 users
Status: Fixed
Owner:
jkrcal@chromium.org
Closed: Sep 2016
Cc:
█ ramyan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Feature

Blocking:
issue 644109



Sign in to add a comment

Measure data for adaptive fetching rates for server-side content suggestion on New Tab Page

Project Member Reported by jkrcal@chromium.org, Sep 7 2016 
In order to introduce adaptive fetching rates in M55 for server-side content suggestions on New Tab Page, we need to collect data about user usage patterns already in M54.

Collect per user averages of number of hours between two successive times when:
 - a NTP is opened,
 - the user scrolls below the fold,
 - the user opens a suggestion.

Later, we will use these averages (computed locally on the client) to classify the client and to tune his fetching rates accordingly.

Comment 1 by fi...@chromium.org, Sep 8 2016

Labels: M-55 zine-articles-v1 zine-triaged
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 8 2016 

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

commit e13510e5938155d4eb7b8a742919a11aa7f8556c
Author: jkrcal <jkrcal@chromium.org>
Date: Thu Sep 08 17:56:20 2016

Measure usage metrics to prepare for adaptive fetching rates in M55

In order to scale to 100% of users in M55, we need adaptive fetching
rates for server-side suggestions on the new tab page. This CL
implements reporting of usage metrics to UMA. We need to collect these
metrics already in M54 so that we have enough data to define default
parameters for the adaptive fetching rates in M55.

Design doc: https://docs.google.com/a/google.com/document/d/1tDl2Dx_ZPnSyoe7hAe55mT2uTeXDaFVphqsWZEjp3EQ/edit?usp=sharing

BUG= 644716 

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

[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/chrome/browser/android/ntp/ntp_snippets_bridge.cc
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/chrome/browser/prefs/browser_prefs.cc
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/BUILD.gn
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/content_suggestions_service.cc
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/content_suggestions_service.h
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/content_suggestions_service_unittest.cc
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/pref_names.cc
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/pref_names.h
[add] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/user_classifier.cc
[add] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/components/ntp_snippets/user_classifier.h
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/ios/chrome/browser/ntp_snippets/ios_chrome_content_suggestions_service_factory.cc
[modify] https://crrev.com/e13510e5938155d4eb7b8a742919a11aa7f8556c/tools/metrics/histograms/histograms.xml

Comment 3 by jkrcal@chromium.org, Sep 9 2016

Status: Fixed (was: Started)

Comment 4 by jkrcal@chromium.org, Sep 9 2016

Labels: zine-16-09-05
This still needs to get merged in M54, if possible.

Comment 5 by treib@chromium.org, Sep 14 2016

Labels: Merge-Request-54

Comment 6 by dimu@chromium.org, Sep 14 2016

Labels: -Merge-Request-54 Merge-Approved-54 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M54 (branch: 2840)
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 14 2016

Labels: -merge-approved-54 merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fac6b13156a4d9c89778334eb9013354ddef81ec

commit fac6b13156a4d9c89778334eb9013354ddef81ec
Author: Marc Treib <treib@chromium.org>
Date: Wed Sep 14 11:08:51 2016

Measure usage metrics to prepare for adaptive fetching rates in M55

In order to scale to 100% of users in M55, we need adaptive fetching
rates for server-side suggestions on the new tab page. This CL
implements reporting of usage metrics to UMA. We need to collect these
metrics already in M54 so that we have enough data to define default
parameters for the adaptive fetching rates in M55.

Design doc: https://docs.google.com/a/google.com/document/d/1tDl2Dx_ZPnSyoe7hAe55mT2uTeXDaFVphqsWZEjp3EQ/edit?usp=sharing

BUG= 644716 

Review-Url: https://codereview.chromium.org/2315273002
Cr-Commit-Position: refs/heads/master@{#417336}
(cherry picked from commit e13510e5938155d4eb7b8a742919a11aa7f8556c)

Review URL: https://codereview.chromium.org/2342553002 .

Cr-Commit-Position: refs/branch-heads/2840@{#361}
Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607}

[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/chrome/browser/android/ntp/ntp_snippets_bridge.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/chrome/browser/prefs/browser_prefs.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/BUILD.gn
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/content_suggestions_service.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/content_suggestions_service.h
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/content_suggestions_service_unittest.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/pref_names.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/pref_names.h
[add] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/user_classifier.cc
[add] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/user_classifier.h
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/ios/chrome/browser/ntp_snippets/ios_chrome_content_suggestions_service_factory.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/tools/metrics/histograms/histograms.xml
Project Member

Comment 8 by bugdroid1@chromium.org, Oct 27 2016 

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

commit fac6b13156a4d9c89778334eb9013354ddef81ec
Author: Marc Treib <treib@chromium.org>
Date: Wed Sep 14 11:08:51 2016

Measure usage metrics to prepare for adaptive fetching rates in M55

In order to scale to 100% of users in M55, we need adaptive fetching
rates for server-side suggestions on the new tab page. This CL
implements reporting of usage metrics to UMA. We need to collect these
metrics already in M54 so that we have enough data to define default
parameters for the adaptive fetching rates in M55.

Design doc: https://docs.google.com/a/google.com/document/d/1tDl2Dx_ZPnSyoe7hAe55mT2uTeXDaFVphqsWZEjp3EQ/edit?usp=sharing

BUG= 644716 

Review-Url: https://codereview.chromium.org/2315273002
Cr-Commit-Position: refs/heads/master@{#417336}
(cherry picked from commit e13510e5938155d4eb7b8a742919a11aa7f8556c)

Review URL: https://codereview.chromium.org/2342553002 .

Cr-Commit-Position: refs/branch-heads/2840@{#361}
Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607}

[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/chrome/browser/android/ntp/ntp_snippets_bridge.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/chrome/browser/prefs/browser_prefs.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/BUILD.gn
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/content_suggestions_service.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/content_suggestions_service.h
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/content_suggestions_service_unittest.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/pref_names.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/pref_names.h
[add] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/user_classifier.cc
[add] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/components/ntp_snippets/user_classifier.h
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/ios/chrome/browser/ntp_snippets/ios_chrome_content_suggestions_service_factory.cc
[modify] https://crrev.com/fac6b13156a4d9c89778334eb9013354ddef81ec/tools/metrics/histograms/histograms.xml
►

Sign in to add a comment