Deprecate the explain_prediction function

alteryx / evalml

EvalML is an AutoML library written in python.

https://evalml.alteryx.com

BSD 3-Clause "New" or "Revised" License

768 stars 86 forks source link

Closed freddyaboulton closed 3 years ago

freddyaboulton commented 3 years ago

Once #1818 is merged, the following two function calls will be the same:

explain_prediction(pipeline, input_features, y, index_to_explain=3)

explain_predictions(pipeline, input_features, y, indices_to_explain=[3])

I think this makes explain_prediciton redundant.

dsherry commented 3 years ago

Yes!

dsherry commented 3 years ago

I think we should follow our pattern for deprecation:

This issue tracks adding a deprecation warning now, and filing an issue to delete explain_prediction altogether
We'll schedule that second issue for the following release

I suspect it would be ok to simply delete this method, but I think its good for us to stay in the habit of giving advance notice for deprecations.