Discussions of the Parallel Programming book
 help / color / mirror / Atom feed
* [PATCH 0/5] future/QC: Improve table layout
@ 2018-02-24 23:47 Akira Yokosawa
  2018-02-24 23:49 ` [PATCH 1/5] future/QC: Improve layout of Table 17.5 by using subfloat Akira Yokosawa
                   ` (5 more replies)
  0 siblings, 6 replies; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-24 23:47 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

From 66f9f9f82d4fef12b6d7e2a399c14fbaa758e3f8 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sun, 25 Feb 2018 08:09:50 +0900
Subject: [PATCH 0/5] future/QC: Improve table layout

Hi Paul,

This patch set is a response to your recent updates in QC section.

Patch #1 retouches Table 17.5

Patch #2 removes alternate-row coloring in Table 17.5, which looks
redundant after the retouch.

Patch #3 is for consistency of spacing around unit symbols as is
suggested in Style Guide.

Patches #4 and #5 are a word choice suggestion and a typo fix.

Patch #2 can be skipped if you'd like to keep the background color.

        Thanks, Akira
--
Akira Yokosawa (5):
  future/QC: Improve layout of Table 17.5 by using subfloat
  future/QC: Remove alternate-row coloring in Table 17.5
  future/QC: Add narrow space in front of percent symbol
  future/QC: Disambiguate 'QC power'
  future/QC: Fix typo

 future/QC.tex | 137 +++++++++++++++++++++++++++++++---------------------------
 perfbook.tex  |   1 +
 2 files changed, 75 insertions(+), 63 deletions(-)

-- 
2.7.4


^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH 1/5] future/QC: Improve layout of Table 17.5 by using subfloat
  2018-02-24 23:47 [PATCH 0/5] future/QC: Improve table layout Akira Yokosawa
@ 2018-02-24 23:49 ` Akira Yokosawa
  2018-02-24 23:51 ` [PATCH 2/5] future/QC: Remove alternate-row coloring in Table 17.5 Akira Yokosawa
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-24 23:49 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

From e3688c8cdf8e8e34d36012a3a3463ce9e8825ef4 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sat, 24 Feb 2018 17:39:08 +0900
Subject: [PATCH 1/5] future/QC: Improve layout of Table 17.5 by using subfloat

This enables us to offload part of row headers to captions of
sub-tables.

In perfbook.tex, font size of sub captions is increased to "small"
from the default of "footnotesize".

Also remove "%" in each cell and indicate the unit in row headers.
By this change, the "S" column-layout specifier of siunitx package
can be used to align decimal points.

Also add headers to indicate what "Qn" and "CXm_n" mean.

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 future/QC.tex | 125 ++++++++++++++++++++++++++++++++--------------------------
 perfbook.tex  |   1 +
 2 files changed, 70 insertions(+), 56 deletions(-)

diff --git a/future/QC.tex b/future/QC.tex
index df464a4..6491985 100644
--- a/future/QC.tex
+++ b/future/QC.tex
@@ -635,66 +635,79 @@ $10^{-4}$).
 \rowcolors{1}{}{lightgray}
 \renewcommand*{\arraystretch}{1.1}
 \small
-\centering\OneColumnHSpace{-0.1in}
-\begin{tabular}{lrrrrr}
+\centering
+\subfloat[Single-Qubit Gate Properties]{
+\begin{tabular}{ll*{5}{S[table-format=2.3]}}
 	\toprule
-	& Q\textsubscript{0} &
-		Q\textsubscript{1} &
-			Q\textsubscript{2} &
-				Q\textsubscript{3} &
-					Q\textsubscript{4} \\
-	% \cmidrule(l){2-4} \cmidrule(l){5-7}
-	% \cmidrule{1-1} \cmidrule(l){2-4} \cmidrule(l){5-7}
-	Gate Per-Operation Error Rate &
-	   0.197\% &
-	  	 0.129\% &
-			 0.197\% &
-				 0.163\% &
-					 0.094\% \\
-	Gate Per-Operation Fidelity &
-	  99.803\% &
-	  	99.871\% &
-			99.803\% &
-				99.837\% &
-					99.906\% \\
-	Gate Readout Error Rate &
-	   4.50\% &
-	  	 3.60\% &
-			 2.00\% &
-				 1.60\% &
-					 2.50\% \\
-	Gate Readout Fidelity &
-	  95.50\% &
-	  	96.40\% &
-			98.00\% &
-				98.40\% &
-					97.50\% \\
-	& & & & & \\
+	& & \multicolumn{5}{c}{Qubit} \\
+	\cmidrule{3-7}
+	\multicolumn{2}{c}{Gate Property} &
+	\multicolumn{1}{c}{Q\textsubscript{0}} &
+		\multicolumn{1}{c}{Q\textsubscript{1}} &
+			\multicolumn{1}{c}{Q\textsubscript{2}} &
+				\multicolumn{1}{c}{Q\textsubscript{3}} &
+					\multicolumn{1}{c}{Q\textsubscript{4}} \\
+	\cmidrule(r){1-2} \cmidrule{3-7}
+	\cellcolor{white}Per-Operation & \cellcolor{white}Error Rate (\%) &
+	 0.197 &
+		 0.129 &
+			 0.197 &
+				 0.163 &
+					 0.094 \\
+	& Fidelity (\%) &
+	99.803 &
+		99.871 &
+			99.803 &
+				99.837 &
+					99.906 \\
+	\cmidrule(r){1-2} \cmidrule{3-7}
+	\cellcolor{white}Readout & \cellcolor{white}Error Rate (\%) &
+	 4.50 &
+		 3.60 &
+			 2.00 &
+				 1.60 &
+					 2.50 \\
+	& Fidelity (\%) &
+	95.50 &
+		96.40 &
+			98.00 &
+				98.40 &
+					97.50 \\
+	\bottomrule
 \end{tabular}
-\begin{tabular}{lrrrrrr}
-	Multi-Qubit Gate Properties (Entanglement) &
-		CX0\_1 &
-			CX1\_2 &
-				CX0\_2 &
-					CX3\_2 &
-						CX4\_2 &
-							CX3\_4 \\
-	Error Rate &
-		   3.46\% &
-			 4.07\% &
-				 3.26\% &
-					 2.76\% &
-						 2.23\% &
-						 	2.66\% \\
-	Gate Readout Fidelity &
-	  96.54\% &
-	  	95.93\% &
-			96.74\% &
-				97.24\% &
-					97.77\% &
-						97.34\% \\
+}
+
+\rowcolors{2}{}{lightgray}
+\subfloat[Multi-Qubit Gate Properties (Entanglement)]{
+\begin{tabular}{ll*{6}{S[table-format=2.2]}}
+	\toprule
+	& & \multicolumn{6}{c}{Entanglement Connection} \\
+	\cmidrule{3-8}
+	\multicolumn{2}{c}{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}
+	\cellcolor{white}Readout & \cellcolor{white}Error Rate (\%) &
+		 3.46 &
+			 4.07 &
+				 3.26 &
+					 2.76 &
+						 2.23 &
+							 2.66 \\
+	& Fidelity (\%) &
+		 96.54 &
+			95.93 &
+				96.74 &
+					97.24 &
+						97.77 &
+							97.34 \\
 	\bottomrule
 \end{tabular}
+}
 \caption{Error Rates For IBM Five-Qubit Chip}
 \label{tab:future:Error Rates For IBM Five-Qubit Chip}
 \end{table*}
diff --git a/perfbook.tex b/perfbook.tex
index e4bba02..8c2288a 100644
--- a/perfbook.tex
+++ b/perfbook.tex
@@ -28,6 +28,7 @@
 \newsubfloat{listing}
 \captionsetup{labelfont=bf}
 \captionsetup[listing]{font=small,labelsep=colon}
+\captionsetup[subfloat]{font=small}
 % \usepackage{breakurl}
 \usepackage{graphicx}
 \usepackage{rotating}
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 2/5] future/QC: Remove alternate-row coloring in Table 17.5
  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 ` Akira Yokosawa
  2018-02-24 23:52 ` [PATCH 3/5] future/QC: Add narrow space in front of percent symbol Akira Yokosawa
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-24 23:51 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

From 2bbc03f5aef7c67348afed2c9e46627fc9740f5d Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sat, 24 Feb 2018 18:03:21 +0900
Subject: [PATCH 2/5] future/QC: Remove alternate-row coloring in Table 17.5

Alternate-row coloring is redundant with the midrules in the tables.
So remove \rowcolors and \cellcolor macros.
Trimming left and right edges of the tables (by "@{}" column-format
specifier) further improves their looks when background colors are
not used.

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 future/QC.tex | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/future/QC.tex b/future/QC.tex
index 6491985..d88aaac 100644
--- a/future/QC.tex
+++ b/future/QC.tex
@@ -632,12 +632,11 @@ $10^{-4}$).
 \end{figure}

 \begin{table*}[tbh]
-\rowcolors{1}{}{lightgray}
 \renewcommand*{\arraystretch}{1.1}
 \small
 \centering
 \subfloat[Single-Qubit Gate Properties]{
-\begin{tabular}{ll*{5}{S[table-format=2.3]}}
+\begin{tabular}{@{}ll*{5}{S[table-format=2.3]}@{}}
 	\toprule
 	& & \multicolumn{5}{c}{Qubit} \\
 	\cmidrule{3-7}
@@ -646,9 +645,9 @@ $10^{-4}$).
 		\multicolumn{1}{c}{Q\textsubscript{1}} &
 			\multicolumn{1}{c}{Q\textsubscript{2}} &
 				\multicolumn{1}{c}{Q\textsubscript{3}} &
-					\multicolumn{1}{c}{Q\textsubscript{4}} \\
+					\multicolumn{1}{c@{}}{Q\textsubscript{4}} \\
 	\cmidrule(r){1-2} \cmidrule{3-7}
-	\cellcolor{white}Per-Operation & \cellcolor{white}Error Rate (\%) &
+	Per-Operation & Error Rate (\%) &
 	 0.197 &
 		 0.129 &
 			 0.197 &
@@ -661,7 +660,7 @@ $10^{-4}$).
 				99.837 &
 					99.906 \\
 	\cmidrule(r){1-2} \cmidrule{3-7}
-	\cellcolor{white}Readout & \cellcolor{white}Error Rate (\%) &
+	Readout & Error Rate (\%) &
 	 4.50 &
 		 3.60 &
 			 2.00 &
@@ -677,9 +676,8 @@ $10^{-4}$).
 \end{tabular}
 }

-\rowcolors{2}{}{lightgray}
 \subfloat[Multi-Qubit Gate Properties (Entanglement)]{
-\begin{tabular}{ll*{6}{S[table-format=2.2]}}
+\begin{tabular}{@{}ll*{6}{S[table-format=2.2]}@{}}
 	\toprule
 	& & \multicolumn{6}{c}{Entanglement Connection} \\
 	\cmidrule{3-8}
@@ -689,9 +687,9 @@ $10^{-4}$).
 				\multicolumn{1}{c}{CX0\_2} &
 					\multicolumn{1}{c}{CX3\_2} &
 						\multicolumn{1}{c}{CX4\_2} &
-							\multicolumn{1}{c}{CX3\_4} \\
+							\multicolumn{1}{c@{}}{CX3\_4} \\
 	\cmidrule(r){1-2} \cmidrule{3-8}
-	\cellcolor{white}Readout & \cellcolor{white}Error Rate (\%) &
+	Readout & Error Rate (\%) &
 		 3.46 &
 			 4.07 &
 				 3.26 &
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 3/5] future/QC: Add narrow space in front of percent symbol
  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 ` Akira Yokosawa
  2018-02-24 23:52 ` [PATCH 4/5] future/QC: Disambiguate 'QC power' Akira Yokosawa
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-24 23:52 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

From 8241f71e5baa65e4043bdc768b9c5ba5f10c61ae Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sat, 24 Feb 2018 18:29:06 +0900
Subject: [PATCH 3/5] future/QC: Add narrow space in front of percent symbol

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 future/QC.tex | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/future/QC.tex b/future/QC.tex
index d88aaac..cffc6e1 100644
--- a/future/QC.tex
+++ b/future/QC.tex
@@ -731,13 +731,13 @@ CX3\_4),
 are shown in
 Table~\ref{tab:future:Error Rates For IBM Five-Qubit Chip}.
 The gate per-operation fidelity is only about an order of magnitude
-off of the $10^{-4}$ threshold, and some qubits in face exceed 99.9\%
+off of the $10^{-4}$ threshold, and some qubits in face exceed 99.9\,\%
 fidelity (see Section~\ref{sec:future:Programming Model} for a list of
 the operations).
 However, reading out the value of a qubit is considerably more
-error-prone, with fidelities ranging from 95\% to 99\%.
+error-prone, with fidelities ranging from 95\,\% to 99\,\%.
 Furthermore, entangling a pair of qubits is even more error-prone,
-with fidelities ranging from 95\% to 98\%.
+with fidelities ranging from 95\,\% to 98\,\%.
 Although these fidelities are sufficient to execute some QC programs, the
 aforementioned desired error rates of $10^{-4}$ (let alone $10^{-8}$) are a
 long way off, and will likely take considerable work and time to achieve.
@@ -934,7 +934,7 @@ Alternatively, silicon-based qubits offer the prospect of
 operating temperatures that are about an order of magnitude higher than
 transmon-based devices, but these silicon devices currently suffer from
 high entanglement error rates, with fidelities ranging from
-73--89\%~\cite{TFWatson2017SiliconQubit}.
+73--89\,\%~\cite{TFWatson2017SiliconQubit}.
 However, it is early days for silicon devices, so perhaps they will
 continue improving.

-- 
2.7.4



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 4/5] future/QC: Disambiguate 'QC power'
  2018-02-24 23:47 [PATCH 0/5] future/QC: Improve table layout Akira Yokosawa
                   ` (2 preceding siblings ...)
  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 ` 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
  5 siblings, 0 replies; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-24 23:52 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

From 625cce030615870d912d19883ba7b755fa07cab8 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sat, 24 Feb 2018 18:57:43 +0900
Subject: [PATCH 4/5] future/QC: Disambiguate 'QC power'

In QC section, "power" is mostly used to mean "electric power".
So, "increase QC power" can be confusing.

"QC capability" sounds a better choice here.

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 future/QC.tex | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/future/QC.tex b/future/QC.tex
index cffc6e1..74bac86 100644
--- a/future/QC.tex
+++ b/future/QC.tex
@@ -743,7 +743,7 @@ aforementioned desired error rates of $10^{-4}$ (let alone $10^{-8}$) are a
 long way off, and will likely take considerable work and time to achieve.

 In fact, as of early 2018 it is believed that adding qubits will not
-increase QC power unless error rates decrease
+increase QC capability unless error rates decrease
 sharply~\cite{IBMResearch2018QuantumVolume}.

 \subsubsection{Thermodynamics}
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 5/5] future/QC: Fix typo
  2018-02-24 23:47 [PATCH 0/5] future/QC: Improve table layout Akira Yokosawa
                   ` (3 preceding siblings ...)
  2018-02-24 23:52 ` [PATCH 4/5] future/QC: Disambiguate 'QC power' Akira Yokosawa
@ 2018-02-24 23:53 ` Akira Yokosawa
  2018-02-25 18:23 ` [PATCH 0/5] future/QC: Improve table layout Paul E. McKenney
  5 siblings, 0 replies; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-24 23:53 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

From 66f9f9f82d4fef12b6d7e2a399c14fbaa758e3f8 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sun, 25 Feb 2018 00:02:51 +0900
Subject: [PATCH 5/5] future/QC: Fix typo

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 future/QC.tex | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/future/QC.tex b/future/QC.tex
index 74bac86..1afae2c 100644
--- a/future/QC.tex
+++ b/future/QC.tex
@@ -189,7 +189,7 @@ a system with fifty qubits~\cite{WillKnight2017IBM50qubits}, which
 is more than triple the sixteen-qubit system made available only five
 months prior.
 It is therefore quite possible that the current exponential growth
-in numbers of qubits might persist for some time time come.
+in numbers of qubits might persist for some time to come.
 Unfortunately, each qubit must be connected to its own signal generator,
 which means that a million-qubit system would also have not only a million
 signal generators, but also a million wires, each of which would be
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH 0/5] future/QC: Improve table layout
  2018-02-24 23:47 [PATCH 0/5] future/QC: Improve table layout Akira Yokosawa
                   ` (4 preceding siblings ...)
  2018-02-24 23:53 ` [PATCH 5/5] future/QC: Fix typo Akira Yokosawa
@ 2018-02-25 18:23 ` Paul E. McKenney
  2018-02-25 22:30   ` [PATCH] future/QC: Remove 'Readout' in header of 'entanglement error rate' row Akira Yokosawa
  5 siblings, 1 reply; 11+ messages in thread
From: Paul E. McKenney @ 2018-02-25 18:23 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

On Sun, Feb 25, 2018 at 08:47:23AM +0900, Akira Yokosawa wrote:
> >From 66f9f9f82d4fef12b6d7e2a399c14fbaa758e3f8 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Sun, 25 Feb 2018 08:09:50 +0900
> Subject: [PATCH 0/5] future/QC: Improve table layout
> 
> Hi Paul,
> 
> This patch set is a response to your recent updates in QC section.
> 
> Patch #1 retouches Table 17.5
> 
> Patch #2 removes alternate-row coloring in Table 17.5, which looks
> redundant after the retouch.
> 
> Patch #3 is for consistency of spacing around unit symbols as is
> suggested in Style Guide.
> 
> Patches #4 and #5 are a word choice suggestion and a typo fix.
> 
> Patch #2 can be skipped if you'd like to keep the background color.

Much better on all counts, thank you!  Queued and pushed.

							Thanx, Paul

> --
> Akira Yokosawa (5):
>   future/QC: Improve layout of Table 17.5 by using subfloat
>   future/QC: Remove alternate-row coloring in Table 17.5
>   future/QC: Add narrow space in front of percent symbol
>   future/QC: Disambiguate 'QC power'
>   future/QC: Fix typo
> 
>  future/QC.tex | 137 +++++++++++++++++++++++++++++++---------------------------
>  perfbook.tex  |   1 +
>  2 files changed, 75 insertions(+), 63 deletions(-)
> 
> -- 
> 2.7.4
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH] future/QC: Remove 'Readout' in header of 'entanglement error rate' row
  2018-02-25 18:23 ` [PATCH 0/5] future/QC: Improve table layout Paul E. McKenney
@ 2018-02-25 22:30   ` Akira Yokosawa
  2018-02-25 23:01     ` Paul E. McKenney
  0 siblings, 1 reply; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-25 22:30 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

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?

        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



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH] future/QC: Remove 'Readout' in header of 'entanglement error rate' row
  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
  2018-02-25 23:22       ` Akira Yokosawa
       [not found]       ` <ed46ece9-7825-3204-a101-a16982e44af8@gmail.com>
  0 siblings, 2 replies; 11+ messages in thread
From: Paul E. McKenney @ 2018-02-25 23:01 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

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
> 
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] future/QC: Remove 'Readout' in header of 'entanglement error rate' row
  2018-02-25 23:01     ` Paul E. McKenney
@ 2018-02-25 23:22       ` Akira Yokosawa
       [not found]       ` <ed46ece9-7825-3204-a101-a16982e44af8@gmail.com>
  1 sibling, 0 replies; 11+ messages in thread
From: Akira Yokosawa @ 2018-02-25 23:22 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

On 2018/02/25 15:01:18 -0800, Paul E. McKenney wrote:
> 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?

Looks good.

I'll take care of horizontal space adjustment of the table for 1c layout later,
if it is ever required.

        Thanks, Akira

> 
> 							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
>>
>>
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] future/QC: Tweak horizontal spacing of Table 17.5
       [not found]       ` <ed46ece9-7825-3204-a101-a16982e44af8@gmail.com>
@ 2018-02-26 16:35         ` Paul E. McKenney
  0 siblings, 0 replies; 11+ messages in thread
From: Paul E. McKenney @ 2018-02-26 16:35 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

On Tue, Feb 27, 2018 at 12:38:53AM +0900, Akira Yokosawa wrote:
> >From ff12fb77a17717794935078d8068c1718e867c56 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Mon, 26 Feb 2018 23:56:48 +0900
> Subject: [PATCH] future/QC: Tweak horizontal spacing of Table 17.5
> 
> By employing \multirow command of the "multirow" package, the width
> of the 1st column of entanglement table can be reduced.
> 
> Note: The 1st argument of \multirow specifies the number of rows to span,
> and the 2nd argument, in this case {1in}, specifies the width of the
> column. Internally it is translated to a \parbox. 
> 
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>

Much better, and thank you again!  Queued and pushed.

							Thanx, Paul

> ---
> Hi Paul,
> 
> Other header structures can be possible, but this looks good enough
> for the moment.
> 
>         Thanks, Akira
> --
>  future/QC.tex | 3 ++-
>  perfbook.tex  | 1 +
>  2 files changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/future/QC.tex b/future/QC.tex
> index bbddf83..13e5dd7 100644
> --- a/future/QC.tex
> +++ b/future/QC.tex
> @@ -674,6 +674,7 @@ $10^{-4}$).
>  \end{tabular}
>  }
> 
> +\OneColumnHSpace{-0.15in}%
>  \subfloat[Multi-Qubit Gate Properties]{
>  \begin{tabular}{@{}ll*{6}{S[table-format=2.2]}@{}}
>  	\toprule
> @@ -686,7 +687,7 @@ $10^{-4}$).
>  						\multicolumn{1}{c}{CX4\_2} &
>  							\multicolumn{1}{c@{}}{CX3\_4} \\
>  	\cmidrule(r){1-2} \cmidrule{3-8}
> -	Qubit Entanglement (\qop{CNOT}) & Error Rate (\%) &
> +	\multirow{2}{1in}{Qubit Entanglement (\qop{CNOT})} & Error Rate (\%) &
>  		 3.46 &
>  			 4.07 &
>  				 3.26 &
> diff --git a/perfbook.tex b/perfbook.tex
> index 8c2288a..efb6750 100644
> --- a/perfbook.tex
> +++ b/perfbook.tex
> @@ -59,6 +59,7 @@
>  \usepackage{braket} % for \ket{} macro in QC section
>  \usepackage{siunitx} % for \num{} macro
>  \sisetup{group-minimum-digits=4,group-separator={,},group-digits=integer}
> +\usepackage{multirow}
> 
>  % custom packages
> 
> -- 
> 2.7.4
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe perfbook" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2018-02-26 16:37 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox