Skip to content

Add overlay timeseries plots (ncview style)#28

Merged
koldunovn merged 2 commits intomainfrom
feature/overlay-timeseries
Apr 8, 2026
Merged

Add overlay timeseries plots (ncview style)#28
koldunovn merged 2 commits intomainfrom
feature/overlay-timeseries

Conversation

@koldunovn
Copy link
Copy Markdown
Collaborator

@koldunovn koldunovn commented Apr 8, 2026

Summary

  • Clicking multiple points now overlays their time series with different colors instead of replacing the previous plot (ncview style)
  • 8-trace ring buffer with distinct colors: blue, red, green, orange, purple, cyan, magenta, brown
  • Y-axis auto-scales across all visible traces; each trace plots against its own time values
  • Legend appears in top-right corner when multiple traces are shown
  • Right-click on image clears all traces
  • Closing the popup preserves traces (reopen to see them again)

Addresses item 2 from #24.

Test plan

  • Click on a point — single trace appears in blue
  • Click on a second point — both traces visible with different colors, Y-axis rescaled
  • Click up to 8 points — all visible with distinct colors and legend
  • Click a 9th point — oldest trace drops off
  • Right-click on the image — all traces cleared
  • Close popup, click new point, reopen — fresh trace
  • Verify colors are distinguishable on both light and dark themes

Clicking on multiple points now overlays their time series with
different colors (ncview style) instead of replacing the previous plot.

- 8-trace ring buffer: blue, red, green, orange, purple, cyan,
  magenta, brown. When full, oldest trace is dropped.
- Y-axis auto-scales across all traces; each trace uses its own
  time values for X positioning.
- Legend in top-right corner when multiple traces are visible.
- Title shows trace count: "Time Series (N traces)".
- Right-click on image clears all traces.
- Closing the popup preserves traces (reopen to see them again).
Matches the user's choice: both right-click and closing the popup
reset all traces, so reopening starts fresh.
@koldunovn koldunovn merged commit 981929a into main Apr 8, 2026
5 checks passed
@koldunovn koldunovn mentioned this pull request Apr 8, 2026
4 tasks
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