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.

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

mapping from Euler angles to quaternions in mapper #607

Closed
jaygambetta opened this issue Jun 28, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@jaygambetta
Copy link
Member

commented Jun 28, 2018

What is the expected enhancement?

To fix some of the mapper i going to change the 1q optimization from euler angles to quaternions and it should fix the mapper issues that keep getting brought up.

This is a low priority as most times it works but the correct way to do it is quaternions.

@ajavadia

This comment has been minimized.

Copy link
Member

commented Jun 28, 2018

Here are 2 tiny tests that trigger this error.

qasm_string_1= """OPENQASM 2.0;
include "qelib1.inc";
qreg qr[2];
cx qr[0],qr[1];
rz(0.7) qr[1];
rx(1.570796) qr[1];
"""
qasm_string_2 = """OPENQASM 2.0;
include "qelib1.inc";
qreg qr[2];
y qr[0];
h qr[0];
s qr[0];
h qr[0];
"""

In both cases, the error occurs during compilation (specifically during yzy_to_zyz). To trigger it:

from qiskit import *
import Qconfig
from pprint import pprint
register(Qconfig.APItoken)
circ = load_qasm_string(qasm_string_1)
qobj = compile(circ, 'ibmqx4')
pprint(qobj)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.