You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for the suggestions, but I have a few questions. What is the intended use for the subsetLearners function?
For the mergeSuperLearners, this should require some checks that both SuperLearner objects where run on the same data, and with the same cross-validation splits.
Hi Eric, thanks for reply, I later realised that there's no point in removing models since SuperLearner is about doing precisely that, so function subsetLearners is meaningless. For the mergeSuperLearners I'll add the checks. Also I thought about adding some logging regarding how much time it takes for a model to train and predict. I'm new to open source development and R, so if there is any roadmap for the planned features I'd love to implement some of them, would be great for learning.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I hope I'm not missing something