Skip to content

Pareto plot improvements #454

Merged
juliuspfadt merged 4 commits intojasp-stats:masterfrom
juliuspfadt:paretonumber
Jan 29, 2026
Merged

Pareto plot improvements #454
juliuspfadt merged 4 commits intojasp-stats:masterfrom
juliuspfadt:paretonumber

Conversation

@juliuspfadt
Copy link
Contributor

@juliuspfadt juliuspfadt commented Jan 26, 2026

  • change Pareto default CI number
  • adds data library examples to unit tests
  • fixes https://github.com/jasp-stats/INTERNAL-jasp/issues/3078
  • reverses the left and right y-axis for shifted accumulation line
  • adds additional support for data having a category variable and a count variable. Duplicate categories are now collapsed

Copy link
Contributor

@vandenman vandenman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe remove these two comments?


.descriptivesParetoPlots_Fill <- function(column, variable, options) {

# tableNote <- ""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# tableNote <- ""

# does the x-variable have duplicated categories, collapse them
if (length(unique(column[[1]])) != nrow(column)) {
column <- aggregate(as.numeric(as.character(column[[2]])), by = list(column[[1]]), FUN = sum)
# tableNote <- gettext("Note: Categories with multiple counts have been collapsed.")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# tableNote <- gettext("Note: Categories with multiple counts have been collapsed.")

@juliuspfadt juliuspfadt merged commit 96e2624 into jasp-stats:master Jan 29, 2026
7 of 8 checks passed
@juliuspfadt juliuspfadt deleted the paretonumber branch January 29, 2026 08:26
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