<div dir="ltr"><div class="gmail_default" style="font-size:small">Hi,</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">I'm (mostly) the messenger, don't shoot me :P</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">It may help to summarize the SLEP:</div><div class="gmail_default" style="font-size:small">1. This can be applied to all methods, not just __init__.</div><div class="gmail_default" style="font-size:small">2. The SLEP doesn't say we have to apply it everywhere. It's mostly that it lets us do that.<br></div><div class="gmail_default" style="font-size:small">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.</div><div class="gmail_default" style="font-size:small">   Therefore clf.fit(X, y) will definitely be allowed.<br></div><div class="gmail_default" style="font-size:small">4. Whether or not sample_weight should be keyword only or not in fit, requires its own discussion, and the route of the discussion</div><div class="gmail_default" style="font-size:small">   is defined in the SLEP.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">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</div><div class="gmail_default" style="font-size:small">a keyword only argument. But the SLEP better defines these conditions.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">I hope that clarifies it a little bit.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Adrin/<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 11, 2019 at 3:23 PM Alexandre Gramfort <<a href="mailto:alexandre.gramfort@inria.fr">alexandre.gramfort@inria.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">hi,<br>
<br>
Adrin do you suggest this for everything or maybe just for __init__<br>
params of estimators<br>
and stuff that can come after X, y in fit eg sample_weights?<br>
<br>
would:<br>
<br>
clf.fit(X, y)<br>
<br>
still be allowed?<br>
<br>
Alex<br>
_______________________________________________<br>
scikit-learn mailing list<br>
<a href="mailto:scikit-learn@python.org" target="_blank">scikit-learn@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/scikit-learn" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/scikit-learn</a><br>
</blockquote></div>