-
Notifications
You must be signed in to change notification settings - Fork 162
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
Event-level reports should indicate how noise has been applied #273
Comments
I'm happy to spec and implement this. |
Do we want to have separate fields for source event ID noise probability and trigger data noise probability? |
Yes I think that should be fine for now. |
@csharrison Are we concerned about the potential for these values to be used for fingerprinting? |
No I am not concerned. Users of the API need to know what the API is doing, so I think defending against this is going to be very challenging and we won't be exposing anything more than browser major version anyways (in majority of cases) which is being sent by default: |
We should consider adding fields like:
metadata_noise_probability
orrandomized_response_probability
This will make it easier to handle multiple implementations with different probabilities (or different experiments in the same implementation). It also signals to developers that values may be noisy.
cc @maudnals @johnivdel
The text was updated successfully, but these errors were encountered: