Skip to content

Cherry-pick: Add static auto-layout graph visualization (#8) - #9

Merged
frlai merged 2 commits into
developfrom
frlai/cherry-pick-pr8-graph-viz
Jul 28, 2026
Merged

Cherry-pick: Add static auto-layout graph visualization (#8)#9
frlai merged 2 commits into
developfrom
frlai/cherry-pick-pr8-graph-viz

Conversation

@frlai

@frlai frlai commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Cherry-picks the squash merge from Add static auto-layout graph visualization #8 (53169de) onto develop — that PR was incorrectly merged into main.
  • Resolves conflicts with develop-only changes:
    • Keep compile_graph(..., graph_configs=None) signature and behavior
    • Keep develop's onnx Python comment in pyproject.toml
    • Merge getting-started docs: 3.10+ / visualization note from Add static auto-layout graph visualization #8, retain :doc:/index`` dependency pointer

Follow-up

  • After this lands, revert 53169de on main.

Test plan

  • CI green on this PR
  • Spot-check leapp.compile_graph(visualize=True) produces PNG on Python 3.11+
  • Confirm graph_configs= still works on develop

lgulich and others added 2 commits July 27, 2026 16:08
* Add static graph visualization auto-layout

* Document leapp-visualization package split

* Plan leapp-visualization package split

* Split graph visualization into leapp-visualization package

* Address graph visualization review feedback

* Gate visualization on Python 3.11

* Remove SVG rendering from graph visualization

Raster and vector output were two independent rendering pipelines that had
to be kept visually equivalent by hand. Per PR review feedback, drop the SVG
renderer and keep PNG as the single graph visualization artifact.

* Install leapp-visualization from source instead of an index
@frlai
frlai merged commit 66bfc85 into develop Jul 28, 2026
4 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.

2 participants