-
Notifications
You must be signed in to change notification settings - Fork 18
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
The latest pennylane-cirq requires simulate
method without analytic
argument.
#69
Comments
Hi @jaeyoo! The latest version of PennyLane replaces the anlaytic keyword argument with |
Thank you @josh146 , I realized that the issue was caused by For the case of cirq.sim.SimulatesExpectationValues.simulate_expectation_values() and it seems to accept |
Glad it worked!
I don't believe so, no; I think this is an artifact of the core of this plugin being written against an older Cirq version. If there are better ways of accessing expectation values, we should mark the plugin for an update to access these more efficient methods. |
Thank you Josh, yes, I hope the latest PennyLane-Cirq supports the latest Cirq too. Could you please add this feature into it? |
@jaeyoo I've created a new issue here (#70), so will close this one for now 🙂 Feel free to comment on the other one in case I have missed any details!
Just double checking, is anything broken using PennyLane with the latest Cirq version? Or is it more that there are newer features available? |
On 0.14.0, there was
analytic
argument inqml.device
, so we could set ifFalse
and then addCirq
-based simulators. However, the latest version has noanalytic
, and it always requiressimulate
method.In details, when we run the colab here, if we change the pennylane & pennylane-cirq versions to the latest, then it says:
The text was updated successfully, but these errors were encountered: