-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN/API: wide_to_long or lreshape #15003
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
cc @Nuffe |
Yes having both is redundant, but I think
I could not make |
Is |
@jreback I really like |
@tdpetrou well reducing the API surface area is good. not averse to modifying |
the is we have 3 functions to do somewhat similar things. happy to consolidate the API. (aside from which documentaiton on |
The simplest addition to To add the functionality of |
I agree the current configuration is not elegant. I made an earlier PR to Looking to R and the "tidyverse" they now and then change their API and introduce new "verbs" for existing concepts: before, long was Pandas' I don't really have a good and general proposal for a solution here, more than that IMHO a nomenclature should perhaps be chosen and stuck with. |
@erikcs I made a major enhancement to melt in #17677. With that, it can simultaneously melt any number of columns, and supports any kind of multiindex (it had very poor support before that) and handles duplicate column names as well. It also has wide_to_long functionality and with a little more tweaking it will exactly replicate it. |
xref #2567
So we should drop one of these.
The text was updated successfully, but these errors were encountered: