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.

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

Fixed plot rendering error #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AbhinavTrips
Copy link

Instead of using sahp_values in st_shap, we need to use shap_values.values , because shap_values holds the shapley values, the base_values and the data. Visualize() error is thrown if just shap_values is used

Instead of using sahp_values in st_shap, we need to use shap_values.values , because shap_values holds the shapley values, the base_values and the data. Visualize() error is thrown if just shap_values is used
@snehankekre
Copy link
Owner

Thank you so much for the PR, @AbhinavDAIICT! I'll have time to review and test it sometime next week, after which I'll release a new version on PyPI.

@snehankekre snehankekre self-requested a review July 27, 2022 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants