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

Updates learner ranker in tutorial notebooks to use repeated kfold CV #69

Merged
merged 32 commits into from
Sep 22, 2020

Conversation

jason-bentley
Copy link
Contributor

PR

This PR modified the CV approach in each of the three main notebooks, also saves the cell output and removes the pre-commit hook for removing Jupyter output.
This PR is sequential to #66 and so commits from Sep 21 are the difference.

Run
To run please navigate to the sphinx folder and use python make.py html and ensure the documentation builds as normal.

Verified

This commit was signed with the committer’s verified signature.
ctavan Christoph Tavan

Verified

This commit was signed with the committer’s verified signature.
ctavan Christoph Tavan

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@jason-bentley jason-bentley added documentation Improvements or additions to documentation sequential Sequential pull requests - please merge in order that it was put in if not specified otherwise labels Sep 21, 2020
@jason-bentley jason-bentley self-assigned this Sep 21, 2020
@jason-bentley jason-bentley changed the title Docs/kfold revisions Updates learner ranker in tutorial notebooks to use repeated kfold CV Sep 22, 2020
@jason-bentley jason-bentley merged commit 2547bae into develop Sep 22, 2020
@j-ittner j-ittner deleted the docs/kfold_revisions branch September 23, 2020 16:42
@j-ittner j-ittner added this to the 1.0.1 milestone Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation sequential Sequential pull requests - please merge in order that it was put in if not specified otherwise
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants