Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Outdated risk calculation diagram #586

Closed
MikeMcC399 opened this issue Apr 16, 2021 · 6 comments
Closed

Outdated risk calculation diagram #586

MikeMcC399 opened this issue Apr 16, 2021 · 6 comments
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation mirrored-to-jira This item is also tracked internally in JIRA

Comments

@MikeMcC399
Copy link
Contributor

MikeMcC399 commented Apr 16, 2021

Where to find the issue

https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf

Describe the issue

According to https://www.coronawarn.app/en/blog/2021-04-16-corona-warn-app-risk-calculation-further-improved/ dated April 16, 2021 the risk calculation has been changed:

"For app users, this means: Previously, an encounter between two people had to last at least 13 minutes in order to be considered for an encounter with an increased risk (red tile). Now, the app considers encounters of at least 9 minutes. Thus, encounters that represent an actual relevant contact with COVID-19 infected persons can be detected even more accurately and represented accordingly in the app as a warning about an encounter with increased risk"

https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf dated 17.03.2021 is therefore now out of date.

Suggested change

  1. Change the 13 minute limit to 9 minute limit in https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf.

  2. Also monitor the implementation of the documentation update from Out of date Risk Assessment document #453.

  3. Revise the FAQ entry https://www.coronawarn.app/en/faq/#encounter_19_calc by linking to any updated and correct documentation and diagrams, should they become available.


Internal Tracking ID: EXPOSUREAPP-6547

@MikeMcC399 MikeMcC399 added bug Something isn't working documentation Improvements or additions to documentation labels Apr 16, 2021
@dsarkar dsarkar added the mirrored-to-jira This item is also tracked internally in JIRA label Apr 18, 2021
@dsarkar
Copy link
Member

dsarkar commented Apr 18, 2021

@MikeMcC399 Thanks. Internal Tracking ID: EXPOSUREAPP-6547


Corona-Warn-App Open Source Team

@dsarkar
Copy link
Member

dsarkar commented Apr 18, 2021

image

@heinezen
Copy link
Member

corona-warn-app/cwa-website#1222 removes the links from the FAQ. I've added a reminder in the initial post to add them back once the PDFs are updated.

@MikeMcC399
Copy link
Contributor Author

@heinezen

corona-warn-app/cwa-website#1222 removes the links from the FAQ. I've added a reminder in the initial post to add them back once the PDFs are updated.

That seems a reasonable step. Most users will not be concerned with the exact details of the risk calculation. Those few users who do want all the details should not be given out-of-date information, if this can be avoided.

@MikeMcC399
Copy link
Contributor Author

The PDF diagram images/risk_calculation/risk_calculation_enf_v2_overview.pdf carries the date/timestamp 17.03.2021, 14:25.

The diagram images/solution_architecture/risk_calculation.svg carries a date/timestamp 23.08.2021, 10:30.

The later diagram has been partially updated to use 9 minutes instead of 13 minutes.

image

On the left-hand side it still shows a 13 minute threshold though:

image

The diagrams in svg and PDF format should both be corrected and re-published.

mlenkeit added a commit that referenced this issue Jan 25, 2022
consistently use 5 and 9 minutes as thresholds throughout the diagram.
additionally, re-generates the pdf for the svg

related to #586, #453, #818
@mlenkeit mlenkeit assigned mlenkeit and unassigned maugst Jan 25, 2022
@MikeMcC399
Copy link
Contributor Author

The contents of both the following files has been corrected and they now carry a timestamp 25.01.2022, 18:00:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working documentation Improvements or additions to documentation mirrored-to-jira This item is also tracked internally in JIRA
Projects
None yet
Development

No branches or pull requests

5 participants