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

Vaccines outline #828

Merged
merged 1 commit into from
Feb 2, 2021
Merged

Vaccines outline #828

merged 1 commit into from
Feb 2, 2021

Conversation

rando2
Copy link
Contributor

@rando2 rando2 commented Feb 2, 2021

Description of the proposed additions or changes

This is a suggestion for the sections of info that we probably want to add about the vaccines (as well as some possible tweaks to the vaccine figure).
However, I'm starting to think we might just want to split this section out and submit the therapeutics paper separately, since that one is basically ready to go and this needs some additions still -- plus we are already like 8000 words over the word limit 😓

Related issues

#738

Suggested reviewers (optional)

@ajlee21 @nilswellhausen I would love to get your feedback since you have both been very involved with what's been done!
@likhithakolla and @cgreene as well, since you had written a lot of the mRNA vaccine section
And @RLordan you have been giving very helpful feedback on #740 and #741 as well, so just in case you have any thoughts!

Checklist

  • Text is formatted so that each sentence is on its own line.
  • Pre-prints cited in this pull request have a GitHub issue opened so that they can be reviewed.

@rando2 rando2 changed the title outline Vaccines outline Feb 2, 2021
@AppVeyorBot
Copy link

AppVeyor build 1.0.3052 for commit ad12a0a is now complete.

Found 8 potential spelling error(s). Preview:content/20.pharmaceuticals.md:818:AstraZeneca
content/20.pharmaceuticals.md:822:Janssen’s
content/20.pharmaceuticals.md:828:CoronaVac
content/20.pharmaceuticals.md:828:SinoVac's
content/20.pharmaceuticals.md:831:Sinopharm
content/20.pharmaceuticals.md:837:Novavax
content/20.pharmaceuticals.md:837:NVX
content/21.nutraceuticals.md:6:repurposed...
The rendered manuscript from this build is temporarily available for download at:

@RLordan
Copy link
Collaborator

RLordan commented Feb 2, 2021

Hey @rando2, my thoughts would be that it probably makes sense to split the vaccine section up into its own manuscript if the therapeutics paper is miles over the word limit. Ultimately, @ajlee21, @nilswellhausen etc. have contributed the most so go with their thoughts.

If you do split, there is plenty updates on each of those vaccines to include. I think maybe the discussion should also briefly address the current state of vaccine distribution e.g. Moderna & Pfizer leading the way in most countries etc. mentioning the logistical issues of transport and distribution currently exposed. That would round the outline off nicely. I can help out towards the end of the week.

@nilswellhausen
Copy link
Contributor

I agree that it probably makes sense to split vaccines since that topic has enough volume to be its own manuscript. That way we can focus on updating the vaccine section separately and go into the data more in-depth rather than trying to squeeze this in here.

@ajlee21
Copy link
Contributor

ajlee21 commented Feb 2, 2021

Yes I very much agree with the above comments. Will start trying to take a look at the vaccine sections later this week

@rando2
Copy link
Contributor Author

rando2 commented Feb 2, 2021

Thank you so much everyone! If someone would be able to approve #740, #741, and #828, I can merge them in (we can always change the content later) and then open a new PR to split the vaccines info out! I agree this will work out for the best!

@RLordan
Copy link
Collaborator

RLordan commented Feb 2, 2021

Thank you so much everyone! If someone would be able to approve #740, #741, and #828, I can merge them in (we can always change the content later) and then open a new PR to split the vaccines info out! I agree this will work out for the best!

Done!

@rando2 rando2 merged commit 61b73a3 into greenelab:master Feb 2, 2021
@rando2 rando2 deleted the vaccine-outline branch February 2, 2021 14:03
@rando2
Copy link
Contributor Author

rando2 commented Feb 2, 2021

Thank you so much @RLordan!

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.

5 participants