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
cleanup after unloading #301
Comments
This was referenced Nov 30, 2022
Could you copy that PR here? |
This would require to undo everything done in Lines 41 to 173 in 930d8b0
inside .onUnload below it.
Somewhat cumbersome and unfortunately not easily automatable as far as I can tell, maybe it would be even worth considering some generalized machinery for this in mlr3misc maybe? |
Yep, we can definitely compress all this for sure, asked @sebffischer for help |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After unloading
mlr3proba
I think the dictionary items that were added by proba should be removed again.I made a similar PR here already: mlr-org/mlr3learners#258
The text was updated successfully, but these errors were encountered: