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

Make Pipeliner use any grid object with sklearn structure #19

Open
lodurality opened this issue Apr 24, 2017 · 0 comments
Open

Make Pipeliner use any grid object with sklearn structure #19

lodurality opened this issue Apr 24, 2017 · 0 comments
Assignees

Comments

@lodurality
Copy link
Member

lodurality commented Apr 24, 2017

As of now, Reskit has a hard coded GridSearchCV as a default grid object. We need to do the following:

  1. Add parameter grid_object which specifies grid object. Right now in the code, we have grid_clf.

  2. Pipeliner should work for testing single models without grid search (grid_object = None). If grid_object = None then Pipeliner doesn't use param_grid and makes cross_val_score on pipelines with params specified in steps.

  3. If we specify grid object then we should have grid_object_params (default None) for non-classifier specific grid search parameters. For example we can specify in grid_object_params n_iters for RandomizedSearchCV. Setting of all grid_object parameters should look something like this grid_object.set_params(**self.grid_objects_params, param_grid=self.param_grid)

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

No branches or pull requests

2 participants