atorus-research / Tplyr

f_str() docs say 'variance' instead of 'var' for the variance summary in desc layers.

Line 83 in 03f7650

#' - `variance`

This should distinctly say var, not variance.

Well darn this appears in the vignette too:

Line 64 in 03f7650

    
             `Variable Names` = c("n", "mean", "sd", "median", "variance", "min", "max", "iqr", "q1", "q3", "missing"),

I'll do a walkthrough and change it everywhere applicable.

…

On Tue, Oct 25, 2022, 2:22 PM Michael Stackhouse ***@***.***> wrote: Well darn this appears in the vignette too: https://github.com/atorus-research/Tplyr/blob/03f7650d932fd2fb5b9609c77ac6b220c991a0c7/vignettes/desc.Rmd#L64 — Reply to this email directly, view it on GitHub <#101 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A24QQLMAC5J22TM27BUISQDWFAQMPANCNFSM6AAAAAAROGW5TM> . You are receiving this because you were assigned.Message ID: ***@***.***>

@mstackhouse it seems like variance shows up in a lot of other files so I would like to read over the different files to see if there are instances where variance is meant to be used instead of var to make sure I don't accidently replace all of them if it's not applicable

@sadchla-codes Only the ones I've noted in the issue need to be changed - so no other situations in that screenshot are problems. Thanks for the thorough investigation!

okay, got it!

Typo in `f_str()` documentation on valid variables