Skip to content
This repository has been archived by the owner on Sep 4, 2023. It is now read-only.

Decoy rate based on installs will be to low #70

Open
3 tasks done
jorants opened this issue Sep 2, 2020 · 5 comments
Open
3 tasks done

Decoy rate based on installs will be to low #70

jorants opened this issue Sep 2, 2020 · 5 comments

Comments

@jorants
Copy link

jorants commented Sep 2, 2020

Describe the bug, issue or concern

The following seems to be the calculation for the decoy rate:
rate = (k * 20)/I
where k = number of key uploads, I= number of installs.

However, k is a measure of actual usage while I is not. Since I overestimates the actual usage the rate will end up to low.

This is easily fixed, to see this, base k on the expected number of uploads:

k = t*(I/P)

where t = positive tests, and P = whole population= 17M.
Substituting back we get

rate = (t*(I/P)*20)/I = (t/P)*20

Or simply: make the rate equal to the positive test rate times 20.

Governance

@damaarten
Copy link
Contributor

Thanks a lot for reviewing the specification and for this feedback!
The 1 to 20 rate is a ball-park ratio we agreed upon with the operational support team early in the project.
Most important about decoy traffic is that it should be indistinguishable from real uploads.
The rate is less important. If the rate were to be near to 1, that would make any upload traffic more likely to be a real upload. But this is not the case.
We can however adjust the decoy probability for all apps, it is a server configuration value that all apps fetch from the server whenever it changes, as part of the key download process. Once the app (system) is fully operational, we will periodically evaluate if we need to adjust this value, based on the number of app installs and daily infection rates.
So in summary, we are aware that the rate of 20 to 1 may turn out different in practice, and we will adjust the value when needed.
We are flexible in adjusting.
I will discuss in the team if we want to add to the spec that we will monitor the decoyProbability value and adjust when needed.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions
Copy link

github-actions bot commented Sep 8, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants