cwa-documentation icon indicating copy to clipboard operation
cwa-documentation copied to clipboard

Add Transmission Risk Level examples to Risk Calculation in Solution Architecture

Open MikeMcC399 opened this issue 2 years ago • 1 comments

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

MikeMcC399 avatar Jan 31 '22 11:01 MikeMcC399

  • PR #845 is starting to address this issue.

MikeMcC399 avatar Feb 07 '22 12:02 MikeMcC399

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 avatar Sep 20 '22 23:09 AnonymousUserUse

@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 avatar Sep 21 '22 04:09 MikeMcC399

@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 avatar Sep 29 '22 08:09 mlenkeit

@mlenkeit

Thanks you for the follow up. Issue closure is fine. 👍🏻

MikeMcC399 avatar Sep 29 '22 09:09 MikeMcC399