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

fast eli5.sklearn.permutation_importance? #336

Open
omarcr opened this issue Sep 1, 2019 · 4 comments
Open

fast eli5.sklearn.permutation_importance? #336

omarcr opened this issue Sep 1, 2019 · 4 comments

Comments

@omarcr
Copy link

omarcr commented Sep 1, 2019

Is there a way to make:

perm = PermutationImportance(estimator, cv='prefit', n_iter=1).fit(X_window_test, Y_test)
fast?

currently I am running an experiment with 3,179 features and the algorithm is too slow (even with cv=prefit) is there a way to make it faster?

@lkugler
Copy link

lkugler commented Sep 4, 2019

@joelrich started an issue (#317) like that but it seemingly received no feedback. I would also vote for a parallel implementation.
How would we implement it to run in parallel? joblib.Parallel?

@jnothman
Copy link
Contributor

jnothman commented Sep 4, 2019 via email

@omarcr
Copy link
Author

omarcr commented Sep 11, 2019

I think @jnothman reference is the best that we currently have. Does anyone know if this will be ported to Eli? thanks,

@folterj
Copy link

folterj commented Dec 1, 2021

It seems even for relatively small training sets, model (e.g. DecisionTreeClassifier, RandomForestClassifier) training is fast, but using permutation_importance on the trained models is incredibly slow. (Currently using model.feature_importances_ as alternative)

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

4 participants