Skip to content

Commit 3a46281

Browse files
fix(plotnine): address review feedback for scatter-marginal
Attempt 1/3 - fixes based on AI review - Remove top/right spines (L-shaped frame) for visual polish (DE-02) - Add bold weight to titles and axis labels for better hierarchy (DE-01) - Remove borders and grid from marginal plots for cleaner appearance - Increase axis line thickness slightly for better visibility - Improve overall visual refinement and design excellence Both light and dark theme renders verified.
1 parent 54ebf3c commit 3a46281

1 file changed

Lines changed: 17 additions & 6 deletions

File tree

  • plots/scatter-marginal/implementations/python

plots/scatter-marginal/implementations/python/plotnine.py

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
""" anyplot.ai
1+
"""anyplot.ai
22
scatter-marginal: Scatter Plot with Marginal Distributions
33
Library: plotnine 0.15.4 | Python 3.13.13
44
Quality: 89/100 | Updated: 2026-05-09
@@ -50,17 +50,18 @@
5050
main_w, main_h = 12, 6.5
5151
marg_w, marg_h = 4, 2.5
5252

53-
# Shared theme
53+
# Shared theme - L-shaped spine (left + bottom only)
5454
base_theme = theme_minimal() + theme(
5555
plot_background=element_rect(fill=PAGE_BG, color=PAGE_BG),
5656
panel_background=element_rect(fill=PAGE_BG, color=PAGE_BG),
5757
panel_grid_major=element_line(color=INK, size=0.3, alpha=0.10),
5858
panel_grid_minor=element_blank(),
59-
panel_border=element_rect(color=INK_SOFT, fill=None, size=0.4),
60-
axis_title=element_text(color=INK, size=20),
59+
panel_border=element_blank(),
60+
axis_title=element_text(color=INK, size=20, weight="bold"),
6161
axis_text=element_text(color=INK_SOFT, size=16),
62-
axis_line=element_line(color=INK_SOFT, size=0.4),
63-
plot_title=element_text(color=INK, size=24, margin={"t": 10, "b": 10}),
62+
axis_line=element_line(color=INK_SOFT, size=0.5),
63+
axis_ticks=element_line(color=INK_SOFT, size=0.4),
64+
plot_title=element_text(color=INK, size=24, weight="bold", margin={"t": 10, "b": 10}),
6465
)
6566

6667
# Top histogram (x distribution)
@@ -74,9 +75,14 @@
7475
figure_size=(main_w, marg_h),
7576
axis_text_x=element_blank(),
7677
axis_ticks_major_x=element_blank(),
78+
axis_ticks_minor_x=element_blank(),
79+
axis_line_x=element_blank(),
7780
axis_title_y=element_blank(),
7881
axis_text_y=element_blank(),
7982
axis_ticks_major_y=element_blank(),
83+
axis_ticks_minor_y=element_blank(),
84+
axis_line_y=element_blank(),
85+
panel_grid_major=element_blank(),
8086
)
8187
)
8288

@@ -92,9 +98,14 @@
9298
figure_size=(marg_w, main_h),
9399
axis_text_y=element_blank(),
94100
axis_ticks_major_y=element_blank(),
101+
axis_ticks_minor_y=element_blank(),
102+
axis_line_y=element_blank(),
95103
axis_title_x=element_blank(),
96104
axis_text_x=element_blank(),
97105
axis_ticks_major_x=element_blank(),
106+
axis_ticks_minor_x=element_blank(),
107+
axis_line_x=element_blank(),
108+
panel_grid_major=element_blank(),
98109
)
99110
)
100111

0 commit comments

Comments
 (0)