add CDF to hist

redhat-partner-solutions · Oct 9, 2023 · 8e05c53 · 8e05c53
1 parent ed72d46
commit 8e05c53
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/vse_sync_pp/plot.py b/src/vse_sync_pp/plot.py
@@ -52,7 +52,7 @@ def _plot_scatter(self, ax):
     def _plot_hist(self, ax):
         counts, bins = np.histogram(
             np.array(self._y_data, dtype=float),
-            bins='scott',
+            bins='fd'
         )
         ax.hist(bins[:-1], bins, weights=counts)
         self._set_yscale(ax)
@@ -66,8 +66,11 @@ def plot(self, filename):
         fig.set_size_inches(10, 8)
         self._plot_scatter(ax1)
         self._plot_hist(ax2)
+        ax3 = ax2.twinx()
+        ax3.set_ylabel('CDF')
+        ax3.ecdf(self._y_data, color="black", linewidth=2)
         plt.savefig(filename)
-        return fig, (ax1, ax2)
+        return fig, (ax1, ax2, ax3)
 
     def plot_scatter(self, filename):
         fig, ax = plt.subplots(1, constrained_layout=True)