Thanks, Joel.  From your response I assume that the use of a y argument to
score_samples is not a violation of the sklearn API, so I'll keep the
method and find a workaround for the check_estimator test as it's currently
written.  I'll comment on the issue as well.



On Mon, Dec 10, 2018 at 2:58 PM Joel Nothman <joel.noth...@gmail.com> wrote:

> We're trying to make check_estimator more flexible (
> https://github.com/scikit-learn/scikit-learn/pull/8022) but this is
> certainly not something we had considered yet. Perhaps suggest it there?
>
> Or for now we could just make the check pass if score_samples yields a
> TypeError with only X...
> _______________________________________________
> scikit-learn mailing list
> scikit-learn@python.org
> https://mail.python.org/mailman/listinfo/scikit-learn
>
_______________________________________________
scikit-learn mailing list
scikit-learn@python.org
https://mail.python.org/mailman/listinfo/scikit-learn

Reply via email to