-
Notifications
You must be signed in to change notification settings - Fork 2
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
Project: prunr package #10
Comments
Yes. I've hit this and I know @jonocarroll has as well. I note that Maybe there is some kind of wrapper function that could enable all said options? I'm thinking along the lines of this package: https://github.com/rbertolusso/intubate Bonus evil Hadley: https://twitter.com/hadleywickham/status/759412516539600896 |
Okay so after reading your links I see the options are not as effective as one might hope. I still reckon that general architecture might be the right way to go though. It could intercept that model object and set references to NULL as you suggest. Just have to somehow ensure garbage collection is happening often enough. |
Ha, I like the Hadley comment :) and By general architecture do you mean that we have a function that first looks at the object type and then removes components based on the type? i.e., there will be a set of rules for an |
I've been down this road a few times and (as the WinVector blog shows) there's lots of data that comes along for the ride and ends up in the final, very complex, model output object. I tend to find though that different analyses require different parts of that object, hence I doubt all of the components are redundant and can be generally removed. A few points I will note as I wait for my flight to BNE:
Food for thought, but an interesting project for sure. I'll be keen to hear what comes of it. |
I'm interested in creating a package to automatically prune large and unneeded components from objects to help with caching performance. Details and motivation in this gist
https://gist.github.com/camroach87/12b658afdd9f2d051721ad21311a960a
Thoughts, suggestions, package name improvements all welcome. Also - if someone has already created something like this please let me know because I will use it all the time :)
The text was updated successfully, but these errors were encountered: