Skip to content
This repository was archived by the owner on Aug 21, 2023. It is now read-only.

Conversation

@ShellyGarion
Copy link
Member

Summary

  • Extended RB overview to contain more explanation on how RB works and simultaneous RB
  • Minor changes in Clifford_Group.ipnyb (added logo)

@dcmckayibm - please review

Details and comments

"source": [
"### The intuition behind RB\n",
"\n",
"The depolarizing channel has parameter $p$, and works like this: with probability $p$, the state remains the same as before; with probability $1-p$, the state becomes the totally mixed state.\n",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you look at our papers we use $\alpha$ for $p$ here. Also you have $1-p^m$ should be $(1-p)^m$. The last equation didn't render.

@dcmckayibm
Copy link
Member

I think the intuition paragraph would be better if you just wrote out the quantum channel

$\rho_f = \alpha \rho_i + (1-\alpha)/2^n * \mathbf{I} $

$\rho_f^m = \alpha^m \rho_i + (1-\alpha)^m/2^n * \mathbf{I} $

Then it's clear that the ground state survival probability just follows.

@ShellyGarion
Copy link
Member Author

@dcmckayibm - I updated the overview following your comments above.

@dcmckayibm
Copy link
Member

Introduction is spelled wrong

@dcmckayibm
Copy link
Member

This sentence is a little confusing "Suppose that we have a sequence of $m$ gates, not necessarily Clifford gates. Each gate is affected by the depolarizing channel with parameter $\alpha$ (same $\alpha$ for all the gates)."

-> I would say that suppose we have gates where the error channel of the gate is a depolarizing channel.
-> I was also wrong it's not (1-alpha)^m it's (1-alpha^m)...then the last equation is exact

@ShellyGarion
Copy link
Member Author

@dcmckayibm - I updated the overview again following your comments above.

@dcmckayibm dcmckayibm self-requested a review March 14, 2019 13:49
Copy link
Member

@attp attp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving since @dcmckayibm approved.

@attp attp merged commit 55b46f2 into Qiskit:master Mar 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants