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

Remove "QuantileRegressor" from Supervised.py to avoid system hanging #401

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

IslamAAli
Copy link

Removing "QuantileRegressor" which is the cause of the system to hang at 64% or 74%. Tested on multiple size datasets and it solves the problem

Removing "QuantileRegressor" which is the cause of the system to hang at 64% or 74%. Tested on multiple size datasets and it solves the problem
@KayO-GH
Copy link

KayO-GH commented Jun 26, 2023

This is a simple but very useful PR. @shankarpandala I think you should look into this.
It solves a problem many of us are facing. Personally, I manually remove that regressor before using LazyPredict, otherwise, it stalls right after using the Poisson Regressor.

I used the code below to get it running without this fix, but the fix would certainly help a great deal, unless there's some other consideration I do not know of:

import sklearn
from lazypredict import Supervised

Supervised.removed_regressors.append("QuantileRegressor")
Supervised.REGRESSORS.remove(('QuantileRegressor', sklearn.linear_model._quantile.QuantileRegressor))
LazyRegressor = Supervised.LazyRegressor

Then use the LazyRegressor as usual.

Copy link

@Borda Borda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's rename the PR such it reflects the suggested change...

@IslamAAli IslamAAli changed the title Update Supervised.py Remove "QuantileRegressor" from Supervised.py to avoid system hanging Mar 4, 2024
@IslamAAli
Copy link
Author

Let's rename the PR such it reflects the suggested change...

Changed the title of the MR as suggested.

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.

3 participants