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

Set LUX sampling to 1% #2145

Merged
merged 1 commit into from
Jun 17, 2021
Merged

Set LUX sampling to 1% #2145

merged 1 commit into from
Jun 17, 2021

Conversation

injms
Copy link
Contributor

@injms injms commented Jun 17, 2021

What

LUX defaults to 100% of events being sent - setting this to 1% will reduce the number of events to the same level that has been set in Speedcurve's admin settings.

Issue on static - alphagov/static#2516

Why

Approximately 29 million events were sent over the course of one day, of which only only 1% were analysed. To avoid the unnecessary sending of events we can only send 1% of events and set Speedcurve to analyse them all.

Changes

  1. Got to https://govuk-publis-update-lux-j6z85j.herokuapp.com/public/
  2. Accept all cookies
  3. Open browser console, and type LUX.getDebug()

An array is returned, one of which shows the sampling rate of 1%, as shown in the screenshot below:
Screenshot 2021-06-17 at 12 58 04

@bevanloon bevanloon temporarily deployed to govuk-publis-update-lux-j6z85j June 17, 2021 12:08 Inactive
@bevanloon bevanloon temporarily deployed to govuk-publis-update-lux-j6z85j June 17, 2021 12:41 Inactive
@bevanloon bevanloon temporarily deployed to govuk-publis-update-lux-j6z85j June 17, 2021 12:42 Inactive
@bevanloon bevanloon temporarily deployed to govuk-publis-update-lux-j6z85j June 17, 2021 13:18 Inactive
LUX defaults to 100% of events being sent - setting this to 1% will
reduce the number of events to the same level that has been set in
Speedcurve's admin settings.
@bevanloon bevanloon temporarily deployed to govuk-publis-update-lux-j6z85j June 17, 2021 13:19 Inactive
@injms injms requested a review from chao-xian June 17, 2021 13:20
@injms injms merged commit 44aa620 into master Jun 17, 2021
@injms injms deleted the update-lux-sample-rate branch June 17, 2021 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants