qiskit-tutorials icon indicating copy to clipboard operation
qiskit-tutorials copied to clipboard

Made changes to improve operators tutorial

Open Guillermo-Mijares-Vilarino opened this issue 3 years ago • 3 comments

Summary

Made some changes to the Summary of Quantum Operations tutorial to make it follow more closely our documentation checklist and guidelines.

Details and comments

  • Moved imports to where they are strictly needed so the users can better understand what imports are needed for each step.
  • Corrected the controlled-$u$ matrix form.
  • Added matrices for most significant bit controlled Hadamard, Rz, U, Toffoli and Fredkin gates for more Physics-oriented users and consistency with most sources.
  • Explained what phase and Pauli gates meant in terms of rotation on the Bloch sphere.
  • Mentioned where each simulator will be used in the introduction.
  • Slightly changed the introduction so it's more focused on the user (using "you will do x" instead of "we will show you x")
  • Added comments to code.
  • Added a brief conclusion about what the user has achieved with this tutorial.
  • Run the notebook.

Fixes Qiskit/qiskit#1522

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jul 06 '22 08:07 CLAassistant

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jul 06 '22 08:07 CLAassistant

This is superseded by

  1. https://github.com/Qiskit/qiskit-terra/pull/8370
  2. https://github.com/Qiskit/qiskit-terra/pull/8391
  3. https://github.com/Qiskit/qiskit-terra/pull/8443

HuangJunye avatar May 09 '23 11:05 HuangJunye