ConfoundingSHAP#551
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Very nice! One question I had is weather we should merge this PR after the refactor of the benchmark? Or what do you think @mariebrockschmidt and @Advueu963? |
|
@mmschlk this would not really introduce many changes in shapiq-benchmark, only a few registrations in the dictionaries of shapiq-benchmark. But also happy to do so after the merge of shapiq-benchmark. |
|
@mariebrockschmidt and @Advueu963, now with 1.6.0 we have the rework of the shapiq-benchmark in. We could now put this one in as well? Are there any things we need to do now or is this already good to go? :) |
mmschlk
left a comment
There was a problem hiding this comment.
From me this is good. But @Advueu963 should still take a look. :)
Motivation and Context
This PR adds the ConfoundingSHAP games defined in the paper:
Public API Changes
Adding the shapiq_games for further benchmarking.
How Has This Been Tested?
example file has been created and functionality was tested locally.
Checklist
CHANGELOG.md(if relevant for users).