Categorical columns about explainerdashboard HOT 3 OPEN

Fix for the first issue is to ignore nan values:

from explainerdashboard.

Hi @AlexanderZender ,nice catch! I'm on holiday until the end of the month so away from keyboard, but if you want to try to write up a PR I can see if I can have a look at it and get it released once I'm back

from explainerdashboard.

I found the issue(?) with my changed columns and values.
The model applied its preprocessing and these changes got applied to self.X in the explainer class.
I currently solved it by making a copy of the passed x in my Wrapper class:

The question is, should the user ensure this, or should explainer dashboard maybe only pass copies down to the model in the first place?
I can see that sometimes copies are indeed passed by explainer dashboard e.g.

I would suggest to only pass copies, to avoid potential issues with other models or pipeline in which the user might not have influence or doesnt want to add a wrapper.

@oegedijk If you dont mind i will open a PR in a bit with both things

from explainerdashboard.