On 11/14/2016 01:29 AM, Anton wrote:

screws you over since it passes the parameters to __init__(). I expected the usage of set_params() here, but I'm getting my gridsearch parameters passed to __init__().
Is this intended?

I don't know what you mean by "screws you over". You just have to explicitly list all parameters.

There is a hidden assumption that next to set_param() some methods may alternatively use __init__() to set parameters. That's why I had to jump through multiple hoops to get a meta-classifier which transparently shadows all variables. Usually you would expect that all parts stick to the convention of using one way to set parameters only (set_params())

Why would you expect that? Given the way clone works, basically no part of scikit-learn does that. Have you read http://scikit-learn.org/dev/developers/contributing.html#rolling-your-own-estimator ?

_______________________________________________
scikit-learn mailing list
[email protected]
https://mail.python.org/mailman/listinfo/scikit-learn

Reply via email to