Skip to content

📝 Use PyGraphviz for documentation diagrams - #1012

Merged
denialhaag merged 5 commits into
mainfrom
codex/docs-pygraphviz
Sep 17, 2026
Merged

denialhaag merged 5 commits into
mainfrom
codex/docs-pygraphviz

Conversation

@denialhaag

@denialhaag denialhaag commented Sep 16, 2026

Copy link
Copy Markdown
Member

Description

🤖 AI text below 🤖

Render documentation diagrams with PyGraphviz 2 or later, whose wheels bundle Graphviz, and remove the system Graphviz package from Read the Docs.

Use temporary to_svg(dd, ...) helpers to keep SVG export calls to one line until the minimum MQT Core version includes PyGraphviz-backed SVG export. Their removal is tracked in #1013.

AI notice

This PR and its contents were created with the assistance of GPT-6 Astra via Codex.

Checklist

  • The pull request only contains commits that are focused and relevant to this change.
  • I have added appropriate tests that cover the new/changed functionality.
  • I have updated the documentation to reflect these changes.
  • I have added entries to the changelog for any noteworthy additions, changes, fixes, or removals.
  • I have added migration instructions to the upgrade guide (if needed).
  • The changes follow the project's style guidelines and introduce no new warnings.
  • The changes are fully tested and pass the CI checks.
  • I have reviewed my own code changes.

If PR contains AI-assisted content:

  • Any agent that created, edited, or submitted GitHub content was explicitly authorized for that scope, as required by our AI Usage Guidelines.
  • Every agent-authored or agent-edited public text body begins with the visible disclosure 🤖 *AI text below* 🤖 (titles are exempt).
  • I have disclosed AI assistance in the PR description.
  • I confirm that I have personally reviewed and understood all AI-generated content, and accept full responsibility for it.

Assisted-by: GPT-6 Astra via Codex
@denialhaag denialhaag self-assigned this Sep 16, 2026
@github-project-automation github-project-automation Bot moved this to In Progress in MQT Simulation Sep 16, 2026
@denialhaag denialhaag added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels Sep 16, 2026
@denialhaag
denialhaag marked this pull request as ready for review September 16, 2026 12:25

@burgholzer burgholzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just an idea 😌

Comment thread docs/simulators/CircuitSimulator.md Outdated
Assisted-by: GPT-6 Astra via Codex
Assisted-by: GPT-6 Astra via Codex
Assisted-by: GPT-6 Astra via Codex
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
python/mqt/ddsim/utils.py 0.0% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

Assisted-by: GPT-6 Astra via Codex

@burgholzer burgholzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 👍🏻

@denialhaag
denialhaag merged commit 806a015 into main Sep 17, 2026
19 of 20 checks passed
@denialhaag
denialhaag deleted the codex/docs-pygraphviz branch September 17, 2026 14:43
@github-project-automation github-project-automation Bot moved this from In Progress to Done in MQT Simulation Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants