Skip to content

Standardize figure styling to thesis palette - #16

Merged
jathavaan merged 1 commit into
mainfrom
feature/benchmark-analysis-notebook
May 25, 2026
Merged

Standardize figure styling to thesis palette#16
jathavaan merged 1 commit into
mainfrom
feature/benchmark-analysis-notebook

Conversation

@jathavaan

Copy link
Copy Markdown
Collaborator

Summary

  • Replace all Material Design colors with unified earthy/muted thesis palette centralized in src/plotting/style.py
  • Generate Databricks node-count colors programmatically via lightness ramps (strategy x node count)
  • Switch font from serif to sans-serif (DejaVu Sans) with mathtext.fontset = "dejavusans"
  • Add CLAUDE.md documenting palette, font, conventions, and semantic color mappings
  • Update packages.sty (thesis repo) with named palette block and \colorlet listings aliases

Test plan

  • All 15 chart functions render without error on synthetic data (70 PNGs)
  • Zero hex literals remain in charts.py (grep verified)
  • Font resolves to DejaVu Sans
  • Public API (color(), label(), metric_label(), workload_label(), DEFAULT_STYLE) unchanged
  • Regenerated figures included

🤖 Generated with Claude Code

Replace Material Design colors with a unified earthy/muted thesis palette
across all matplotlib charts. Palette is centralized in style.py as single
source of truth. Databricks node-count colors generated programmatically
via lightness ramps. Font changed to sans-serif (DejaVu Sans) with matching
mathtext. packages.sty in thesis repo also updated with named palette block.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 25, 2026 15:02
@jathavaan
jathavaan merged commit 924b589 into main May 25, 2026
1 check failed
@jathavaan
jathavaan deleted the feature/benchmark-analysis-notebook branch May 25, 2026 15:03
@jathavaan
jathavaan removed the request for review from Copilot May 25, 2026 15:24
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.

1 participant