Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix cross validation evaluation #337

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

Closed
wants to merge 5 commits into from

Conversation

lucasmoura
Copy link
Contributor

@lucasmoura lucasmoura commented Mar 8, 2017

The cross_validation function was not working on learning.py. Running the compare function, which is responsible for comparing multiple algorithms against their cross validation score would generate an error. With that said, this MR does the following:

  • Update cross_validation function to resemble more the book pseudocode
  • Update cross_validation_wrapper to resemble more the book pseudocode
  • Add tests to function train_and_test
  • Update print_table function on utils.py to better handle the cross validation output.
  • Add dummy size variable to some learner implementations, in order to maintain cross_validation similiarity with the book algorithm. This should be updated in the future, in order for the size parameter to be a way to better configure a learner.

@lucasmoura lucasmoura changed the title Fix cross_validation function Fix cross validation for learners Mar 8, 2017
@lucasmoura lucasmoura changed the title Fix cross validation for learners Fix cross validation evaluation Mar 8, 2017
@lucasmoura lucasmoura force-pushed the fix_cross_validation branch 3 times, most recently from 70dcee8 to f42ef9f Compare March 22, 2017 17:05
@lucasmoura
Copy link
Contributor Author

Updating this MR, because #388 already fixes the bug in leave one out cross validation

@lucasmoura
Copy link
Contributor Author

Updating this MR, because #471 already updated the print_table function for python3

@lucasmoura lucasmoura force-pushed the fix_cross_validation branch from 1af3c52 to 60b73a5 Compare April 7, 2017 18:44
@lucasmoura lucasmoura force-pushed the fix_cross_validation branch from 60b73a5 to 685e2a5 Compare April 18, 2017 12:14
@norvig norvig closed this Mar 14, 2018
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