VowpalWabbit / vowpal_wabbit / 7733 / 1
76%
master: 76%

DEFAULT BRANCH: master
Ran
Files 255
Run time 35s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 75.635%. Remained the same
7733.1

push

travis-ci

web-flow 
[FIX] Make Python Examples Up To Date With Scikit-Learn Version 0.22.1 (#2301)

* [fix] Add dist folder to .gitignore

- This doesn't let a developer mistakenly commit newly build files to the repository

* [fix] Make sklearn_vs example up to date with sklearn v0.22.1

- Update outdated help link in cell 2
- As per v0.22.1, change `from sklearn.grid_search import RandomizedSearchCV` to `from sklearn.model_selection import RandomizedSearchCV`
- Add Pandas to handle cv_results_ and update other data get operations accordingly.
- Use itertuples since they are faster compared to iterrows in pandas DataFrame.

- https://www.dataschool.io/how-to-update-your-scikit-learn-code-for-2018/#gridsearchandrandomizedsearchhavechangedhowtheyreportresults
- https://stackoverflow.com/a/41022840
- Github issue #2281

* [dbg] Suppress FutureWarning for import from sklearn.linear_model.base

- Importing From sklearn.linear_model.base has been depricated in v0.22 and will be removed in v0.24
- We need to import from the same since LinearClassifierMixin and SparseCoefMixin haven't been added to linear_model module yet
- Warning suppression is important since it hampers the aesthetics of the output and confuses beginners

- https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/linear_model/__init__.py
- https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/linear_model/_base.py

* [fix] Remove outputs from sklearn_vw

* [fix] Ignore models generated in python/examples in git history

- Prevents developers from accidently commiting the saved models from examples

* allow backward compatibility for importing RandomizedSearch CV

* improve comment explaining reason to ignore future warning

Co-authored-by: thealphadollar <thealphadollar>
Co-authored-by: Jack Gerrits <jackgerrits@users.noreply.github.com>

16083 of 21264 relevant lines covered (75.63%)

593782.34 hits per line

Source Files on job 7733.1