Closed enisnazif closed 7 years ago
Hi @enisnazif , this looks good and I would have merged it. Unfortunately, LOO Cross-validation has already been added in #23. I think you should regularly update your fork of Xcessiv since it is changing very quickly right now.
Don't worry though, you're on the right track. There are lots of other metrics and estimators that aren't in Xcessiv yet. In particular, I'd like the sklearn regressors to be added next. But feel free to add whichever you like :)
Ok, thanks that's good to know. I'll start on the sklearn regressors :)
Added Leave One Out Cross validation as part of #15
I'm keen to finish implementing all of the cv / metrics within sklearn, just wanted to make sure I was doing it right since this is my first pull request!