Skip to content

test_state_preparation_via_coherent_alias_sampling in state_preparation_test.py is sensitive to decomposition changes #6197

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

Closed
NoureldinYosri opened this issue Jul 14, 2023 · 0 comments · Fixed by #6204
Labels
area/cirq-ft Issues related to the Cirq-FT sub-package kind/bug-report Something doesn't seem to work.

Comments

@NoureldinYosri
Copy link
Collaborator

Description of the issue
The test test_state_preparation_via_coherent_alias_sampling in state_preparation_test.py is sensitive to qubit counts. the circuit it builds has ~25 qubits, more than this and the test fails due to OOM.

The test doesn't use the hight level circuit it builds but rather its decomposition. so every time we add a new decomposition that uses ancilla we increase the qubit count of the circuit in the test which the test fail.

How to reproduce the issue
multiple ways.
Either remove any of the workarounds we are using to reduce the qubit count or reduce the epsilon parameter of the test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cirq-ft Issues related to the Cirq-FT sub-package kind/bug-report Something doesn't seem to work.
Projects
None yet
1 participant