pie chart color mapping doesn't work properly when aggregating data

Open

#2,468 opened on Jan 13, 2026 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 3/5
Estimated time: 1-2 days
Newbie friendliness: 45/100

Issue type: Bug
Clarity: Mostly clear
Activity status: Stale
Tech stack: r
Domain: data-visualization

Research direction

Start by running the two provided plot_ly examples with mtcars and compare the pie slice colors for aggregated and unaggregated data. Review issue #1375 and trace the color handling for pie charts; done when the supplied colors consistently map to the correct groups without requiring extra color entries.

Written by the indexing model from the issue text.

Description

This seems to be maybe a partial regression related to #1375. Using similar code to that issue:

library(plotly)
plot_ly(mtcars[c(1,3,5), c("cyl", "drat")], labels = ~cyl, values = ~drat, type = 'pie', 
        marker = list(colors = c("cyan", "magenta", "black"))) %>%
    layout(xaxis = list(showgrid = FALSE, zeroline = FALSE, showticklabels = FALSE),
           yaxis = list(showgrid = FALSE, zeroline = FALSE, showticklabels = FALSE))

plot_ly(mtcars[, c("cyl", "drat")], labels = ~cyl, values = ~drat, type = 'pie', 
        marker = list(colors = c("cyan", "magenta", "black"))) %>%
    layout(xaxis = list(showgrid = FALSE, zeroline = FALSE, showticklabels = FALSE),
           yaxis = list(showgrid = FALSE, zeroline = FALSE, showticklabels = FALSE))

It seems to apply the default palette color to the first group.

Interestingly, if you bang it in there a few more times it shows (though with different order).

plot_ly(mtcars[, c("cyl", "drat")], labels = ~cyl, values = ~drat, type = 'pie', 
        marker = list(colors = c("cyan", "magenta", "black", "magenta", "magenta"))) %>%
    layout(xaxis = list(showgrid = FALSE, zeroline = FALSE, showticklabels = FALSE),
           yaxis = list(showgrid = FALSE, zeroline = FALSE, showticklabels = FALSE))

Dominant language: R
Stars: 2.7k
Forks: 641
PR merge metrics: No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

print.check_outliers() mislabels thresholds when method is a combination given in non-canonical order Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

easystats/performance#950 · Opened Sep 23, 2026
Adjust the cohort counts dissemination - only include cohorts in the manifest Open

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

OHDSI/Picard#129 · Opened Sep 23, 2026
Unable to use 'start_time' 'end_time' params for get_airnow_monitors() Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

briandconnelly/airnow#9 · Opened Sep 23, 2026
plot.R: non-NA row-count filter collapses to a scalar, giving wrong point nudge spacing Open

bug

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

Merck/forestly#161 · Opened Sep 23, 2026
Copy cohorts to keep old cohorts Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

OHDSI/CohortConstructor#774 · Opened Sep 23, 2026

More R issues