Skip to content

add logic to speed up permutation feature importance#4037

Merged
ozzieD merged 12 commits into
mainfrom
4036-dfs-trans-fast-perm-FI
Mar 2, 2023
Merged

add logic to speed up permutation feature importance#4037
ozzieD merged 12 commits into
mainfrom
4036-dfs-trans-fast-perm-FI

Conversation

@ozzieD

@ozzieD ozzieD commented Feb 27, 2023

Copy link
Copy Markdown
Contributor

@CLAassistant

CLAassistant commented Feb 27, 2023

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@codecov

codecov Bot commented Feb 27, 2023

Copy link
Copy Markdown

Codecov Report

Merging #4037 (62b1bf1) into main (a322411) will increase coverage by 31.0%.
The diff coverage is 100.0%.

@@           Coverage Diff            @@
##            main   #4037      +/-   ##
========================================
+ Coverage   68.8%   99.7%   +31.0%     
========================================
  Files        349     349              
  Lines      37489   37523      +34     
========================================
+ Hits       25763   37405   +11642     
+ Misses     11726     118   -11608     
Impacted Files Coverage Δ
...ponents/transformers/preprocessing/featuretools.py 100.0% <100.0%> (+1.5%) ⬆️
evalml/pipelines/pipeline_base.py 98.5% <100.0%> (+0.1%) ⬆️
evalml/tests/component_tests/test_featuretools.py 100.0% <100.0%> (+100.0%) ⬆️
...understanding_tests/test_permutation_importance.py 100.0% <100.0%> (ø)
evalml/automl/automl_search.py 99.6% <0.0%> (+0.2%) ⬆️
...elines/components/transformers/column_selectors.py 92.9% <0.0%> (+1.5%) ⬆️
evalml/utils/gen_utils.py 99.3% <0.0%> (+1.5%) ⬆️
evalml/objectives/standard_metrics.py 100.0% <0.0%> (+1.7%) ⬆️
...onents/estimators/regressors/catboost_regressor.py 100.0% <0.0%> (+2.3%) ⬆️
... and 177 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ozzieD ozzieD changed the title Draft: add logic to speed up permutation feature importance add logic to speed up permutation feature importance Feb 28, 2023
Comment thread evalml/pipelines/pipeline_base.py Outdated

@jeremyliweishih jeremyliweishih left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just requesting one additional coverage but looks great otherwise!

@jeremyliweishih jeremyliweishih left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM once tests pass! great work 🦾

@tamargrey tamargrey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm! Nice work!

@ozzieD ozzieD merged commit df29850 into main Mar 2, 2023
@ozzieD ozzieD deleted the 4036-dfs-trans-fast-perm-FI branch March 2, 2023 21:39
@chukarsten chukarsten mentioned this pull request Mar 15, 2023
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.

_fast_permutation_importance doesn't run if DFS Transformer present, even when dfs feature pre-exist.

4 participants