Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CircuitOperation.qubit_map allows mapping qubits of different dimension #3982

Closed
daxfohl opened this issue Apr 1, 2021 · 0 comments · Fixed by #3983
Closed

CircuitOperation.qubit_map allows mapping qubits of different dimension #3982

daxfohl opened this issue Apr 1, 2021 · 0 comments · Fixed by #3983
Labels
kind/bug-report Something doesn't seem to work.

Comments

@daxfohl
Copy link
Contributor

daxfohl commented Apr 1, 2021

Description of the issue

If you do CircuitOperation(..., qubit_map={LineQubit(0), LineQid(1, dimension=3)}), it is happy but then fails during simulation with obscure message.

How to reproduce the issue

See above

Expected behavior

It should fail at circuit construction time with a friendly error message.

Notes

The check needs to happen in with_qubit_mapping as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug-report Something doesn't seem to work.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Disallow qubit_map with conflicting dimensions in subcircuits daxfohl/Cirq
1 participant
@daxfohl