-
Notifications
You must be signed in to change notification settings - Fork 7
FIX-bug-in-weighting-logic #168
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 ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #168 +/- ##
==========================================
- Coverage 96.83% 96.67% -0.16%
==========================================
Files 7 7
Lines 663 662 -1
==========================================
- Hits 642 640 -2
- Misses 21 22 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
pbiecek
left a comment
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.
Awesome, thank you for this fix
after this cross implementation verification, I think that the package is ready for version 1.0
|
@pbiecek : Thanks a lot! Good idea to increment the version to stable 1.0.0 |
|
Cool! Thanks for the heads up, this looks great. |
With input from Mario Wuethrich and Ian Covert and his repo, we have fixed a bug in how
kernelshap()calculates Kernel weights.Ping @dswatson : Note that the calculation of the exact A matrix in Covert-Lee is actually correct. We were mislead by wrong weights back then.
Ping @wueth
The resulting SHAP values were only slightly off
First example in the readme (random forest):
Comparison with Python's "shap"
Now part of the unit tests.
Python
R