-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[Meta-issue] deprecation tracker for upcoming releases #3507
Comments
Do we really want to deprecate Edit: I would also suggest to remove |
Good point. I didn't give that one sufficient thought when I searched the repo for the word "deprecated". I'll make a PR to remove that warning. Whilst I'm at it, I shall also ensure we have the |
This is a meta-issue to track any deprecation cycles within the shap package.
Maintainers, feel free to edit this tracker as necessary,
Recommendations for which warning to use:
For deprecated parameters in e.g. plotting functions, we probably want
FutureWarning
.Excerpt from PEP0565:
Scheduled for removal
Items that raise a DeprecationWarning due to be removed in an specific release:
0.45.0
Parameters which were renamed:
Begin raising warnings for:
l1_reg="auto"
auto_size_plot
0.46.0
To be removed:
auto_size_plot
title
parameter ofshap.plots._violin.py
0.47.0
l1_reg
away from"auto"
numfeatures(10)
#3858shap.bar_plot
approximate
inTreeExplainer
initialisationapproximate
parameter to TreeExplainer for consistency, and deprecate the argument in the explainer's init method #3834Remove unused / dead code:
shap/utils/_masked_models.py:_build_delta_masked_inputs()
shap/explainers/_explainer.py:Explainer._compute_main_effects()
_build_delta_masked_inputs
andExplainer._compute_main_effects
#38560.48.0
To be removed:
shap.plots._violin.py
: remove unusedtitle
parameter0.49.0
shap.bar_plot
to use new plotting API, deleteshap.plots._bar.bar_legacy
feature_peturbation="interventional"
anddata=None
, as per MAINT refactor feature_perturbation in Tree explainers #2624approximate
inTreeExplainer
initialisation, as per FIX: Improve handling of theapproximate
parameter to TreeExplainer for consistency, and deprecate the argument in the explainer's init method #3834l1_reg="auto"
At some point, at our leisure
Remove unused / dead code:
_text.py
, got the test coverage up and looked over other functions in that module e.g._text.saliency_plot()
and_text.heatmap()
.The text was updated successfully, but these errors were encountered: