cwa-documentation
cwa-documentation copied to clipboard
Add Transmission Risk Level examples to Risk Calculation in Solution Architecture
Where to find the issue
The following document:
CORONA-WARN-APP SOLUTION ARCHITECTURE MOBILE APPLICATIONS Risk Calculation
Describe the issue
The Risk Calculation section in the CORONA-WARN-APP SOLUTION ARCHITECTURE document describes how the app calculates a risk score so that it is able to tell the user if they have been at risk of infection.
The blog post Behind the scenes of the Corona-Warn-App from December 30, 2020 links to a video and to a copy of the slides used in the presentation.
The video and slides contain some information which is missing from the Solution Architecture document. For instance, the table of TRL levels based on "Days since Onset of Symptoms" is missing in the Solution Architecture document. There are also no example calculations given in the Solution Architecture document.
Suggested change
Either link the Solution Architecture document to the presentation slides and video or add the information directly into the CORONA-WARN-APP SOLUTION ARCHITECTURE document.
Since the presentation and video are no longer completely up-to-date in terms of the details, it would need some comment about what has changed if this material is simply linked in.
Additional information
- This issue is related to the deprecation of the outdated document How does the Corona-Warn-App identify an increased risk? as proposed in PR https://github.com/corona-warn-app/cwa-documentation/pull/829.
- PR #845 is starting to address this issue.
Is this issue addressed? By the way, I do not think the information in the presentation from December 30, 2020 is up to date. Only transmission risk level >=3 is counted now.
@AnonymousUserUse
Is this issue addressed?
No. See the open PR https://github.com/corona-warn-app/cwa-documentation/pull/845 from @mlenkeit. I suspect that this issue is low in the current order of priorities.
@MikeMcC399 I think it is complete now with the merge of #845. If not, please re-open.
@AnonymousUserUse the Solution Architecture is updated now with the missing information. If you're just interested in the changes, you can take a look here.
@mlenkeit
Thanks you for the follow up. Issue closure is fine. 👍🏻