-
Notifications
You must be signed in to change notification settings - Fork 91
Moves permutation importance from evalml/pipeline/utils.py to graph_utils.py #880
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #880 +/- ##
=======================================
Coverage 99.75% 99.75%
=======================================
Files 195 195
Lines 8503 8505 +2
=======================================
+ Hits 8482 8484 +2
Misses 21 21
Continue to review full report at Codecov.
|
docs/source/changelog.rst
Outdated
@@ -39,6 +40,7 @@ Changelog | |||
**Breaking Changes** | |||
* Pipelines' static ``component_graph`` field must contain either ``ComponentBase`` subclasses or ``str``, instead of ``ComponentBase`` subclass instances :pr:`850` | |||
* Rename ``handle_component`` to ``handle_component_class``. Now standardizes to ``ComponentBase`` subclasses instead of ``ComponentBase`` subclass instances :pr:`850` | |||
* ``calculate_permutation_importances`` and ``graph_permutation_importances`` were moved from `evalml.pipeline.utils` to `evalml.pipeline.graph_utils` :pr:`880` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we haven't released this yet, no need to add to breaking changes.
Also, perhaps you can just add the PR number to the end of the previous entry in "Enhancements" for #860
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!! Left one comment about the changelog but otherwise 🚢 !
Closes #875