jaybee84 / ml-in-rd

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

Update 05.model-complexity.md #192

Closed jaybee84 closed 2 years ago

jaybee84 commented 2 years ago

Purpose

addresses #175

Directions for reviewers

Which areas should receive a particularly close look?

model_complexity.md has newer text to define ML related terms as well as provide examples before concepts. Please take a look to see if the draft achieved this. Refer to the linked issue to get more context on the changes.

Is there anything that you want to discuss further?

Are there things missing? Does it need more restructuring?

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 2 years ago

AppVeyor build 1.0.474 for commit 37f576281491a535584a884e6864852857b884d5 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...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.475 for commit 06ebed8d8e9879197da81190cf65718aef1eac35 is now complete.

Found 9 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 content/05.model-complexity.md:7:TODO...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.478 for commit f3d15a7bd61f9c3a54dd0aa4861deea778477f2f is now complete.

Found 14 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 content/05.model-complexity.md:3:TODO content/05.model-complexity.md:4:TODO content/05.model-complexity.md:5:TODO content/05.model...
The rendered manuscript from this build is temporarily available for download at:

jaybee84 commented 2 years ago

I had to merge in draft branch to resolve conflicts. Please take a look at the 06.model-complexity.md file and you can ignore the rest. :)

jaybee84 commented 2 years ago

oh no! I messed up the diff :(

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.497 for commit 94ae802995b7db6eb07d1aed728909f43d30fa9b is now complete.

Found 18 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 content/04. box_1_experimental_design.md:8:dataframe content/04. box_1_experimental_design.md:28:subsampling content/06.model-comp...
The rendered manuscript from this build is temporarily available for download at:

jaybee84 commented 2 years ago

pinging this PR to bump the review request up on everyone's inbox :)

jaybee84 commented 2 years ago

Still need to review content/04. box_1_experimental_design.md

@allaway : the box_1_experimental_design was reviewed and merged in an earlier PR.. so you can skip it if you dont have the time. Thanks for reviewing this one... will make the necessary changes.

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.498 for commit d0e3b4ab4804033a8107d1a6cf61b5da6dc3f904 is now complete.

Found 16 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 content/04. box_1_experimental_design.md:28:subsampling content/06.model-complexity.md:4:TODO content/06.model-complexity.md:13:ac...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.499 for commit 8b31c321df2393f60230afdb2a9e171e6dae47b2 is now complete.

Found 16 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 content/04. box_1_experimental_design.md:28:subsampling content/06.model-complexity.md:4:TODO content/06.model-complexity.md:13:ac...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.500 for commit 8b31c321df2393f60230afdb2a9e171e6dae47b2 is now complete.

Found 16 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 content/04. box_1_experimental_design.md:28:subsampling content/06.model-complexity.md:4:TODO content/06.model-complexity.md:13:ac...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.501 for commit e9351bc5ecf84ded5e0a7d10b3692b9bf8264971 is now complete.

Found 16 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 content/04. box_1_experimental_design.md:28:subsampling content/06.model-complexity.md:4:TODO content/06.model-complexity.md:13:ac...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot commented 2 years ago

AppVeyor build 1.0.502 for commit e9351bc5ecf84ded5e0a7d10b3692b9bf8264971 is now complete.

Found 16 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 content/04. box_1_experimental_design.md:28:subsampling content/06.model-complexity.md:4:TODO content/06.model-complexity.md:13:ac...
The rendered manuscript from this build is temporarily available for download at: