verde icon indicating copy to clipboard operation
verde copied to clipboard

Fix pandas warnings about .ravel()

Open leouieda opened this issue 1 year ago • 2 comments

Pandas was issuing FutureWarnings about calling Series.ravel, which could be fixed by calling np.ravel() instead, and a keyword argument of the groupby .apply changing the default to not include the index in the calculations (which is the right thing to do).

leouieda avatar Apr 17 '24 19:04 leouieda

@santisoler the other warning about aggregate I couldn't find a way to fix. I think we'll just have to live with it until Pandas makes a new release. Or we pass strings to BlockReduce instead of functions but I'm not a huge fan.

leouieda avatar Apr 17 '24 19:04 leouieda

So the pandas warning about apply won't work on older pandas...

I think we should leave it as is and let pandas change their default when they want. It's not like we're doing something completely wrong since the tests pass with or without the new argument.,

leouieda avatar Apr 17 '24 19:04 leouieda