Skip to content

[Fix] Support trace-specific color sequences in Plotly Express via templates #5437

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.

Sign up for GitHub

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

Merged
emilykl merged 13 commits into from
Dec 5, 2025
+75 −3
Merged

[Fix] Support trace-specific color sequences in Plotly Express via templates #5437

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
965b3bd
Add trace-specific color sequence support from template.data
antonymilne Nov 26, 2025
83faec8
Support trace-specific color sequences in Plotly Express via templates
antonymilne Nov 26, 2025
7bdec7f
Add trace-specific color sequences in Plotly Express via templates
antonymilne Nov 26, 2025
befea7d
Minor
antonymilne Nov 27, 2025
d578ab4
Format code with ruff
antonymilne Nov 27, 2025
a2dc490
Add CHANGELOG entry for trace-specific color sequences fix
antonymilne Dec 1, 2025
74543f3
Update changelog
antonymilne Dec 1, 2025
675c668
Refactor trace-specific color handling in apply_default_cascade
antonymilne Dec 4, 2025
c24b05c
Remove redundant color_discrete_sequence check in apply_default_cascade
antonymilne Dec 4, 2025
076ff14
Update tests/test_optional/test_px/test_px.py
antonymilne Dec 4, 2025
a3853d2
Ruff
antonymilne Dec 4, 2025
af1546c
Merge branch 'main' into main
antonymilne Dec 4, 2025
e8339e0
Fix imshow
antonymilne Dec 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Refactor trace-specific color handling in apply_default_cascade 
- Make constructor parameter required (was optional with None default)
- Refactor trace-specific color extraction to only assign when colors are found
- Improve code clarity by checking for non-empty color list before assignment
  • Loading branch information
@antonymilne
antonymilne committed Dec 4, 2025
commit 675c668fcfefab9b4b3b1a0ed81a36ac22e807d4
8 changes: 6 additions & 2 deletions plotly/express/_core.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1003,7 +1003,7 @@ def one_group(x):
return ""


def apply_default_cascade(args, constructor=None):
def apply_default_cascade(args, constructor):
# first we apply px.defaults to unspecified args

for param in defaults.__slots__:
Expand Down Expand Up @@ -1047,12 +1047,16 @@ def apply_default_cascade(args, constructor=None):
else:
trace_type = constructor().type
if trace_data_list := getattr(args["template"].data, trace_type, None):
args["color_discrete_sequence"] = [
trace_specific_colors = [
trace_data.marker.color
for trace_data in trace_data_list
if hasattr(trace_data, "marker")
and hasattr(trace_data.marker, "color")
]
# If template contains at least one color for this trace type, assign to color_discrete_sequence
if any(trace_specific_colors):
args["color_discrete_sequence"] = trace_specific_colors

if not args["color_discrete_sequence"] or not any(
args["color_discrete_sequence"]
):
Expand Down