Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update About.mdx #766

Merged
merged 3 commits into from
Jul 28, 2020
Merged

Conversation

samanthahamilton
Copy link
Contributor

Issue #687

Related issues and PRs

-n/a

Description

-Improved styling to optimize for reading (like a blog)
-Improve sentence structure and paragraphs

Impacted Areas in the application

-/about page

Testing

-n/a

Additional Notes

-.docx file attached with comments for copyedit changes
-Can continue with other pages after review

neherlabCOVID19_Review_SH_26Jul2020.docx

@vercel
Copy link

vercel bot commented Jul 26, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/neherlab/covid19-scenarios/36w7c56cd
✅ Preview: https://covid19-scenarios-git-fork-samanthahamilton-improve-styling.neherlab.vercel.app

@ivan-aksamentov
Copy link
Member

@samanthahamilton Hi Samantha, thanks. It looks great!
I think there's just a little bug in a comment below. Could you please take a look?

- critically ill individuals either return to regular hospital or die. Again, this depends on the age and on whether they receive intensive care or not.

* Susceptible individuals are exposed/infected through contact with infectious individuals. Each infectious individual causes on average $R_0$ secondary infections while they are infectious.
- Transmissibility of the virus could have seasonal varia
Copy link
Member

@ivan-aksamentov ivan-aksamentov Jul 27, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Transmissibility of the virus could have seasonal varia

This sentence seems to be cut short, compared to the old one

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, my apologies. I have made changes to fix it; hopefully, it's resolved!
Cheers.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@samanthahamilton Thanks!
It looks good to me. I will just let Richard @rneher to review and merge if he's okay with the changes.

Addressing bug re. short sentence
Copy link
Member

@rneher rneher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much. Really useful points. I commented on a few points below, which really are all problems with our initial text, not your edits. I can either fix them after we merge your contributions, or we incorporate them now. Whichever you prefer.


* Susceptible individuals are exposed/infected through contact with infectious individuals. Each infectious individual causes on average $R_0$ secondary infections while they are infectious.
- Transmissibility of the virus could have seasonal variation which is parameterized with the parameter "seasonal forcing" (amplitude) and "peak month" (month of most efficient transmission).
* Exposed individuals progress to a symptomatic/infectious state after an average latency. This progression happens in three stages to ensure the distribution of times spent in the exposed compartment is more realistic than a simple exponential model.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"...average latency." -> "...latency period."

@@ -73,10 +65,11 @@ Many respiratory viruses such as influenza, common cold viruses (including other
## Transmission reduction

The tool allows one to explore temporal variation in the reduction of transmission by infection control measures.
This is implemented as a curve through time that can be dragged by the mouse to modify the assumed transmission. The curve is read out and used to change the transmission relative to the base line parameters for $R_0$ and seasonality.
This is implemented as a curve through time that can be dragged by the mouse to modify the assumed transmission. The curve is read out and used to change the transmission relative to the base line parameters for $R_0$ and seasonality.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohhh, this is outdated.
"Control measures can be specified as time intervals with start and end date during which transmission is reduced by a certain amount."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is a better alternative to remove line 76 (68 in the edit) entirely, and replace it with "Control measures can be specified as time intervals with start and end date during which transmission is reduced by a certain amount."?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes! that is what I meant.

The parameters of this model fall into three categories: a time dependent infection rate $\beta(t)$ time scales of transition to a different subpopulation $t_l$, $t_i$, $t_h$, $t_c$, and age specific parameters $m_a$, $c_a$ and $f_a$ that determine relative rates of different outcomes.
The latency time from infection to infectiousness is $t_l$, the time an individual is infectious after which he/she either recovers or falls severely ill is $t_i$, the time a sick person recovers or deteriorates into a critical state is $t_h$, and the time a person remains critical before dying or stabilizing is $t_c$.
The fraction of infectious that are asymptomatic or mild is $m_a$, the fraction of severe cases that turn critical is $c_a$, and the fraction of critical cases that are fatal is $f_a$.
The parameters of this model fall into three categories: (1) A time dependent infection rate $\beta(t)$ time scales of transition to a different subpopulation $t_l$, $t_i$, $t_h$, $t_c$, and age specific parameters $m_a$, $c_a$ and $f_a$ that determine relative rates of different outcomes;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there needs to be a comma after "$\beta(t)$"

The latency time from infection to infectiousness is $t_l$, the time an individual is infectious after which he/she either recovers or falls severely ill is $t_i$, the time a sick person recovers or deteriorates into a critical state is $t_h$, and the time a person remains critical before dying or stabilizing is $t_c$.
The fraction of infectious that are asymptomatic or mild is $m_a$, the fraction of severe cases that turn critical is $c_a$, and the fraction of critical cases that are fatal is $f_a$.
The parameters of this model fall into three categories: (1) A time dependent infection rate $\beta(t)$ time scales of transition to a different subpopulation $t_l$, $t_i$, $t_h$, $t_c$, and age specific parameters $m_a$, $c_a$ and $f_a$ that determine relative rates of different outcomes;
(2) The latency time from infection to infectiousness is $t_l$, the time an individual is infectious after which he/she either recovers or falls severely ill is $t_i$, the time a sick person recovers or deteriorates into a critical state is $t_h$, and the time a person remains critical before dying or stabilizing is $t_c$;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't really a 1,2,3... what is now under (1) is an overview, the two following points give detail. Pretty confusing wording on our part...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe I understand this section more fully now. How about this edit...?

"The parameters of this model fall into three categories, which determine the relative rates of different outcomes: (1) A time-dependent infection rate [B(t)]; (2) Time scales of transition to a different subpopulation [tl], [ti], [th], [tc]; and (3) Age-specific parameters [ma]. [ca], and [fa]. Here the latency time from infection to infectiousness is presented as [tl], the time an individual is infectious after which he/she either recovers or falls severely ill is [ti], the time a sick person recovers or deteriorates into a critical state is [th], and the time a person remains critical before dying or stabilizing is t[c]. While the fraction of infectious individuals that are asymptomatic or mild is [ma], the fraction of severe cases that turn critical is [ca], and the fraction of critical cases that are fatal is [fa]."

Hopefully, this more clearly explains the parameter categories and the variables within them?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, I like this!

Edits for clarification on lines 29, 68, and 105-107
@codeclimate
Copy link

codeclimate bot commented Jul 27, 2020

Code Climate has analyzed commit 280b907 and detected 0 issues on this pull request.

View more on Code Climate.

@ivan-aksamentov
Copy link
Member

@samanthahamilton Merging this. Please keep up the great work!

@ivan-aksamentov ivan-aksamentov merged commit 50d0154 into neherlab:master Jul 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants