1694847 - Remove or update probes expiring in Firefox 89: SCROLL_INPUT_METHODS

Status: RESOLVED FIXED

(Core :: Panning and Zooming, task)

Description

[telemetry-probes]

The following Firefox probes will expire in the next major Firefox nightly release: version 89 [1].

SCROLL_INPUT_METHODS
apz.scrollwheel_overshoot

What to do about this:

1. If one, some, or all of the metrics are no longer needed, please remove them from their definitions files (Histograms.json, Scalars.yaml, Events.yaml).
2. If one, some, or all of the metrics are still required, please submit a Data Collection Review [2] and patch to extend their expiry. There is a shorter form for data collection renewal [3].

If you have any problems, please ask for help on the #data-help Slack channel or the #telemetry Matrix room at https://chat.mozilla.org/#/room/#telemetry:mozilla.org. We'll give you a hand.

Your Friendly, Neighborhood Telemetry Team

[1] https://wiki.mozilla.org/Release_Management/Calendar
[2] https://wiki.mozilla.org/Firefox/Data_Collection
[3] https://github.com/mozilla/data-review/blob/master/renewal_request.md

This is an automated message sent from probe-scraper. See https://github.com/mozilla/probe-scraper for details.

Comment 1

[Botond Ballo [:botond]]

Eric, my understanding based on bug 1672576 comment 4 is that you were interested in these probes for scrolling experimentation. Would you like them renewed further?

Comment 2

[esmyth]

While potentially useful in the future, I wasn't able to use the probes for the experiment and didn't have time to validate them separately. I've transitioned out of perf, so ni'ing Bas regarding keeping/future use.

Comment 3

[Bas Schouten (:bas.schouten)]

(In reply to Botond Ballo [:botond] from comment #1)

Eric, my understanding based on bug 1672576 comment 4 is that you were interested in these probes for scrolling experimentation. Would you like them renewed further?

Realistically I don't see us getting to this in the short term. Especially with Kats gone, unless someone closer to APZ is interested in working on our user facing scroll behavior, we can let these expire.

Comment 4

[Botond Ballo [:botond]]

Linking a few other related bugs:

• bug 1660933, in which mousewheel scroll animations were sped up
• bug 1670003, which re-added the telemetry probes in question in part to help inform whether we want to keep the speed-up (as user feedback was mixed)
• bug 1680077, which tracks possibly reverting the speedup

I haven't followed these discussions in detail, and so I'm not sure exactly how the probes were meant to inform a decision here. However, it would be nice to resolve the question of the animation speed one way or another.

Comment 5

[Kartikaya Gupta (email:[e-mail])]

I'm always happy to delete some code...

Comment 6

[Kartikaya Gupta (email:[e-mail])]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=48180c6f259a6d0e77755fccd7aecfbc6832fb75

Comment 7

[Kartikaya Gupta (email:[e-mail])]

Attachment: Bug 1694847 - Remove the expired SCROLL_INPUT_METHODS telemetry probe. r?botond

Comment 8

[Kartikaya Gupta (email:[e-mail])]

Attachment: Bug 1694847 - Remove expired overshoot detector telemetry probe. r?botond

Depends on D116320

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9211e7bd2637
Remove the expired SCROLL_INPUT_METHODS telemetry probe. r=botond
https://hg.mozilla.org/integration/autoland/rev/b7e10a54750f
Remove expired overshoot detector telemetry probe. r=botond

Comment 10

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/9211e7bd2637
https://hg.mozilla.org/mozilla-central/rev/b7e10a54750f