-
Notifications
You must be signed in to change notification settings - Fork 22
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
Explainer api for local classifiers #102
Closed
ashishpatel16
wants to merge
37
commits into
scikit-learn-contrib:main
from
ashishpatel16:explainer_api
Closed
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
62c218d
added initial implementation of explainer api for lcppn
ea1fff8
fixed lints
c4d75c5
fixed lints
299af62
added an _explain_lcppn implementation and some tests provided
iwan-tee 0ea8956
modified docstrings
1efd946
explainer for lcpn implemented + tests added and some cases fixed
iwan-tee 7dcb52f
Merge branch 'explainer_api_lcpn' into explainer_api
iwan-tee 1de360c
tests added + some bugs fixed
iwan-tee 933b1f6
base
iwan-tee c57abed
basic implementation
iwan-tee a829ce0
LCPL explanator implementation + test
iwan-tee 33f2cbc
added tests for hierarchy without roots
c06d8a7
check on root node added
iwan-tee c597fce
minor updates
b79e5f4
codestyling
iwan-tee 8a643f1
codestyling
iwan-tee 606c1eb
Merge branch 'explainer_master' into explainer_api_lcpl
ashishpatel16 ca6c654
Update Explainer.py
ashishpatel16 9936dc3
Merge pull request #1 from ashishpatel16/explainer_api_lcpl
ashishpatel16 82573be
Merge pull request #2 from ashishpatel16/explainer_api_lcpn
ashishpatel16 d53e8d9
added support for xarray for lcppn
2449928
Merge branch 'explainer_master' into explainer_api
ashishpatel16 759489f
Update Explainer.py
ashishpatel16 0771c08
Update Explainer.py
ashishpatel16 4eb6f5c
fixed errors with classifier with single class
3955521
updated test cases and removed cached explainers
7c2f4d2
removed cached explainers
b12bdc3
modified predict proba to return dict
986b61c
Merge branch 'main' into explainer_api
ashishpatel16 eb11c0e
updated get_predict_proba to return only traversed prediction probabi…
8c700e4
updated fork
53a90a0
separate test file for explainer
5e74762
Update Explainer.py
ashishpatel16 b1f3656
_get_traversed_nodes edited
iwan-tee 2a12087
fixed lints
84f6e39
fixed conflicts
b09f8da
refactored and cleaned up code
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
@jannikgro I have updated the function to return a dict, however it currently just returns predict_proba for all nodes in the hierarchy. I'm woking on another version which only gives the dict for only traversed nodes.