-
Notifications
You must be signed in to change notification settings - Fork 367
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
[BREAKING] deprecate name => fun in favor of fun => name in describe #2401
[BREAKING] deprecate name => fun in favor of fun => name in describe #2401
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good with fixed indentation!
@matthieugomez - what I changed here is what we all agree to change. Now - do you have strong preference for your proposal? If yes, just to make sure, you envision the signature to be:
where stats is a single stat or a vector of statistics to calculate? (if we are breaking things I would not oppose this that much, but I want to make sure that your proposal has the general support) |
I would prefer it but it's not the hill I'm ready to die on ;) It's
interesting to hear you were using it in a different way.
…On Mon, Aug 31, 2020 at 10:20 AM Bogumił Kamiński ***@***.***> wrote:
@matthieugomez <https://github.com/matthieugomez> - what I changed here
is what we all agree to change.
Now - do you have strong preference for your proposal? If yes, just to
make sure, you envision the signature to be:
describe(df, cols=:; stats=[:mean, :min, :median, :max, :nmissing, :eltype])
where stats is a single stat or a vector of statistics to calculate?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2401 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABPPPXPI6I2OHRPXT4L6FO3SDPLXTANCNFSM4QQQQACQ>
.
|
@nalimilan - do you have any preference here? |
I don't have a strong opinion. At least what is clear is that functions which take a |
Thank you for considering it with such care. Given all pros and cons I am leaning to leave the PR as it stands now. |
@nalimilan - so do we go for this or wait a bit for more feedback (I feel all has been said already). |
Yeah, let's go with this since there's no perfect solution. |
Co-authored-by: Milan Bouchet-Valat <nalimilan@club.fr>
Thank you! |
Fixes #2385