Revert "Global improvement of rendering with Python tools"

This reverts commit 34e1ae726e27fdd7c41f6d80d8ed7f6504dc3a0d.

1 files changed, 15 insertions, 77 deletions

diff --git a/src/python/gudhi/persistence_graphical_tools.py b/src/python/gudhi/persistence_graphical_tools.py
index 4a690241..246280de 100644
--- a/src/python/gudhi/persistence_graphical_tools.py
+++ b/src/python/gudhi/persistence_graphical_tools.py
@@ -5,7 +5,6 @@
 # Copyright (C) 2016 Inria
 #
 # Modification(s):
-#   - 2020/02 Theo Lacombe: Added more options for improved rendering and more flexibility.
 #   - YYYY/MM Author: Description of the modification
 
 from os import path
@@ -44,7 +43,6 @@ def __min_birth_max_death(persistence, band=0.0):
         max_death += band
     return (min_birth, max_death)
 
-
 def plot_persistence_barcode(
     persistence=[],
     persistence_file="",
@@ -54,9 +52,7 @@ def plot_persistence_barcode(
     inf_delta=0.1,
     legend=False,
     colormap=None,
-    axes=None,
-    fontsize=16,
-    title="Persistence barcode"
+    axes=None
 ):
     """This function plots the persistence bar code from persistence values list
     or from a :doc:`persistence file <fileformats>`.
@@ -85,18 +81,11 @@ def plot_persistence_barcode(
     :param axes: A matplotlib-like subplot axes. If None, the plot is drawn on
         a new set of axes.
     :type axes: `matplotlib.axes.Axes`
-    :param fontsize: Fontsize to use in axis.
-    :type fontsize: int
-    :param title: title for the plot.
-    :type title: string
     :returns: (`matplotlib.axes.Axes`): The axes on which the plot was drawn.
     """
     try:
         import matplotlib.pyplot as plt
         import matplotlib.patches as mpatches
-        from matplotlib import rc
-        plt.rc('text', usetex=True)
-        plt.rc('font', family='serif')
 
         if persistence_file != "":
             if path.isfile(persistence_file):
@@ -174,7 +163,7 @@ def plot_persistence_barcode(
                 loc="lower right",
             )
 
-        axes.set_title(title)
+        axes.set_title("Persistence barcode")
 
         # Ends plot on infinity value and starts a little bit before min_birth
         axes.axis([axis_start, infinity, 0, ind])
@@ -194,11 +183,7 @@ def plot_persistence_diagram(
     inf_delta=0.1,
     legend=False,
     colormap=None,
-    axes=None,
-    aspect_equal=False,
-    fontsize=16,
-    title="Persistence diagram",
-    greyblock=True
+    axes=None
 ):
     """This function plots the persistence diagram from persistence values
     list or from a :doc:`persistence file <fileformats>`.
@@ -229,23 +214,11 @@ def plot_persistence_diagram(
     :param axes: A matplotlib-like subplot axes. If None, the plot is drawn on
         a new set of axes.
     :type axes: `matplotlib.axes.Axes`
-    :param aspect_equal: if True, force plot to be square shaped.
-    :type aspect_equal: boolean
-    :param fontsize: Fontsize to use in axis.
-    :type fontsize: int
-    :param title: title for the plot.
-    :type title: string
-    :param greyblock: if we want to plot a grey patch on the lower half plane for nicer rendering. Default True.
-    :type greyblock: boolean
     :returns: (`matplotlib.axes.Axes`): The axes on which the plot was drawn.
     """
     try:
         import matplotlib.pyplot as plt
         import matplotlib.patches as mpatches
-        from matplotlib import rc
-        plt.rc('text', usetex=True)
-        plt.rc('font', family='serif')
-
 
         if persistence_file != "":
             if path.isfile(persistence_file):
@@ -283,27 +256,18 @@ def plot_persistence_diagram(
         # Replace infinity values with max_death + delta for diagram to be more
         # readable
         infinity = max_death + delta
-        axis_end = max_death + delta / 2
         axis_start = min_birth - delta
 
         # line display of equation : birth = death
         x = np.linspace(axis_start, infinity, 1000)
         # infinity line and text
-        axes.plot([axis_start, axis_end], [axis_start, axis_end], linewidth=1.0, color="k")
-        axes.plot([axis_start, axis_end], [infinity, infinity], linewidth=1.0, color="k", alpha=alpha)
-        # Infinity label
-        yt = axes.get_yticks()
-        yt = np.append(yt, infinity)
-        ytl = yt.tolist()
-        ytl[-1] = r'$+\infty$'
-        axes.set_yticks(yt)
-        axes.set_yticklabels(ytl)
+        axes.plot(x, x, color="k", linewidth=1.0)
+        axes.plot(x, [infinity] * len(x), linewidth=1.0, color="k", alpha=alpha)
+        axes.text(axis_start, infinity, r"$\infty$", color="k", alpha=alpha)
         # bootstrap band
         if band > 0.0:
             axes.fill_between(x, x, x + band, alpha=alpha, facecolor="red")
-        # lower diag patch
-        if greyblock:
-            axes.add_patch(mpatches.Polygon([[axis_start, axis_start], [axis_end, axis_start], [axis_end, axis_end]], fill=True, color='lightgrey'))
+
         # Draw points in loop
         for interval in reversed(persistence):
             if float(interval[1][1]) != float("inf"):
@@ -329,13 +293,11 @@ def plot_persistence_diagram(
                 ]
             )
 
-        axes.set_xlabel("Birth", fontsize=fontsize)
-        axes.set_ylabel("Death", fontsize=fontsize)
+        axes.set_xlabel("Birth")
+        axes.set_ylabel("Death")
         # Ends plot on infinity value and starts a little bit before min_birth
-        axes.axis([axis_start, axis_end, axis_start, infinity + delta])
-        axes.set_title(title, fontsize=fontsize)  # a different fontsize for the title?
-        if aspect_equal:
-            axes.set_aspect("equal")
+        axes.axis([axis_start, infinity, axis_start, infinity + delta])
+        axes.set_title("Persistence diagram")
         return axes
 
     except ImportError:
@@ -351,11 +313,7 @@ def plot_persistence_density(
     dimension=None,
     cmap=None,
     legend=False,
-    axes=None,
-    aspect_equal=False,
-    fontsize=16,
-    title="Persistence density",
-    greyblock=True
+    axes=None
 ):
     """This function plots the persistence density from persistence
     values list or from a :doc:`persistence file <fileformats>`. Be
@@ -397,25 +355,11 @@ def plot_persistence_density(
     :param axes: A matplotlib-like subplot axes. If None, the plot is drawn on
         a new set of axes.
     :type axes: `matplotlib.axes.Axes`
-    :param aspect_equal: if True, force plot to be square shaped.
-    :type aspect_equal: boolean
-    :param fontsize: Fontsize to use in axis.
-    :type fontsize: int
-    :param title: title for the plot.
-    :type title: string
-    :param greyblock: if we want to plot a grey patch on the lower half plane 
-                         for nicer rendering. Default True.
-    :type greyblock: boolean
     :returns: (`matplotlib.axes.Axes`): The axes on which the plot was drawn.
     """
     try:
         import matplotlib.pyplot as plt
-        import matplotlib.patches as mpatches
         from scipy.stats import kde
-        from matplotlib import rc
-        plt.rc('text', usetex=True)
-        plt.rc('font', family='serif')
-
 
         if persistence_file != "":
             if dimension is None:
@@ -474,18 +418,12 @@ def plot_persistence_density(
         # Make the plot
         img = axes.pcolormesh(xi, yi, zi.reshape(xi.shape), cmap=cmap)
 
-        if greyblock:
-            axes.add_patch(mpatches.Polygon([[birth.min(), birth.min()], [death.max(), birth.min()], [death.max(), death.max()]], fill=True, color='lightgrey'))
-
         if legend:
             plt.colorbar(img, ax=axes)
 
-        axes.set_xlabel("Birth", fontsize=fontsize)
-        axes.set_ylabel("Death", fontsize=fontsize)
-        axes.set_title(title, fontsize=fontsize)
-        if aspect_equal:
-            axes.set_aspect("equal")
-
+        axes.set_xlabel("Birth")
+        axes.set_ylabel("Death")
+        axes.set_title("Persistence density")
         return axes
 
     except ImportError: