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 50.discussion.md #600

Merged
merged 24 commits into from
Nov 12, 2020
Merged

Update 50.discussion.md #600

merged 24 commits into from
Nov 12, 2020

Conversation

tlukan
Copy link
Collaborator

@tlukan tlukan commented Aug 14, 2020

Description of the proposed additions or changes

Related issues

closes #684

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.

@tlukan tlukan requested a review from rando2 August 14, 2020 14:58
@rando2 rando2 added the in progress text still being developed label Aug 14, 2020
rando2 pushed a commit to rando2/covid19-review that referenced this pull request Oct 23, 2020
@rando2 rando2 mentioned this pull request Oct 23, 2020
2 tasks
@AppVeyorBot
Copy link

AppVeyor build 1.0.2278 for commit 3a97c2a failed.

tlukan and others added 2 commits October 23, 2020 17:56
@AppVeyorBot
Copy link

AppVeyor build 1.0.2281 for commit 5fc0306 is now complete.

Found 11 potential spelling error(s). Preview:content/08.transmission.md:320:tlukan
content/50.discussion.md:89:Adhanom
content/50.discussion.md:89:choropleth
content/50.discussion.md:89:choropleth
content/50.discussion.md:89:choropleth
content/50.discussion.md:89:choropleths
content/50.discussion.md:89:choropleths
content/50.discussion.md:89:Ghebreyesus
content/50.discussion.md:89:inconclusivity
content/50.discussion.md:89:Tedros
content/50.discussion.md:91:Americna...
The rendered manuscript from this build is temporarily available for download at:

@rando2 rando2 added Text for Review Feedback on this text is welcome! and removed in progress text still being developed labels Oct 29, 2020
@AppVeyorBot
Copy link

AppVeyor build 1.0.2291 for commit e1bd854 is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

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.

@tlukan I'm so sorry this took so long! I think you did an awesome job and raised the most important points for this section. I expanded a bit on your ideas to make the style match the sections above a bit more. I'd love to hear what you think!

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.

Add in figure references

@AppVeyorBot
Copy link

AppVeyor build 1.0.2309 for commit b6e8b44 is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.2314 for commit 350d7a9 is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.2388 for commit 581905c is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

Copy link
Collaborator

@RLordan RLordan left a comment

Choose a reason for hiding this comment

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

Just one suggestion, thanks @rando2

HM Rando and others added 2 commits November 11, 2020 08:12
Co-authored-by: Anthony Gitter <agitter@users.noreply.github.com>
Co-authored-by: RLordan <62627112+RLordan@users.noreply.github.com>
@rando2
Copy link
Contributor

rando2 commented Nov 11, 2020

Thank you so much @agitter and @RLordan!

@AppVeyorBot
Copy link

AppVeyor build 1.0.2400 Build execution time has reached the maximum allowed time for your plan (60 minutes).

@agitter
Copy link
Collaborator

agitter commented Nov 11, 2020

The AppVeyor build timed out, but I confirmed that Figure 6 now appears correctly in the GitHub Actions PDF artifact. I only reviewed the figure issues and didn't read the new text carefully.

@AppVeyorBot
Copy link

AppVeyor build 1.0.2432 for commit df38294 is now complete....

Found 0 potential spelling error(s). Preview: The rendered manuscript from this build is temporarily available for download at:

Copy link
Member

@cgreene cgreene left a comment

Choose a reason for hiding this comment

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

I had a few thoughts - mostly there are some quite long paragraphs here that I think should be broken up a bit more. Feel free to merge it when you're ready.

China, the United States, and France are among the countries with the most clinical trial recruiting for trials with single-country enrollment.
Many countries have little to no clinical trial recruiting, with the continents of Africa and South America much less represented than Asia, Europe, and North America.
Trials that recruit across multiple countries do appear to broaden geographic representation, but these trials seem to be heavily dominated by the United States and European Union.
Dual concerns arise from this skewed geographic representation in clinical trial recruitment: first, it suggests that treatments such as remdesivir that are promising but primarily available to clinical trial participants are unlikely to be accessible by people in many countries, and second, it raises the concern that the findings of clinical trials will be based on participants from many of the wealthiest countries, which may lead to ambiguity in whether the findings can be extrapolated to COVID-19 patients elsewhere.
Copy link
Member

Choose a reason for hiding this comment

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

What do you think about breaking up this paragraph somewhere around here.

Co-authored-by: Casey Greene <cgreene@users.noreply.github.com>
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.

Address @cgreene's concerns and add some more paragraph breaks! These did turn out to be gigantic paragraphs.

@rando2
Copy link
Contributor

rando2 commented Nov 12, 2020

@agitter if you get a chance, would you be able to change your review (assuming this looks OK? I committed your suggestions!)

@AppVeyorBot
Copy link

AppVeyor build 1.0.2467 for commit 6372eae is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.2474

Found 0 potential spelling error(s). Preview: for commit 9bf0b3c is now complete....
The rendered manuscript from this build is temporarily available for download at:

Copy link
Collaborator

@agitter agitter left a comment

Choose a reason for hiding this comment

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

The figure didn't show up after it was moved. I pushed a commit. Let's wait to check that preview build before merging.

@AppVeyorBot
Copy link

AppVeyor build 1.0.2483 for commit 68eaec0 is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@agitter
Copy link
Collaborator

agitter commented Nov 12, 2020

The figure appears correctly in this last build.

@rando2
Copy link
Contributor

rando2 commented Nov 12, 2020

Super exciting! Thank you @agitter and congrats @tlukan on your first PR!

@rando2 rando2 merged commit bfe3471 into greenelab:master Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Text for Review Feedback on this text is welcome!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"6.2.4 Access to and Representation in Clinical Trials" is blank
6 participants