Skip to content
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.

Sign up for GitHub

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

Jump to bottom

87 implement explainer #105

Merged
merged 4 commits into from
Apr 9, 2024
Merged

87 implement explainer #105

merged 4 commits into from
Apr 9, 2024

Conversation

hbaniecki
Copy link
Collaborator

@hbaniecki hbaniecki commented Apr 8, 2024 •
edited
Loading

Added a first attempt at the shap.Explainer class #87

Refactored (for now):

  • x_explain -> x
  • background_data -> data
  • x_data -> data
  • x_background -> data

hbaniecki added 4 commits April 8, 2024 17:21
@hbaniecki
add basic explainer api, refactor code
e276fd1 
`x_explain` -> `x`
`background_data` -> `data`
@hbaniecki
add some automatic predict functions; refactor code
a8f9810 
`x_data` -> `data`
@hbaniecki
fix predict in imputer
b3920ca 
`x_background` -> `data`
@hbaniecki
resolve conflicts
6f4c8ef
@mmschlk
Copy link
Owner

mmschlk commented Apr 9, 2024

do you want me to fix the code-quality check before we merge?

@mmschlk mmschlk merged commit 270b6c1 into main Apr 9, 2024
4 of 5 checks passed
@mmschlk mmschlk deleted the 87-implement-explainer branch April 11, 2024 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hbaniecki @mmschlk