Fix num oracle queries in QAE (#1242)

* fix num oracle queries * add reno * fix statevector case Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com> Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
qiskit-community · Sep 15, 2020 · 01efe24 · 01efe24
1 parent 5f52856
commit 01efe24
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/qiskit/aqua/algorithms/amplitude_estimators/ae.py b/qiskit/aqua/algorithms/amplitude_estimators/ae.py
@@ -435,7 +435,7 @@ def _run(self) -> 'AmplitudeEstimationResult':
                 self._ret['value'] = val
 
         # count the number of Q-oracle calls
-        self._ret['num_oracle_queries'] = self._M - 1
+        self._ret['num_oracle_queries'] = self._ret['shots'] * (self._M - 1)
 
         # get MLE
         self._run_mle()

diff --git a/releasenotes/notes/ae-num-oracle-queries-a14bd74321768de3.yaml b/releasenotes/notes/ae-num-oracle-queries-a14bd74321768de3.yaml
@@ -0,0 +1,7 @@
+---
+fixes:
+  - |
+    For the amplitude estimation algorithms, we define the number of oracle queries 
+    as number of times the Q operator/Grover operator is applied. This includes 
+    the number of shots. That factor has been included in MLAE and IQAE but 
+    was missing in the 'standard' QAE.