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

Fix density plots #352

Merged
merged 2 commits into from
Mar 6, 2025
Merged

Conversation

vandenman
Copy link
Contributor

Broken by @RensDofferhoff in 440f765

image

preferably next time open a PR so we can see if any unit tests fail 😅

because this is also broken in the current release:

image

This fix is more of a band-aid until we fully commit to preloadData.

@JohnnyDoorn let me know if you don't have time to review this!

@vandenman
Copy link
Contributor Author

I'll also fix the failing i18n check (see https://github.com/jasp-stats/jaspDescriptives/actions/runs/13316068109/job/37190295021?pr=351) so please don't merge this even if you review it.

@shun2wang
Copy link
Contributor

shun2wang commented Feb 28, 2025

I'll also fix the failing i18n check (see https://github.com/jasp-stats/jaspDescriptives/actions/runs/13316068109/job/37190295021?pr=351) so please don't merge this even if you review it.

you can look at #349 I also pushed a fix for it.

@vandenman
Copy link
Contributor Author

@shun2wang thanks! I merged your pr :)

@JohnnyDoorn
Copy link
Contributor

Good catch @vandenman !
There still seems something broken if a spit variable is added, where .descriptivesDensityPlotsFill() is not updated to include the preloadData argument

image

@vandenman
Copy link
Contributor Author

Oops @JohnnyDoorn good catch! Should be fixed now.

@vandenman vandenman merged commit 85c0c4e into jasp-stats:master Mar 6, 2025
5 checks passed
@vandenman vandenman deleted the fix_density_plots branch March 6, 2025 14:14
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.

3 participants