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

Fixing sklearn error when using RandomizedSearchCV with Interpret's LogisticRegression #315

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

luisffranca
Copy link
Contributor

This PR fixes #301

No get_params method was implemented in interpret.glassbox.linear.LogisticRegression which caused RandomizedSearchCV to fail.

With this fix, get_params and set_params are implemented and Interpret's LinearRegression and LogisticRegression can be used with RandomizedSearchCV and other model selection options.

Signed-off-by: Luis França <luis.franca@microsoft.com>
@luisffranca luisffranca force-pushed the lamorinfranc/fix-sklearn-compatibility-issue branch from 48c52b5 to b7a99af Compare January 14, 2022 20:22
@paulbkoch paulbkoch force-pushed the develop branch 4 times, most recently from 989a6a0 to e6b74cd Compare January 28, 2024 05:57
@paulbkoch paulbkoch force-pushed the develop branch 5 times, most recently from 94cc351 to 385a66b Compare March 31, 2024 02:30
@paulbkoch paulbkoch force-pushed the develop branch 3 times, most recently from a06b85c to 51be653 Compare August 1, 2024 09:36
@paulbkoch paulbkoch force-pushed the develop branch 3 times, most recently from f2f2a4c to d7d5fd2 Compare September 1, 2024 12:36
@paulbkoch paulbkoch force-pushed the develop branch 2 times, most recently from 4401ae3 to 05e3b26 Compare September 26, 2024 21:15
@paulbkoch paulbkoch force-pushed the develop branch 6 times, most recently from f8501a4 to e0369a7 Compare December 10, 2024 06:25
@paulbkoch paulbkoch force-pushed the develop branch 3 times, most recently from 43e3711 to bc20d4b Compare December 24, 2024 01:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Logistic Regression and Randomized Search CV
1 participant