-
Notifications
You must be signed in to change notification settings - Fork 5.4k
[Telemetry] Add section about experiments #8965
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
base: main
Are you sure you want to change the base?
Conversation
docs/configure/telemetry.md
Outdated
|
||
VS Code uses an A/B experimentation system to roll out new features to a subset of users before making them generally available. This helps us validate that a new feature is working as expected across a diverse set of users before rolling it out to everyone. By participating in experimentation, you help us improve the quality of VS Code and can help shape the future of the product through early feedback. | ||
|
||
To enable this experimentation system, VS Code uses the usage data telemetry data to determine which users should receive the new feature. If you disable usage data telemetry by setting `setting(telemetry.telemetryLevel)` to `error`, `crash`, or `off`, experimentation is not available to you and the roll out of new features to you might be delayed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo "data telemetry data"
docs/configure/telemetry.md
Outdated
|
||
VS Code uses an A/B experimentation system to roll out new features to a subset of users before making them generally available. This helps us validate that a new feature is working as expected across a diverse set of users before rolling it out to everyone. By participating in experimentation, you help us improve the quality of VS Code and can help shape the future of the product through early feedback. | ||
|
||
To enable this experimentation system, VS Code uses the usage data telemetry data to determine which users should receive the new feature. If you disable usage data telemetry by setting `setting(telemetry.telemetryLevel)` to `error`, `crash`, or `off`, experimentation is not available to you and the roll out of new features to you might be delayed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would also explain why this is. For example "to validate if a feature is good, we need to see how users use the feature, thus users that disable telemetry are not part of feature roll-outs since we can not measure impact"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good. Left minor comments,
I suggest SteVen also gives it a review.
@sbatten Can you review the updates to the telemetry doc? Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, small comments
@sbatten Thanks for reviewing - applied your suggestions! |
@sbatten Can you do a final review to get this published. |
This pull request updates the telemetry documentation for Visual Studio Code to clarify how telemetry data is used, especially regarding feature rollouts and experimentation. The changes improve transparency about the role of telemetry in enabling early access to new features and update instructions and terminology for disabling telemetry.
Fixes #8949