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

Use sklearn RepeatedKFold cross validation routine. #115

Merged
merged 1 commit into from
Sep 10, 2018

Conversation

jasonlaska
Copy link
Member

@mnarayan for your review

@jasonlaska jasonlaska merged commit 6f6a98b into develop Sep 10, 2018
@jasonlaska jasonlaska deleted the feature/remove_custom_k_repeated_cv branch September 10, 2018 23:30
@mnarayan mnarayan self-requested a review September 11, 2018 01:43

### Repeated KFold cross-validation

* ["Cross-validation pitfalls when selecting and assessing regression and classification models"](https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3994246/) D. Krstajic, L. Buturovic, D. Leahy, and S. Thomas, 2014.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like to leave this reference on why use repeatedKFold in. I find that several folks don't know that this is necessary, so it explains our preferred approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants