ModelOriented / ModelOriented/DALEX
Check if `shap` accepts `pd.DataFrame` (and for which Explainers)
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1.5k
- Forks
- 172
- PR merge metrics
- No merged PRs in 30d
Description
...and possibly remove the conversion from pandas to numpy in ShapWrapper.
Then, explainer.model_info['arrays_accepted'] would become redundant, which in fact fixes https://github.com/ModelOriented/DALEX/issues/507 by making the scikit Pipeline fail gracefully in the downstream package (with an appropriate exception message).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect ShapWrapper and the explainer.model_info['arrays_accepted'] handling first. Check whether shap accepts pandas DataFrames for each relevant Explainer, then determine whether the conversion can be removed. Done means the supported Explainers are established and the scikit Pipeline failure is reported with an appropriate exception message.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- machine-learning
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100