Discussions of the Parallel Programming book
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH] future/QC: Remove 'Readout' in header of 'entanglement error rate' row
Date: Sun, 25 Feb 2018 15:01:18 -0800	[thread overview]
Message-ID: <20180225230118.GH2855@linux.vnet.ibm.com> (raw)
In-Reply-To: <b8083f74-2290-0280-de67-cdcd69083502@gmail.com>

On Mon, Feb 26, 2018 at 07:30:08AM +0900, Akira Yokosawa wrote:
> >From 4d1964b2296525414181b8e9b8fc7a77297fa838 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Mon, 26 Feb 2018 07:10:27 +0900
> Subject: [PATCH] future/QC: Remove 'Readout' in header of 'entanglement error rate' row
> 
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> ---
> Paul,
> 
> Looking at the retouched table once again, I find
> "readout of entanglement gate" doesn't make sense. The "Readout"
> in the header should be removed, shouldn't it?

It is quite confusing, now that you mention it!

I took a slightly different approach with your Reported-by, so please
take a look and tell me what you think.  I considered adding a few
qubit operators to the "Qubit Operator" heading like I did for "Qubit
Entanglement", but wasn't sure whether this would help.  Thoughts?

							Thanx, Paul

>         Thanks, Akra
> --
>  future/QC.tex | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/future/QC.tex b/future/QC.tex
> index 1afae2c..a5b1758 100644
> --- a/future/QC.tex
> +++ b/future/QC.tex
> @@ -677,26 +677,26 @@ $10^{-4}$).
>  }
> 
>  \subfloat[Multi-Qubit Gate Properties (Entanglement)]{
> -\begin{tabular}{@{}ll*{6}{S[table-format=2.2]}@{}}
> +\begin{tabular}{@{}l*{6}{S[table-format=2.2]}@{}}
>  	\toprule
> -	& & \multicolumn{6}{c}{Entanglement Connection} \\
> -	\cmidrule{3-8}
> -	\multicolumn{2}{c}{Gate Property} &
> +	 & \multicolumn{6}{c}{Entanglement Connection} \\
> +	\cmidrule{2-7}
> +	Gate Property &
>  		\multicolumn{1}{c}{CX0\_1} &
>  			\multicolumn{1}{c}{CX1\_2} &
>  				\multicolumn{1}{c}{CX0\_2} &
>  					\multicolumn{1}{c}{CX3\_2} &
>  						\multicolumn{1}{c}{CX4\_2} &
>  							\multicolumn{1}{c@{}}{CX3\_4} \\
> -	\cmidrule(r){1-2} \cmidrule{3-8}
> -	Readout & Error Rate (\%) &
> +	\cmidrule(r){1-1} \cmidrule{2-7}
> +	Error Rate (\%) &
>  		 3.46 &
>  			 4.07 &
>  				 3.26 &
>  					 2.76 &
>  						 2.23 &
>  							 2.66 \\
> -	& Fidelity (\%) &
> +	Fidelity (\%) &
>  		 96.54 &
>  			95.93 &
>  				96.74 &
> -- 
> 2.7.4
> 
> 


  reply	other threads:[~2018-02-25 23:00 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-24 23:47 [PATCH 0/5] future/QC: Improve table layout Akira Yokosawa
2018-02-24 23:49 ` [PATCH 1/5] future/QC: Improve layout of Table 17.5 by using subfloat Akira Yokosawa
2018-02-24 23:51 ` [PATCH 2/5] future/QC: Remove alternate-row coloring in Table 17.5 Akira Yokosawa
2018-02-24 23:52 ` [PATCH 3/5] future/QC: Add narrow space in front of percent symbol Akira Yokosawa
2018-02-24 23:52 ` [PATCH 4/5] future/QC: Disambiguate 'QC power' Akira Yokosawa
2018-02-24 23:53 ` [PATCH 5/5] future/QC: Fix typo Akira Yokosawa
2018-02-25 18:23 ` [PATCH 0/5] future/QC: Improve table layout Paul E. McKenney
2018-02-25 22:30   ` [PATCH] future/QC: Remove 'Readout' in header of 'entanglement error rate' row Akira Yokosawa
2018-02-25 23:01     ` Paul E. McKenney [this message]
2018-02-25 23:22       ` Akira Yokosawa
     [not found]       ` <ed46ece9-7825-3204-a101-a16982e44af8@gmail.com>
2018-02-26 16:35         ` [PATCH] future/QC: Tweak horizontal spacing of Table 17.5 Paul E. McKenney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180225230118.GH2855@linux.vnet.ibm.com \
    --to=paulmck@linux.vnet.ibm.com \
    --cc=akiyks@gmail.com \
    --cc=perfbook@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox