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

Pathogenesis - Some responses to reviewer 2 #883

Merged
merged 14 commits into from
Mar 23, 2021
Merged

Pathogenesis - Some responses to reviewer 2 #883

merged 14 commits into from
Mar 23, 2021

Conversation

RLordan
Copy link
Collaborator

@RLordan RLordan commented Mar 1, 2021

Hey @rando2 - I added a bit on the variants. Also ticked off a few other small requests for reviewer 2. There are still some remaining as highlighted below.

Complete - Reviewer 2 points: 1,2,4,6,10, and 12

Points remaining - 3,5,7,8,9,11

Description of the proposed additions or changes

Related issues

Suggested reviewers (optional)

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.

Hey @rando2 - I added a bit on the variants. Also ticked off a few other small requests for reviewer 2. There are still some remaining as highlighted below.

Complete - Reviewer 2 points: 1,2,4,6,10, and 12

Points remaining - 3,5,7,8,9,11
@RLordan RLordan requested a review from rando2 March 1, 2021 22:18
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
@AppVeyorBot
Copy link

AppVeyor build 1.0.3429 The rendered manuscript from this build is temporarily available for download at:

@agitter
Copy link
Collaborator

agitter commented Mar 7, 2021

I created a checklist in #892 that we can use to track which requests have been resolved by this pull request once it is merged.

@RLordan
Copy link
Collaborator Author

RLordan commented Mar 8, 2021

I created a checklist in #892 that we can use to track which requests have been resolved by this pull request once it is merged.

Great idea, thanks @agitter

@rando2
Copy link
Contributor

rando2 commented Mar 15, 2021

I'm so sorry for the slow response on this! I am at least partly back from the other project now and am hoping to get to this PR today (currently working on some of the vaccine PRs so I can help keep that moving forward!) Thank you both for taking the lead!

Copy link
Contributor

@rando2 rando2 left a comment

Choose a reason for hiding this comment

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

This reviewer is so helpful! I think these are great responses @RLordan and sorry I've been slow to come back to helping with this. I had one thing that I read a little differently than you did but I can definitely understand your reading as well, so not sure which they meant!

content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
content/07.pathogenesis.md Outdated Show resolved Hide resolved
RLordan and others added 10 commits March 22, 2021 19:02
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Co-authored-by: HM Rando <halie.rando@cuanschutz.edu>
Copy link
Contributor

@rando2 rando2 left a comment

Choose a reason for hiding this comment

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

This is just to resolve the merge conflict with #885 (I don't want to create more mess since Ronan and I are editing this file at the same time)

content/07.pathogenesis.md Show resolved Hide resolved
@RLordan
Copy link
Collaborator Author

RLordan commented Mar 22, 2021

This reviewer is so helpful! I think these are great responses @RLordan and sorry I've been slow to come back to helping with this. I had one thing that I read a little differently than you did but I can definitely understand your reading as well, so not sure which they meant!

Hey @rando2 , no problem, I think you were right on that line. Let me know if you want me to approve or commit anything else here.

In terms of priority at the moment, Do you want me to focus on getting vaccines or evolution (update on variants) updated first.

I also mentioned Amruta might be able to help with the vaccines manuscript. I met her today and she said she can give a brief synopsis on India's candidates. She just came back from India so she has all the vaccine gossip.

Co-authored-by: Anthony Gitter <agitter@users.noreply.github.com>
@AppVeyorBot
Copy link

AppVeyor build 1.0.3742 for commit a1050f1 is now complete.

Found 1 potential spelling error(s). Preview:content/22.vaccines.md:361:IgGs...
The rendered manuscript from this build is temporarily available for download at:

@rando2
Copy link
Contributor

rando2 commented Mar 23, 2021

Hey @RLordan, that sounds awesome! From my own personal perspective (my classroom time is about to get much, much heavier as of April 5, and then I will probably have some other limitations come up right as the class ends), I think I'd prefer to focus on getting all of the preprints up and then tackle the gentler reviews, since that requires less creative energy (for me at least!)

I have a student coming on to help address Review 1.3-- he is going to read those 4 papers and then write up a synopsis (I'll tag him when he sets up his GitHub username). It would be great to get Amruta involved ASAP since I think vaccines is so close! I'm planning to sit down and engage with the pharma reviews today and then take a critical look at that one and vaccines this week.

@rando2 rando2 merged commit 1926d27 into master Mar 23, 2021
@RLordan
Copy link
Collaborator Author

RLordan commented Mar 23, 2021

Hey @RLordan, that sounds awesome! From my own personal perspective (my classroom time is about to get much, much heavier as of April 5, and then I will probably have some other limitations come up right as the class ends), I think I'd prefer to focus on getting all of the preprints up and then tackle the gentler reviews, since that requires less creative energy (for me at least!)

I have a student coming on to help address Review 1.3-- he is going to read those 4 papers and then write up a synopsis (I'll tag him when he sets up his GitHub username). It would be great to get Amruta involved ASAP since I think vaccines is so close! I'm planning to sit down and engage with the pharma reviews today and then take a critical look at that one and vaccines this week.

Gotcha! look forward to welcoming your student. I will try get Amruta involved ASAP. I will try fill in the blanks in the vaccine paper during the week.

agitter added a commit that referenced this pull request Mar 30, 2021
rando2 pushed a commit to rando2/covid19-review that referenced this pull request Aug 3, 2021
rando2 pushed a commit to rando2/covid19-review that referenced this pull request Aug 4, 2021
rando2 pushed a commit to rando2/covid19-review that referenced this pull request Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants