Skip to content
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

Feature Request: total_col for numeric variables #17

Closed
nrkoehler opened this Issue Mar 21, 2019 · 7 comments

Comments

Projects
None yet
2 participants
@nrkoehler
Copy link

nrkoehler commented Mar 21, 2019

Would you consider making the total_col argument of the summary_factorlist() function work for numeric variables? So far, this argument works for factors (and maybe character vars) only. Thanks in advance, Norbert

ewenharrison added a commit that referenced this issue Mar 21, 2019

@ewenharrison

This comment has been minimized.

Copy link
Owner

ewenharrison commented Mar 21, 2019

Thanks. This is done. Let me know if it doesn’t do what you expect/want.

@nrkoehler

This comment has been minimized.

Copy link
Author

nrkoehler commented Mar 21, 2019

Oh, thanks a lot - that was quick! :-) I have just installed version 0.9.1 and it works. Best regards, Norbert

@nrkoehler

This comment has been minimized.

Copy link
Author

nrkoehler commented Mar 22, 2019

Sorry, but I just discovered that the column_total option of the summary_factorlist() function does not work for numeric variables as I expected. I would like the the function to return mean and standard deviation (or median and IQR) for the whole sample. Instead it returns the number and percentages of non-missing values. Best regards, Norbert

@ewenharrison

This comment has been minimized.

Copy link
Owner

ewenharrison commented Mar 25, 2019

Thanks Norbert. Ah I see now what you were asking for. That is not planned at the moment, but we will bear in mind for the next re-write. Thank you again.

@nrkoehler

This comment has been minimized.

Copy link
Author

nrkoehler commented Mar 25, 2019

Ok, thank you!

@ewenharrison

This comment has been minimized.

Copy link
Owner

ewenharrison commented Mar 27, 2019

This is now done in 0.9.2

@nrkoehler

This comment has been minimized.

Copy link
Author

nrkoehler commented Mar 28, 2019

Thanks a lot!

ewenharrison added a commit that referenced this issue Apr 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.