Hi, I'm (mostly) the messenger, don't shoot me :P
It may help to summarize the SLEP: 1. This can be applied to all methods, not just __init__. 2. The SLEP doesn't say we have to apply it everywhere. It's mostly that it lets us do that. 3. It doesn't make ALL inputs a keywords only argument. The common ones such as X and y in fit(X, y) will stay as they are. Therefore clf.fit(X, y) will definitely be allowed. 4. Whether or not sample_weight should be keyword only or not in fit, requires its own discussion, and the route of the discussion is defined in the SLEP. In other words, if an input parameter is used as a positional argument less frequently than X% of the time, then it can/should be a keyword only argument. But the SLEP better defines these conditions. I hope that clarifies it a little bit. Adrin/ On Wed, Sep 11, 2019 at 3:23 PM Alexandre Gramfort < alexandre.gramf...@inria.fr> wrote: > hi, > > Adrin do you suggest this for everything or maybe just for __init__ > params of estimators > and stuff that can come after X, y in fit eg sample_weights? > > would: > > clf.fit(X, y) > > still be allowed? > > Alex > _______________________________________________ > 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
