-
Notifications
You must be signed in to change notification settings - Fork 586
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
Add DefaultPulse device #3638
Add DefaultPulse device #3638
Conversation
Hello. You may have forgotten to update the changelog!
|
[sc-31947] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this @lillian542 💯
dim (int): dimensions of the system (2 for qubits, 3 for qutrits, etc) | ||
drift(Operator): optional Hamiltonian term representing a constant drift term for the system Hamiltonian |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we perhaps omit these terms for now, until we're ready to use them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be also nice to raise a warning when drift is not None, saying that this currently does nothing
[sc-31947] |
Just fyi, @trbromley and I discussed today that for the v0.29.0 we would actually not promote using |
will use default.qubit.jax instead |
Set up dummy device (currently default.qubit with addition of
dt
,dim
anddrift
attributes) as first step in a device to support pulse programming