-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
modelStudio(), explainer_mlr3() and NAs #71
Comments
Hi,
I can tell that (2) is within reach, while (3) needs more thought process. If you have more issues don't hesitate. |
Hi, Many thanks for the quick response. Great to hear 1 has been dealt with and 2 is within reach. Point 3 is indeed a tricky one. For the time being, my intention is to hard-code the chunks of code in |
Hi, Thank You @andreassot10 for an extensive example. Right now explain_mlr3 supports GraphLearner objects. You can check it downloading the package from github. In case of any problems feel free to raise an issue! |
That's amazing, thanks! |
TODO: add Line 601 in 867e12a
and more |
Hi, I opened an issue on Thanks |
Hi @Tato14, |
Hi @hbaniecki, should be from mlr3pipelines (page 195) |
@Tato14 sure it works, but now I have another problem: |
@hbaniecki I apologise for the inconvenience. I add the missing variables in the code. Now it should work properly. |
Hi,
There's a glitch with
modelStudio
when usingmlr3
pipelines with data with missing values.It looks like
modelStudio()
doesn't know how to impute missing data before crunching the numbers, even when the user has incorporated a pipe operator for missing values in themlr3
pipeline. In fact,modelStudio()
does not even recognizemlr3
learners if their class is other than[1] "LearnerClassifRanger" "LearnerClassif" "Learner" "R6"
(e.g. tryclass(learner)
for a Random Forest learner). If you have a pipeline, whose class is[1] "GraphLearner" "Learner" "R6"
,modelStudio()
doesn't know how to handle it.Package
DALExtra
'sexplainer_mlr3()
suffers from the same issue, although this can be dealt with by providing custom functions for argumentspredict_function
andresidual_function
.Below is an example of a pipeline that imputes missing data and then balances classes. Note that it works fine when there are no missing data, but returns an error otherwise.
Example 1: no missing data
Working just fine.
Example 2: missing data
We get errors and no plot:
Is there a way to pass imputed data from
explainer_mlr3()
tomodelStudio()
just like you can pass predictions and residuals with argumentspredict_function
andresidual_function
respectively? Any chances of implementing this please?Thanks
The text was updated successfully, but these errors were encountered: