-
Notifications
You must be signed in to change notification settings - Fork 83
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
Display original feature values in explain predictions #2759
Display original feature values in explain predictions #2759
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2759 +/- ##
=======================================
+ Coverage 99.8% 99.8% +0.1%
=======================================
Files 301 301
Lines 27850 27897 +47
=======================================
+ Hits 27787 27834 +47
Misses 63 63
Continue to review full report at Codecov.
|
dcc6412
to
344d34a
Compare
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.
Looks great, thank you!!
I wasn't expecting you to pick this issue ASAP but this is wonderful 🏃♂️😁
for feature_name, feature_value in zip( | ||
explanation["feature_names"], explanation["feature_values"] | ||
): | ||
if feature_name == "lat": |
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.
This is just confirming that even if our original value is nan, we don't error, right?
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.
Right, it'll be displayed in the report like any other feature value!
❤️ |
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.
We live to serve our users. Thanks for taking the initiative.
344d34a
to
0b52400
Compare
Pull Request Description
Fixes #1359
Users really wanted us to display the original feature value even when there was a standard scaler, so decided to do this in this sprint because it's a small change.
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
.