Skip to content

Refactor Graphviz graph generation to use shared graph functions#2557

Merged
ipspace merged 1 commit intodevfrom
graphviz_refactor
Jul 29, 2025
Merged

Refactor Graphviz graph generation to use shared graph functions#2557
ipspace merged 1 commit intodevfrom
graphviz_refactor

Conversation

@ipspace
Copy link
Owner

@ipspace ipspace commented Jul 27, 2025

* Implements ideas from #2535 and #2547 for Graphviz graphs
* Adds generic 'object style' mechanism that replaces old 'colors'
  and 'margins' settings
* Corrects #2534 for Graphviz
Copy link
Collaborator

@DanPartelly DanPartelly left a comment

Choose a reason for hiding this comment

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

This looks good to me. Tried it with several topologies, and the results are identical to d2 graphs (save for final layout, but that's dependent on layout engines). Thank you.

@ipspace ipspace merged commit 7b56bef into dev Jul 29, 2025
11 checks passed
@ipspace ipspace deleted the graphviz_refactor branch July 29, 2025 16:25
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