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

Redact categorical joined spatial aggregate results using per-category subscriber counts #1940

Open
jc-harrison opened this issue Feb 10, 2020 · 1 comment
Labels
bug Something isn't working FlowMachine Issues related to FlowMachine

Comments

@jc-harrison
Copy link
Member

Currently, joined spatial aggregate results are redacted using a count of the total number of subscribers in each region.

When aggregating a categorical metric (handset is the only one exposed at the moment, I think) using the 'distr' method, one row per category is returned for each location, giving the proportion of subscribers in each category. If the number of subscribers in region A who fall into category 3 is fewer than 15, results for region A should be redacted to preserve privacy. However, in the current implementation, region A results will only be redacted if the total number of subscribers in region A (across all categories) is fewer than 15.

@jc-harrison jc-harrison added bug Something isn't working FlowMachine Issues related to FlowMachine labels Feb 10, 2020
@greenape greenape added the P-Now label Sep 23, 2021
@jc-harrison
Copy link
Member Author

jc-harrison commented Dec 8, 2021

In light of #4677, I'd be inclined to drop the 'distr' method from JoinedSpatialAggregate altogether (or at least un-expose it) - 'distr' has to be handled completely differently to the other methods, and accepts a different set of metrics (which complicates the parameter validation), and the same result could be achieved by dividing the results of a LabelledSpatialAggregate by the results of an un-labelled SpatialAggregate.

@greenape greenape removed the P-Now label Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working FlowMachine Issues related to FlowMachine
Projects
None yet
Development

No branches or pull requests

2 participants