Closed jaybee84 closed 2 years ago
AppVeyor build 1.0.459 for commit b51a8b96035ec5eed5975bb28e42a96f7764fe60 is now complete. Found 4 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO...
AppVeyor build 1.0.460 for commit c4bcadbeb36c7b74a9eed832172f61503df45044 is now complete. Found 4 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO...
AppVeyor build 1.0.461 for commit 96e1add7e5c25cd4297f0d12930ed0990e61d0d9 is now complete. Found 4 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO...
AppVeyor build 1.0.462 for commit 96e1add7e5c25cd4297f0d12930ed0990e61d0d9 is now complete. Found 4 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO...
AppVeyor build 1.0.465 for commit 8766933d1c90a66558ae3b7e888e6a5b26c01b27 is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
I revised this section to incorporate suggestions from the earlier review. I have tried to tailor this box to a "data-centric" view + "rare-disease specific" view instead of a general primer of ML experiment design.
I have intentionally left out hyperparameter tuning to encourage brevity of this box, but happy to add in if the group thinks it is necessary.
AppVeyor build 1.0.466 for commit ac241f827b83d7db6a79c52ff3561b17d7e19124 is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.467 for commit 186b8d0d6904a4c633e05e88daee896f63ecf70e is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.468 for commit c29b310a97e609f97c9be5d6f2f0edd25423c28d is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.469 for commit acf4fc5b658cdddec78c9adca7b159840b8e81ce is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.470 for commit 29f7f45dba3bdf7320147d369da73b328e14cb39 is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.471 for commit 532d9fc8223499cba608a7009ea96904959477e7 is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.472 for commit 8bdef847d4eceaa7e46ad430e73afb3443386bdf is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.473 for commit 139e19678feadbfe1c073599a400d524c1d159ef is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
AppVeyor build 1.0.476 for commit 6756546c412e951e45c75b0494baeedd829218e6 is now complete. Found 8 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO
content/03.combining-datasets.md:5:TODO
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:8:collinearity
content/03.combining-datasets.md:9:TODO...
Purpose
once approved closes #185
Directions for reviewers
Which areas should receive a particularly close look?
Most of what is included here is common knowledge in ML/computational world. What are some good references to add here?
Is there anything that you want to discuss further?
I have not gone into too much detail about normalization etc. since they will be in text proper. Are there other things to add here that I have missed?
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.