-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Test circuit drawer uses "π" instead of "pi" #5218
Conversation
@1ucian0 We'd discussed also doing the dagger for sdg, tdg, and sxdg for the text drawer. Something like |
I dont think there is |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Welcome to the community!
Thank you for helping me. Looking forward to continuing helping |
I dont think there is |
This commit fixes the usage of internal only explicitly private attributes of basicaer's job class. These look they were used to manually create a job id which was manually set ahead of time despite still being a random uuid and then launching a simulation from that constructed job object. Which is both the incorrect order to run a simulation (runs generate jobs, not the other way around) but also served no functional purpose. This commit removes this as this is a blocker for Qiskit/qiskit#5218 which is upgrading the basic aer provider to use the latest versioned provider interface which removes all of those private methods and also moves to a synchronous execution model.
* Stop using basic aer internal private attributes This commit fixes the usage of internal only explicitly private attributes of basicaer's job class. These look they were used to manually create a job id which was manually set ahead of time despite still being a random uuid and then launching a simulation from that constructed job object. Which is both the incorrect order to run a simulation (runs generate jobs, not the other way around) but also served no functional purpose. This commit removes this as this is a blocker for Qiskit/qiskit#5218 which is upgrading the basic aer provider to use the latest versioned provider interface which removes all of those private methods and also moves to a synchronous execution model. * Fix lint * Update run_circuits.py Fix copyright Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Summary
Fixed #4959
Details and comments
Change pi_check function when output is "text" from "pi" to "π". Also, the test cases were updated to check different circuits with the new symbol "π". Different encoders were tested to see if they accepted "π" and gave good results.