Skip to content

Conversation

@josephleekl
Copy link
Contributor

This PR enables QIR-EE to use any Pennylane Lightning simulator backend (more info here).

Instruction for building the main app is included in src/qirlightning/README.md. A simple demo is also included within src/qirlightning/simple_demo.

Please feel free to play with it and ask any questions, happy to answer them!

wongey and others added 30 commits November 5, 2024 21:10
which was accidentally copied into the top level
Remark: Compares only empty output for now until we find a way to properly store the result and not just print as we go.

Temporarily using lowercase qsim. Will modify class names later with the others at the same time.
This commit will fail tests since I am documenting the file name change separately.
bell_ccx.ll to examples folder and dynamicbv.ll to test data folder
@sethrj sethrj mentioned this pull request Jun 23, 2025
@wongey
Copy link
Collaborator

wongey commented Jun 23, 2025

Hi @josephleekl -- we are working on the C-API implementation and enabled a single runtime interface, which should make passing runtime/quantum interfaces easier so that one can run an arbitrary number of shots. See #37 for diff. Kindly merge the 'main' branch in and update the qir-lightning app and tests accordingly! Thank you.

@josephleekl
Copy link
Contributor Author

Hi @josephleekl -- we are working on the C-API implementation and enabled a single runtime interface, which should make passing runtime/quantum interfaces easier so that one can run an arbitrary number of shots. See #37 for diff. Kindly merge the 'main' branch in and update the qir-lightning app and tests accordingly! Thank you.

Thanks for the heads up @wongey , I've now merged the main branch and updated our runtime/app similar to qsim. It should work now, do let me know if there are any issues!

@wongey
Copy link
Collaborator

wongey commented Jul 21, 2025

Hi @josephleekl -- thanks for pulling the main branch in! Can you activate the qir-lightning tests in the CI? Heads up that they are not currently passing locally.

Copy link
Member

@sethrj sethrj left a comment

Choose a reason for hiding this comment

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

Working through this; there are some infrastructure/software engineering issues that to be addressed :)

@josephleekl
Copy link
Contributor Author

Thanks @sethrj for your comments! I've now made some updates following your suggestions.

@josephleekl josephleekl requested a review from sethrj October 22, 2025 21:48
@sethrj
Copy link
Member

sethrj commented Oct 23, 2025

I love that we're one black cat reviewing another 😹

Copy link
Member

@sethrj sethrj 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 really good! Thanks for adding the capability and waiting so long for my reviews 😅

@wongey wongey enabled auto-merge (squash) November 14, 2025 12:15
@wongey wongey merged commit 7827409 into ORNL-QCI:main Dec 2, 2025
6 of 11 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.

4 participants