jaybee84 / ml-in-rd

Manuscript for perspective on machine learning in rare disease
Other
2 stars 1 forks source link

Rewrite "composite approaches" section #139

Closed jaclyn-taroni closed 3 years ago

jaclyn-taroni commented 3 years ago

Purpose

Addresses #128. This attempts to tie how the challenges presented throughout the manuscript are addressed by the approaches highlighted in this section (DeepProfile and MultiPLIER).

Directions for reviewers

I'll note that, ideally, before this goes in someone takes another pass (or we plan another pass prior to submission), as I didn't closely read this after writing it and spending time away like I like to. But to address the first bulletpoint below, someone besides me needs to look!

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.

AppVeyorBot commented 3 years ago

AppVeyor build 1.0.311 for commit 76f2fed25dde94820b6aadf7071133d02808b908 is now complete.

Found 6 potential spelling error(s). Preview:content/05.prior-knowledge.md:25:TODO content/06.multiple-approaches-required.md:13:TODO content/06.multiple-approaches-required.md:14:VAE's content/06.multiple-approaches-required.md:16:TODO content/06.multiple-approaches-required.md:17:histopathology content/07.definitions.md:4:TODO...
The rendered manuscript from this build is temporarily available for download at:

jaclyn-taroni commented 3 years ago

I will file an issue for linking between sections and then use the URL to tie to a specific version of DeepProfile.

AppVeyorBot commented 3 years ago

AppVeyor build 1.0.312 for commit 6153c3ece25d05b6df51d43b721cc28fe45ebbd7 is now complete.

Found 6 potential spelling error(s). Preview:content/05.prior-knowledge.md:25:TODO content/06.multiple-approaches-required.md:13:TODO content/06.multiple-approaches-required.md:14:VAE's content/06.multiple-approaches-required.md:16:TODO content/06.multiple-approaches-required.md:17:histopathology content/07.definitions.md:4:TODO...
The rendered manuscript from this build is temporarily available for download at:

jaclyn-taroni commented 3 years ago

@jaybee84 I know you approved, but I'd like you to take another look before this goes in! I tried to address your comments without changing sentences to be in passive voice where possible.

AppVeyorBot commented 3 years ago

AppVeyor build 1.0.313 for commit 4f7949c2cdb7b6b35286b6ef54212f96f6e84858 is now complete.

Found 6 potential spelling error(s). Preview:content/05.prior-knowledge.md:25:TODO content/06.multiple-approaches-required.md:12:TODO content/06.multiple-approaches-required.md:13:VAE's content/06.multiple-approaches-required.md:15:TODO content/06.multiple-approaches-required.md:16:histopathology content/07.definitions.md:4:TODO...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot commented 3 years ago

AppVeyor build 1.0.314 for commit 788c48564d95e7d64dc1c72ce1e3440b6f184230 is now complete.

Found 6 potential spelling error(s). Preview:content/05.prior-knowledge.md:25:TODO content/06.multiple-approaches-required.md:12:TODO content/06.multiple-approaches-required.md:13:VAE's content/06.multiple-approaches-required.md:15:TODO content/06.multiple-approaches-required.md:16:histopathology content/07.definitions.md:4:TODO...
The rendered manuscript from this build is temporarily available for download at: