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

descriptives: add weights support #425

Merged

Conversation

raviselker
Copy link
Member

Implements weights support for a number of descriptive statistics. Not all descriptives functionality is supported yet (most notable, the plots), but the user gets a warning message when they select an unsupported option.

@raviselker raviselker force-pushed the feature/weighted-descriptives branch from 42d44d5 to 54ce215 Compare February 12, 2025 22:28
Implements weights support for a number of descriptive statistics.
Not all descriptives functionality is supported yet (most notable, the plots),
but the user gets a warning message when they select an unsupported option.
@raviselker raviselker force-pushed the feature/weighted-descriptives branch from 753de50 to a90cd05 Compare February 19, 2025 14:04
Previously, weighted descriptive stats were shown as missing values in
the table. This commit changes this to NaN values. Also adds a "not supported yet" footnote for the non transposed
table
@raviselker raviselker force-pushed the feature/weighted-descriptives branch from a90cd05 to 179d8e5 Compare February 19, 2025 15:02
@jonathon-love
Copy link
Member

did some testing; throws an error with quartiles

Screenshot 2025-02-23 at 14 40 03

Tooth Growth 2.omv.zip

@raviselker
Copy link
Member Author

Good catch! For some reason I was unable to replicate it in my R version (maybe a different way of dealing with NA values in if statements between the jamovi R version and my local R version), but I was able to test it in jamovi using your example. I added a commit that fixes this.

@jonathon-love jonathon-love merged commit 22c92b3 into jamovi:master Mar 3, 2025
5 checks passed
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

Successfully merging this pull request may close these issues.

2 participants