Skip to content

feat(letsplot): implement scatter-basic#465

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

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

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 7, 2025

Summary

Implements scatter-basic for letsplot library.

Parent Issue: #207
Sub-Issue: #410
Base Branch: plot/scatter-basic
Attempt: 2/3

Implementation

  • plots/letsplot/point/scatter-basic/default.py

Features

  • Uses lets-plot ggplot2-style grammar of graphics
  • Python Blue (#306998) color for data points
  • Proper sizing (4800x2700px via scale=3)
  • theme_minimal() with custom font sizes for readability
  • Follows all quality criteria and style guide requirements

Note

Previous PR #449 was closed due to transient GitHub Actions infrastructure timeout (unrelated to code).

Add theme_minimal() for a cleaner look with subtle gridlines.
@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 1565d81 into plot/scatter-basic Dec 7, 2025
15 of 16 checks passed
@github-actions github-actions bot deleted the auto/scatter-basic/letsplot branch December 7, 2025 10:27
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