-
Notifications
You must be signed in to change notification settings - Fork 365
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
EstimatorV2 extended_stabilizer fails" invalid instructions {"instructions": {save_expval}} #2121
Comments
So strange. When I run the code in @zlatko-minev 's repro, I get a different error. I'm using the same Aer version and Python 0.14.1, but my OS is Fedora Linux 40.
|
I found qiskit-aer/src/simulators/extended_stabilizer/extended_stabilizer_state.hpp Lines 401 to 405 in 83d679e
|
This is fixed by PR #2120 |
Thank you Sent from my iPhoneOn May 27, 2024, at 5:25 AM, Jun Doi ***@***.***> wrote:
So strange. When I run the code in @zlatko-minev 's repro, I get a different error. I'm using the same Aer version and Python 0.14.1, but my OS is Fedora Linux 40.
Traceback (most recent call last):
File "/home/kevinsung/projects/documentation/test.py", line 13, in <module>
estimator = EstimatorV2(options=options)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/kevinsung/.pyenv/versions/documentation/lib/python3.11/site-packages/qiskit_aer/primitives/estimator_v2.py", line 74, in __init__
self._backend = AerSimulator(method=method, **self.options.backend_options)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: qiskit_aer.backends.aer_simulator.AerSimulator() got multiple values for keyword argument 'method'
This is fixed by PR #2120
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
I'm feeling difficulty to solve expval bug in extended stabilizer, so I think I will not include fix in Aer 014.2 |
Thank you for trying. Can you add a little more on what the issue is? That's tricky to resolve. I hope somebody else here might be able to contribute as well. The thing is that we're increasingly moving to using the estimator class and so this is now the default preferred pathway. |
I opened PR #2161 for this issue. |
Thanks! What do you mean by accuracy for non Clifford’s? Just the number of t gates and cost associated in memory with that Sent from my iPhoneOn Jun 3, 2024, at 5:57 AM, Jun Doi ***@***.***> wrote:
I opened PR #2161 for this issue.
I'm fixing bugs in save_expval and now it works for clifford circuits
I think it is not easy because of accuracy for non-clifford circuits
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
It seems the estimation is unstable for non clifford circuits that causes wrong expectation values |
You mean as a function of the truncation?And if a user increases the truncation it converges. On Jun 3, 2024, at 10:31 PM, Jun Doi ***@***.***> wrote:
It seems the estimation is unstable for non clifford circuits that causes wrong expectation values
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Informations
What is the current behavior?
Can't run extended stabilizer with Estimator, see code:
Steps to reproduce the problem
Return
What is the expected behavior?
Should run extended stabilizer with Estimator.
Suggested solutions
NA
The text was updated successfully, but these errors were encountered: