Skip to content

feat(highcharts): implement scatter-basic#479

Merged
github-actions[bot] merged 1 commit intoplot/scatter-basicfrom
auto/scatter-basic/highcharts
Dec 7, 2025
Merged

feat(highcharts): implement scatter-basic#479
github-actions[bot] merged 1 commit intoplot/scatter-basicfrom
auto/scatter-basic/highcharts

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 7, 2025

Summary

Implements scatter-basic for highcharts library.

Parent Issue: #207
Sub-Issue: #442
Base Branch: plot/scatter-basic
Attempt: 1/3

Implementation

  • plots/highcharts/scatter/scatter-basic/default.py

Details

  • Basic scatter plot with 100 random data points showing positive correlation
  • Uses Python Blue (#306998) color for data points (from style guide)
  • X and Y axis labels with readable font sizes
  • Grid lines at 10% opacity for subtle visual guidance
  • Chart dimensions: exactly 4800 x 2700 px (16:9 aspect ratio)
  • Uses PIL to crop screenshot to exact dimensions required
  • Headless Chrome + Selenium for PNG export
  • Inline Highcharts JS embedding (required for headless Chrome)

- Use PIL to crop screenshot to exact required dimensions
- Previous element screenshot produced 4800x2661 px
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 7, 2025

✅ Plot Tests Passed

Python 3.13 (required): passed

Compatibility: 3.12 3.13

Note: Only Python 3.13 is required to pass. Python 3.12 is tested for compatibility.

@claude claude bot added the ai-approved Quality OK, ready for merge label Dec 7, 2025
@github-actions github-actions bot merged commit 97b5327 into plot/scatter-basic Dec 7, 2025
8 checks passed
@github-actions github-actions bot deleted the auto/scatter-basic/highcharts branch December 7, 2025 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-approved Quality OK, ready for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants