Skip to content

refactor: rename circuit drawing module to visualization - #461

Merged
TheGupta2012 merged 2 commits into
qBraid:mainfrom
danielgaskins:refactor/circuit-visualization-module
Oct 23, 2026
Merged

TheGupta2012 merged 2 commits into
qBraid:mainfrom
danielgaskins:refactor/circuit-visualization-module

Conversation

@danielgaskins

Copy link
Copy Markdown
Contributor

Summary of changes

Move circuit drawing from pyqasm.printer to pyqasm.visualization and update its imports. pyqasm.draw keeps the same interface; direct drawing imports now use pyqasm.visualization.

This is the prerequisite requested in #454, leaving printer.py available for the QASM printer. No drawing behavior changes.

All 1,011 tests pass, with three existing skips. Docs, Black, isort, Pylint, mypy and license-header checks pass.

@argus-eye

argus-eye Bot commented Oct 22, 2026

Copy link
Copy Markdown

Argus review

Auto-review is off for this repo. Tick the box below to run a review on this PR.

  • Trigger Argus review

Estimated cost

  • Files changed: 3
  • Diff lines (±): 4
  • Historical avg: ~347.1k tokens · ~$1.28 · across last 15 review(s)

Tip: you can also comment @argus-eye review at any time.

@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: qBraid/pyqasm
  • Branch: main
  • File: .coderabbit.yaml
  • Commit: 62b012e
  • Layers: Repository YAML
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: eb6f7203-9f4d-4dc2-abe2-d15efff7dd47

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@TheGupta2012

TheGupta2012 commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Thanks @danielgaskins for this change, lgtm! Can you please add a CHANGELOG entry as this is a breaking change in terms of the drawing API? Anyone who is using pyqasm's drawing facilities would need to update their API to pyqasm.visualization now.

@TheGupta2012
TheGupta2012 merged commit bf84d93 into qBraid:main Oct 23, 2026
2 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.

3 participants