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

ADD logistic regression with crossvalidation #28

Closed
wants to merge 57 commits into from

Conversation

KEggensperger
Copy link
Contributor

@KEggensperger KEggensperger commented Feb 24, 2017

This PR adds a logistic regression benchmark using crossvalidation that allows to evaluate one fold at a time.

Furthermore it:

  • Fixes a typo leading to a crash when using trying to load data for logistic regression
  • Introduces "rng" as a further argument to objective_function/objective_function_test (related to de-randomize benchmarks #2)
  • Fixes some PEP-8 issues

NOTE: This PR can only be merged after add-auto-sklearn is merged. As it is it fails unittests because of scikit-learn version conflicts

@KEggensperger KEggensperger requested review from mfeurer and aaronkl and removed request for mfeurer and aaronkl February 24, 2017 12:18
@KEggensperger KEggensperger changed the base branch from master to development February 24, 2017 12:28
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.

None yet

3 participants