jaybee84 / ml-in-rd

Manuscript for perspective on machine learning in rare disease

Other

2 stars 1 forks source link

Round Robin #210

Closed allaway closed 2 years ago

allaway commented 2 years ago

Purpose

205

Directions for reviewers

Which areas should receive a particularly close look?

I made some fairly extensive changes to wording in a couple of sections, please make sure I didn't change the intent.

Is there anything that you want to discuss further?

The vignette about laser excision shows up twice (outlook, combining-datasets). We should remove it from one of the two sections, but not sure which.

Is the pull request ready for review?

Yes

Manuscript checklist

Unless otherwise noted above, this PR will be considered ready for review when all four items have been checked.

[x] All changes to text follow "one sentence per line" [Manubot instructions]
[x] All citations follow the Manubot citation instructions
[ ] All changes or additions to tables follow the Manubot table instructions
[ ] All figures follow the Manubot figure instructions
[ ] All terms that need to be added to glossary have been added to #50
[ ] There are no new spelling errors identified by the Manubot spellchecker

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.576 for commit 39396cbf572ccedf93ec9f28eff85dc37b52fca5 is now complete.

Found 10 potential spelling error(s). Preview:

content/02.intro.md:13:TODO content/04. box_1_experimental_design.md:36:modeller content/06.model-complexity.md:26:recalibrated content/06.model-complexity.md:35:Kullback content/06.model-complexity.md:35:Leibler content/06.model-complexity.md:40:WIP content/08.multiple-approaches-required.md:24:WIP content/09.outlook.md:17:TODO content/09.outlook.md:22:biobanking content/09.outlook.md:22:intiatives...

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

manuscript-1.0.576-39396cb.html

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.577 for commit b07eec11359db2080a167c780df1488bb531f767 is now complete.

Found 4 potential spelling error(s). Preview:

content/02.intro.md:13:TODO content/06.model-complexity.md:40:WIP content/08.multiple-approaches-required.md:24:WIP content/09.outlook.md:17:TODO...

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

manuscript-1.0.577-b07eec1.html

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.578 for commit 6c44130928eadec13889dc762aed54c1a3fe384c is now complete.

Found 4 potential spelling error(s). Preview:

content/02.intro.md:13:TODO content/06.model-complexity.md:40:WIP content/08.multiple-approaches-required.md:24:WIP content/09.outlook.md:17:TODO...

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

manuscript-1.0.578-6c44130.html

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.579 for commit ae9fb8628afcd69aaad6c46a319badb2975d7e7d is now complete.

Found 4 potential spelling error(s). Preview:

content/02.intro.md:13:TODO content/06.model-complexity.md:40:WIP content/08.multiple-approaches-required.md:24:WIP content/09.outlook.md:17:TODO...

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

manuscript-1.0.579-ae9fb86.html