Jérôme Dockès
Jérôme Dockès
Since #1233 we can apply a transformer to a flexible set of columns using [Expr.skb.apply](https://skrub-data.org/dev/reference/generated/skrub.Expr.skb.apply.html) and a skrub selector for the `cols` parameter, and for more complex cases (like the...
Thanks @mrastgoo! I'm not sure we need to add a parameter to the fetcher, at least it is not required for addressing #1178 . we could shuffle it in the...
and IIUC the reordering is not optional and no new parameter is exposed to the user right?
>> and IIUC the reordering is not optional and no new parameter is exposed to the user right? > > As you wish > Ok in that case I'd rather...
Hey @mrastgoo ! really sorry about the delay getting back to you. The reason is that there has been a major revamp of the dataset fetchers after we ran into...
I have no objection to renaming `get_pipeline` to `make_learner`. We just need to decide what to do with `skrub.tabular_learner()` which according to this new terminology does not return a learner....
thanks a lot for this summary @rcap107 . I agree with everything you say 🙂 , including the second ("my own take") part
thank you very much for all this work @rcap107 !
discussed IRL at the weekly skrub meeting: there are a couple of other tweaks we want to apply first - deleting the stdout ("processing column x/ y") once the report...
IIRC we decided in a real-life meeting that given other changes we have made to the report (some of them outline in my comment above) it is not worth causing...