feat(python,rust!): Infer values columns in DataFrame.pivot when values is None

[mcrumiller]

Resolves #12087.

I think we hold off until the next breaking release.

[stinodego]

I think we hold off until the next breaking release.

Yes, this is breaking. You can remove the deprecation decorator from the method too.

[mcrumiller]

@stinodego got a Chocolatey CI error here, should keep an eye open to see if it pops up elsewhere.

[stinodego]

@stinodego got a Chocolatey CI error here, should keep an eye open to see if it pops up elsewhere.

This is a known issue - I reported it here: ts-graphviz/setup-graphviz#557

[mcrumiller]

@stinodego I think we should revert the set usage, unless you can think of a better way out. The order of the output columns in pivot is now unpredictable. I suppose we could sort them to make the output predictable? Preserving the order would be nice though.

[stinodego]

@stinodego I think we should revert the set usage, unless you can think of a better way out. The order of the output columns in pivot is now unpredictable. I suppose we could sort them to make the output predictable? Preserving the order would be nice though.

Ah, I hadn't considered that. Let's revert it then 👍

[mcrumiller]

Ah, I hadn't considered that. Let's revert it.

I have a way to both use the faster sets path and preserve column order.

[stinodego]

I have a way to both use the faster sets path and preserve column order.

I don't think it's worth allocating a new set here. Let's just go with the initial logic which was simplest. We can further optimize this later if required.

I made the update and this is now good to merge 👍

[mcrumiller]

Thanks for the help @stinodego!