search

corona-warn-app / cwa-documentation

Project overview, general documentation, and white papers. The CWA development ends on May 31, 2023. You still can warn other users until April 30, 2023. More information:
https://coronawarn.app/en/faq/#ramp_down
Apache License 2.0
3.28k stars 344 forks source link

Add Transmission Risk Level examples to Risk Calculation in Solution Architecture #832

Closed MikeMcC399 closed 2 years ago

MikeMcC399 commented 2 years ago

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 commented 2 years ago
AnonymousUserUse commented 2 years ago

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.

MikeMcC399 commented 2 years ago

@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.

mlenkeit commented 2 years ago

@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.

MikeMcC399 commented 2 years ago

@mlenkeit

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