Skip to content

WW Accessor: Update model understanding#2247

Merged
freddyaboulton merged 2 commits into2035-use-ww-accessorfrom
2035-update-model-understanding
May 11, 2021
Merged

WW Accessor: Update model understanding#2247
freddyaboulton merged 2 commits into2035-use-ww-accessorfrom
2035-update-model-understanding

Conversation

@freddyaboulton
Copy link
Copy Markdown
Contributor

Pull Request Description

Update model understanding module


After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 10, 2021

Codecov Report

Merging #2247 (75a863f) into 2035-use-ww-accessor (e728ec5) will increase coverage by 11.7%.
The diff coverage is 100.0%.

Impacted file tree graph

@@                   Coverage Diff                   @@
##           2035-use-ww-accessor   #2247      +/-   ##
=======================================================
+ Coverage                  69.2%   80.9%   +11.7%     
=======================================================
  Files                       280     280              
  Lines                     24012   23990      -22     
=======================================================
+ Hits                      16609   19392    +2783     
+ Misses                     7403    4598    -2805     
Impacted Files Coverage Δ
evalml/model_understanding/graphs.py 100.0% <100.0%> (+83.7%) ⬆️
...nderstanding/prediction_explanations/explainers.py 100.0% <100.0%> (+77.7%) ⬆️
...s/prediction_explanations_tests/test_explainers.py 100.0% <100.0%> (+100.0%) ⬆️
...lml/tests/model_understanding_tests/test_graphs.py 100.0% <100.0%> (+100.0%) ⬆️
...del_understanding_tests/test_partial_dependence.py 100.0% <100.0%> (+100.0%) ⬆️
...understanding_tests/test_permutation_importance.py 100.0% <100.0%> (+100.0%) ⬆️
evalml/pipelines/component_graph.py 100.0% <0.0%> (+1.3%) ⬆️
...lines/components/transformers/preprocessing/lsa.py 100.0% <0.0%> (+2.8%) ⬆️
evalml/pipelines/pipeline_base.py 98.6% <0.0%> (+2.9%) ⬆️
...components/transformers/encoders/target_encoder.py 100.0% <0.0%> (+2.9%) ⬆️
... and 16 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 e728ec5...75a863f. Read the comment docs.

@freddyaboulton freddyaboulton marked this pull request as ready for review May 10, 2021 19:39
Copy link
Copy Markdown
Contributor

@bchen1116 bchen1116 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
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.

Love it anytime the subtracted lines is greater than added lines and the code looks cleaner.

@freddyaboulton freddyaboulton merged commit 829912e into 2035-use-ww-accessor May 11, 2021
@freddyaboulton freddyaboulton deleted the 2035-update-model-understanding branch May 11, 2021 23:14
freddyaboulton added a commit that referenced this pull request May 25, 2021
* WW Accessor: Update demo datasets, preprocessing, and utils (#2172)

* Update demos, utils, and preprocessing

* Updating make test commands

* Update ww requirement

* Add test that infer_feature_types preserves schema

* Add test that infer_feature_types raises errors with invalid schema

* load_data always returns woodwork info. Deleted return_pandas

* Updating docstrings

* Release notes for first PR

* Deleting redundant tests

* update test name

* WW 0.2.0: Update Data Checks (#2182)

* Updating data checks

* Undo accidental edit

* Fixing typo where we assign to ww.init

* WW Accessor: Updating objectives (#2185)

* Updating objectives

* Deleting superfluous pd.Series

* WW Accessor: Updating Components (#2191)

* Update components - first commit

* Update delayed_features_transformer to not use assign

* Fixing tests

* Skipping Boolean with Nan test in imputers

* Fixing base sampler _prepare_data

* Fixing target imputer null bool test

* Fix test skips

* Addressing comments

* Clean up sampler tests

* Editing docstrings

* Update to ww 0.3.0

* Fixing components that didn't have merge conflicts

* WW 0.2.0 Update: Updating pipelines (#2205)

* Updating pipelines

* Fixing docstrings

* Fix stacked test

* Add tests to check input data not modified

* Use rename in component graph

* WW Accessor: Update model understanding (#2247)

* Update model understanding module

* Removing unused import

* WW Accessor: Update Automl (#2243)

* Updating pipelines

* Fixing docstrings

* Fix stacked test

* Update automl

* Update to preserve schema

* Preserve schema in train_pipelines/score_pipelines

* No ww drop

* Update standard scaler

* Use weak ref feture branch

* Add tests to check input data not modified

* Use weak-ref branch

* Use weak ref branch

* Fixing tests

* Set ww back to 0.3.0

* Adding objective tests that use automl

* Add back minimal-dependencies-flag

* Fix test command

* Updating docstrings adding dask test to check schema is sent

* Fixing import order in test_dask_engine

* Fixing last remaining docstrings

* Fix typo in ignore command

* Use ww init series

* No doctest modules yet

* Fix docstrings & use schemas stored in automl_config

* WW Update: Update docs and docstrings (#2292)

* Fixing docs and docstrings

* Update docstring in highly_null_data_check

* Delete warning from init

* Remove print(X.ww)

* Fixing notebook python version :(

* Update release notes

* Delete _convert_woodwork_types_wrapper completely

* Fixing coverage

* Add back standard scaler try

* Remove _convert_woodwork_types_wrapper from docs'

* Linting docs

* Merging main

* Fix release notes
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