Skip to content

Miscellaneous fixes to the unified compiler #2274

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mudit2812 merged 7 commits into from
Dec 9, 2025
Merged

Miscellaneous fixes to the unified compiler #2274

mudit2812 merged 7 commits into from
Dec 9, 2025

Conversation

@mudit2812
Copy link
Contributor

@mudit2812 mudit2812 commented Dec 8, 2025 •
edited
Loading

  • Update MLIR graph tests so that they are skipped if at least one of the following conditions is not met:
    • The Python graphviz package is not installed
    • The system-level graphviz package is not installed
  • Update a test for the xDSL Quantum dialect that was causing the --force-flaky workflow to fail
  • Update version of PennyLane in .dep-versions
  • Move changelog entry for qml.specs integration with xDSL to the correct section

mudit2812
mudit2812 commented Dec 8, 2025
View reviewed changes
@mudit2812 mudit2812 requested a review from paul0403 December 8, 2025 16:03
paul0403
paul0403 reviewed Dec 8, 2025
View reviewed changes
Copy link
Member

@paul0403 paul0403 left a comment

Choose a reason for hiding this comment

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

Looks good, you have my approval pending resolution of the which issue from Rashid 🥳

Also the CI failures are not related to your changes, something went wrong with nanobind this morning. The failures should go away now if you push another commit.

@paul0403 paul0403 added the unified compiler Pull requests for the integration with xDSL label Dec 8, 2025
@mudit2812 mudit2812 merged commit af3e13b into main Dec 9, 2025
44 checks passed
@mudit2812 mudit2812 deleted the xdsl-misc-fixes branch December 9, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@paul0403 paul0403 paul0403 approved these changes

@rashidnhm rashidnhm rashidnhm approved these changes

@dime10 dime10 Awaiting requested review from dime10

Assignees

No one assigned

Labels

unified compiler Pull requests for the integration with xDSL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mudit2812 @dime10 @paul0403 @rashidnhm