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

Add denominator to LBT14 #682

Open
mhallal1 opened this issue Oct 24, 2023 · 5 comments
Open

Add denominator to LBT14 #682

mhallal1 opened this issue Oct 24, 2023 · 5 comments

Comments

@mhallal1
Copy link

After discussion with @BFalquet, I am creating this issue to ask for the option to be able to add the denominators of the population to the table LBT14.

@clarkliming
Copy link
Contributor

clarkliming commented Oct 26, 2023

do you mean total N in the header? it is already included.

Maybe you can add some details to this question

image

@BFalquet
Copy link
Contributor

They would like to have 71 / 121 instaead of 72 (53.7%)

@mhallal1
Copy link
Author

They would like to have 71 / 121 instaead of 72 (53.7%)

exactly, thanks @BFalquet.

@oreillb1
Copy link

@mhallal1 @BFalquet I believe this will help output reviewers to understand the reported data.

@clarkliming
Copy link
Contributor

@BFalquet as lbt14 use the same analysis across the template, we can consider the following:

  1. expose the arguments like stats, formats, etc in the main function
  2. expose only the required arguments
  3. do not expose these arguments, but use ... to tell the user that this handles extra arguments for that occurrence

anyway this is non-standard so have to keep the original functionality

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants