-
Notifications
You must be signed in to change notification settings - Fork 86
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.
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
Support setp and delay pulses #355
Comments
To be able to run a protocol similar to the one described in https://arxiv.org/pdf/2008.04302.pdf,
These instructions allow changing LO freq. per pulse (rather than fixed for the whole experiment). Note: we may not be able to simulate these two instructions since Aer doesn't support them (see this) |
Topics about pulses are interesting to me. Is this issue still relevant? |
Yea, this issue is still relevant. Please feel free to work on this if you're interested. BTW, we can get around this issue by pre-modulating the pulse signal in the code, but it'd be nice to have these pulse instructions supported in XACC. |
I started implementation (a draft pull request #464 ) and have a few questions:
|
Hi @flatslm,
|
Hi @tnguyen-ornl |
Thanks, @flatslm. |
Hi @tnguyen-ornl. |
IBM has introduced two new native pulses
setp
(set phase) anddelay
(in addition tofc
andacquire
).These two instructions can be used inside pulse cmd-def JSON. We need to update
IBMAccelerator::contributeInstructions
to contribute these new instructions so that the pulse cmd-def Composite can be constructed.The text was updated successfully, but these errors were encountered: