Skip to content

Feature/woodelf stii fsii fbii - #595

Open
ron-wettenstein wants to merge 6 commits into
mmschlk:mainfrom
ron-wettenstein:feature/woodelf_stii_fsii_fbii
Open

Feature/woodelf stii fsii fbii#595
ron-wettenstein wants to merge 6 commits into
mmschlk:mainfrom
ron-wettenstein:feature/woodelf_stii_fsii_fbii

Conversation

@ron-wettenstein

Copy link
Copy Markdown
Contributor

Motivation and Context

I recently supported STII, FSII and FBII indices in Woodelf.
This PR allow shapiq to use Woodelf also on these indices.
TreeExplainer will use Woodelf on n*m>100000 for interventional SHAP.
It can also use Woodelf on STII, FSII and FBII in path dependent when backend="woodelf" (but not by default).

shapiq[tree] now depends on woodelf_explainer=0.4.7 version (instead of 0.4.6).


Public API Changes

  • No Public API changes
  • Yes, Public API changes (Details below)

How Has This Been Tested?

Added tests the compare shapiq's output to Woodelf's on these indices and make sure they are identical.
Also validated it locally.
The woodelf package also has its own tests for this logic.


Checklist

  • The changes have been tested locally.
  • Documentation has been updated (if the public API or usage changes).
  • An entry has been added to CHANGELOG.md (if relevant for users).
  • The code follows the project's style guidelines.
  • I have considered the impact of these changes on the public API.

@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant