Riccardo Cappuzzo
Riccardo Cappuzzo
Closing this
The reason I thought of this was to address the case in which datetimes are using locale specific formats, e.g. French day/month names, which I don't think are parsed properly...
In the skrub meeting we discussed the problem and decided that it would be useful to have a simpler way to implement this in the ToDatetime transformer, rather than having...
> Hey @rcap107 thanks for reviewing. I iterated on the documentation; let me know what you think It's much better, thanks a lot! I fixed a small typo, but aside...
> Just a thought, having quantile_range as a tuple instead of two separate values might make it harder for people to tune it in some hyperparameter optimization. I think it's...
> > This can be addressed in another PR > > you mean, another one for the same release? because it's not as convenient to change the parametes after release,...
Due to various reworks in the code, and the fact this PR has not seen activity in a long time, it's better to just close it and start from scratch.
How about "FrameOps" rather than "DataOps" to make it explicit that we are working with dataframes? I feel like "Data" is another very overloaded term.
I'll try to summarize the various discussions we had about this topic with the skrub devs. - We have two distinct "objects", the object with the `.skb` namespace and that...
I spent more time trying to come up with a different way to explain the expressions and this is what I came up with. Premise: I was trying to keep...