Skip to content

Add legend positioning to graph and LeNet views - #186

Merged
Tariolle merged 1 commit into
willyfh:mainfrom
mgalore:feat/issue-185-legend-position
Jul 19, 2026
Merged

Add legend positioning to graph and LeNet views#186
Tariolle merged 1 commit into
willyfh:mainfrom
mgalore:feat/issue-185-legend-position

Conversation

@mgalore

@mgalore mgalore commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add the existing six-value legend_position API to graph and LeNet static and animated views
  • share legend validation and canvas placement with the flow view so all three styles stay consistent
  • expose the option through render(), animation, and the MCP capabilities manifest
  • add graph and LeNet gallery examples plus regression coverage for defaults, invalid values, placement, and forwarding

Verification

  • PATH="$PWD/.venv/bin:$PATH" .venv/bin/pytest -q — 311 passed
  • .venv/bin/pre-commit run --all-files — all hooks passed
  • MPLBACKEND=Agg .venv/bin/python docs/examples/graph/plot_legend_position_graph.py
  • MPLBACKEND=Agg .venv/bin/python docs/examples/lenet_style/plot_legend_position_lenet_style.py
  • git diff --check

Closes #185

@Tariolle Tariolle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks clean to me

@Tariolle
Tariolle merged commit 4cf7a30 into willyfh:main Jul 19, 2026
9 checks passed
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.

Add legend_position support to graph_view and lenet_view

2 participants