interpret-text icon indicating copy to clipboard operation
interpret-text copied to clipboard

API Inconsistencies with interpretml/interpret library

Open eddiestudies opened this issue 5 years ago • 0 comments

Looking through the API differences between interpret-text-contrib and intpret we found 3 explain_local calls in interpret-text-contrib:

  • def explain_local(self, input_text, abs_sum_to_one=False):

  • def explain_local(self, text: str, **kwargs) -> _create_local_explanation:

  • def explain_local(self, text, classes=None, predicted_label=None, true_label=None, num_iteration=150):

Interpret's spec is more like:

  • def explain_local(self, X, y=None, **kwargs)

Will be updating the explain_local calls to the above spec to improve interop with interpret. The spec is still not exact, but a push in the right direction.

We will also explicitly fail for calls of a list instead of the current failure mode. Future support for a list of strings is on the roadmap.

eddiestudies avatar Apr 28 '20 17:04 eddiestudies