Skip to content

feat(plotly): implement line-basic#502

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

feat(plotly): implement line-basic#502
github-actions[bot] merged 1 commit intoplot/line-basicfrom
auto/line-basic/plotly

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude Bot commented Dec 7, 2025

Summary

Implements line-basic for plotly library.

Parent Issue: #205
Sub-Issue: #454
Base Branch: plot/line-basic
Attempt: 1/3

Implementation

  • plots/plotly/scatter/line-basic/default.py

Changes

  • Uses go.Scatter with mode="lines" for line plot
  • Python Blue (#306998) color as per style guide
  • 4800x2700px output (1600x900 scaled 3x)
  • Clean white template with subtle grid
  • No legend for single series (per quality criteria)
  • Appropriate font sizes for high resolution output

- Remove unnecessary pandas import (use simple lists)
- Disable legend for single series (per quality criteria)
- Increase font sizes for better readability at 4800x2700px
- Increase line width from 2 to 3 for better visibility
- Adjust margins for proper layout
@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 5001a77 into plot/line-basic Dec 7, 2025
8 checks passed
@github-actions github-actions Bot deleted the auto/line-basic/plotly branch December 7, 2025 20:57
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