You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Due to our limited ability to test distribution queries so far, I think there are pieces of this functionality we've overlooked.
Currently Relay performs results aggregation and obfuscation on the Count property of results submitted by sub nodes. This works well for Availability queries.
With other types of analyses (here focusing on Distribution) a results file is submitted, with many rows and counts.
Relay will definitely need to aggregate the row counts in these files for correct distribution representation.
Relay may need to apply obfuscation to individual row values
Acceptance Criteria
Distribution queries are handled successfully end to end
Results files submitted for Distribution Queries are correctly aggregated for more than one downstream sub node.
Results files submitted for Distribution Queries are correctly obfuscated by Relay
Tests pass.
Documentation added.
Confirm creation
This issue is ready
The text was updated successfully, but these errors were encountered:
beforan
transferred this issue from Health-Informatics-UoN/hutch-cohort-discovery
Jan 24, 2025
Is this the right issue type?
Summary
Due to our limited ability to test distribution queries so far, I think there are pieces of this functionality we've overlooked.
Currently Relay performs results aggregation and obfuscation on the
Count
property of results submitted by sub nodes. This works well for Availability queries.With other types of analyses (here focusing on Distribution) a results file is submitted, with many rows and counts.
Relay will definitely need to aggregate the row counts in these files for correct distribution representation.
Relay may need to apply obfuscation to individual row values
Acceptance Criteria
Confirm creation
The text was updated successfully, but these errors were encountered: