Skip to content

Accept features in AutoMLSearch and add DFSTransformer to AutoML pipelines#3309

Merged
jeremyliweishih merged 49 commits intomainfrom
js_2919_add_features
Mar 22, 2022
Merged

Accept features in AutoMLSearch and add DFSTransformer to AutoML pipelines#3309
jeremyliweishih merged 49 commits intomainfrom
js_2919_add_features

Conversation

@jeremyliweishih
Copy link
Collaborator

@jeremyliweishih jeremyliweishih commented Feb 7, 2022

Fixes #2919.
graphviz

@codecov
Copy link

codecov bot commented Feb 7, 2022

Codecov Report

Merging #3309 (37c4b90) into main (129e76a) will increase coverage by 0.1%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main   #3309     +/-   ##
=======================================
+ Coverage   99.6%   99.7%   +0.1%     
=======================================
  Files        329     329             
  Lines      32241   32394    +153     
=======================================
+ Hits       32112   32265    +153     
  Misses       129     129             
Impacted Files Coverage Δ
evalml/tests/component_tests/test_components.py 99.3% <ø> (ø)
...valml/automl/automl_algorithm/default_algorithm.py 100.0% <100.0%> (ø)
...lml/automl/automl_algorithm/iterative_algorithm.py 98.4% <100.0%> (+0.1%) ⬆️
evalml/automl/automl_search.py 99.7% <100.0%> (+0.1%) ⬆️
evalml/pipelines/component_graph.py 99.8% <100.0%> (+0.1%) ⬆️
...ponents/transformers/preprocessing/featuretools.py 100.0% <100.0%> (ø)
evalml/pipelines/utils.py 99.5% <100.0%> (+0.1%) ⬆️
evalml/tests/automl_tests/test_automl.py 99.5% <100.0%> (+0.1%) ⬆️
...valml/tests/automl_tests/test_default_algorithm.py 100.0% <100.0%> (ø)
...lml/tests/automl_tests/test_iterative_algorithm.py 100.0% <100.0%> (ø)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 129e76a...37c4b90. Read the comment docs.

)
first_x_component = (
pipeline.component_graph.compute_order[0]
if pipeline.component_graph.compute_order[0] != "Label Encoder"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

AFAIK this is the only case where a component will come out of make_pipelines with a component that alters y as the very first component. I could add more logic here but that would involve instantiating a component and checking if it affects x or y.

@jeremyliweishih jeremyliweishih marked this pull request as ready for review February 10, 2022 15:49
Copy link
Contributor

@chukarsten chukarsten left a comment

Choose a reason for hiding this comment

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

Left a few questions, mostly for my education and understanding, but this looks good and looks like you accomplished what you set off to do!

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.

Add DFSTransformer as part of AutoML when features are accepted as input

3 participants

Comments