Conversation
|
@natestemen Greetings, Hope you're doing well. Kindly have a look to see if it's good to go on the Quick question, I couldn't add |
natestemen
left a comment
There was a problem hiding this comment.
Thanks for getting this started! The first thing to do would be to add CUDAQ to the supported frontends:
Line 76 in 899e07a
Once it's in there, it will be automatically tested via lots of the other tests. A few small, individual tests would also be good to see though. Just small circuit conversions.
| # We need to transpile as `openqasm3_to_cudaq` cannot work | ||
| # with arbitrary gates, i.e., ccx, rxx, etc. |
There was a problem hiding this comment.
It specifically only works with u3 and cx?
There was a problem hiding this comment.
If you mean qbraid's conversion, no, only single qubit gates and single controlled single qubit gates. Issue is that cudaq doesn't have ccx or mcx. They just do cx([0, 1, 2], [3]) for instance for mcx(replace indices there with qr[i] of course for correct syntax).
So, I thought it'd be better to just use u3 and cx when converting to cudaq side. It keeps maintenance easier, and would work for all custom gates.
Copy that.
Sure thing, I'll see if the tests pass for it. |
|
Do I need to add the cudaq install to anywhere else besides pyproject.toml? |
|
Sorry by the way for pushing this without making sure all tests pass. I'm having some trouble getting the test CI to run fully (keeps getting stuck at the end). I'm gonna test module by module for now. |
|
Not a problem for continuing to test things out using GitHub actions here! Since cudaq doesn't distribute wheels for non-linux OS's (NVIDIA/cuda-quantum#2065 (comment)), it might be worth considering how to run the tests only on linux builds. This is certainly possible, but we'll want to consider the tradeoff of complexity if we have to add lots of hacks to make it work. Typically for an isolated feature I might just say throw the tests in a new file and run those on the linux builds, but since this feature modifies |
Sure, I'll update the workflow minimally to enable this with cudaq. |
|
My ongoing notes on remaining tests that are failing. I fixed most, but these remain at the moment. |
|
It's not installing |
|
I'll circle back to this soon. |
Description
Added initial commit for tackling #2200 .
License
Before opening the PR, please ensure you have completed the following where appropriate.