Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

survey: adjust the survey prompt probability computation #2545

Closed
hyangah opened this issue Nov 22, 2022 · 3 comments
Closed

survey: adjust the survey prompt probability computation #2545

hyangah opened this issue Nov 22, 2022 · 3 comments
Assignees
Milestone

Comments

@hyangah
Copy link
Contributor

hyangah commented Nov 22, 2022

The current logic is biased and favors users who open editors for the first time near the end of the month. Our requirements are:

  • Prompting should be approximately uniformly distributed throughout the month and year. As much as possible, we should avoid cyclical behavior.

  • The sample of prompted users should approximate our 28d user base. That is to say that any 28d user is approximately equally likely to be prompted throughout the month. Obviously for users that only use VS Code once a month, this means they are more likely to be prompted in each individual session.

  • We should aim for 200-300 responses per month. Currently we're around 300.

cc @pjweinb @findleyr @fflewddur

@gopherbot gopherbot added this to the Untriaged milestone Nov 22, 2022
@hyangah hyangah modified the milestones: Untriaged, vscode-go/later Dec 2, 2022
@pjweinb
Copy link

pjweinb commented Dec 7, 2022 via email

@fflewddur
Copy link
Contributor

This sounds great to me. Very elegant solution, thanks @pjweinb!

gopherbot pushed a commit that referenced this issue Dec 12, 2022
Users are occasionally asked to take the gopls survey. The logic
is a little complicated, but in the existing code, if the user is
prompted, it will be for a random day in the rest of the current
month. In the new code all days of the year are pretty nearly equally
probable. The new code chooses a random day in the next 28 days.

There's nothing special about 28, but it's large enough to smooth out
day-of-the-week effects, and small enough that a chosen user
will be prompted for a survey within a month.

Fixes vscode-go/#2545

Change-Id: Ifaccaa5e8e4e8b1a1050768461ed38d4e144561e
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/456041
TryBot-Result: kokoro <noreply+kokoro@google.com>
Run-TryBot: Peter Weinberger <pjw@google.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
@hyangah hyangah modified the milestones: vscode-go/later, v0.37.0 Dec 13, 2022
@hyangah
Copy link
Contributor Author

hyangah commented Dec 13, 2022

Closed with go.dev/cl/457475.
Thanks @pjweinb

@hyangah hyangah closed this as completed Dec 13, 2022
@golang golang locked and limited conversation to collaborators Dec 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants