Conform to sklearn estimator API
JonathanShor opened this issue · comments
Jonathan Shor commented
Changes required include, at least:
.fit(X)
returns the class instance, not the labels, to allow chained calls (clf.fit(X).predict(X)
).- Add
.predict(X)
. - Add
.predict_proba(X)
.
3a. Not sure that we need.predict_log_proba(X)
Adam Gayoso commented
What will .predict_proba(X)
return? The voting average or all p-values?
Adam Gayoso commented
I also see a number of issues with the docstring, but I'm assuming we will update those with the 2.0 release.