Make menu plots respect logarithmic_intensity and display_title confi…

…guration

src/shiver/views/plots.py

@@ -6,9 +6,9 @@
 
 
 @manage_workspace_names
-def do_1d_plot(workspaces, display_name, intensity_limits=None, log_scale=False):
+def do_1d_plot(workspaces, display_name, intensity_limits=None, log_scale=False, errors=False, overplot=False):
     """Create an 1D plot for the provided workspace"""
-    fig = plot_md_ws_from_names(workspaces, False, False)
+    fig = plot_md_ws_from_names(workspaces, errors, overplot)
     min_limit = intensity_limits["min"] if intensity_limits is not None and "min" in intensity_limits else None
     max_limit = intensity_limits["max"] if intensity_limits is not None and "max" in intensity_limits else None
 

src/shiver/views/workspace_tables.py

@@ -25,8 +25,10 @@
 from shiver.presenters.polarized import PolarizedPresenter
 from shiver.models.polarized import PolarizedModel
 
+from shiver.configuration import get_data
+
 from .invalid_styles import INVALID_QLISTWIDGET
-from .plots import do_colorfill_plot, do_slice_viewer, plot_md_ws_from_names
+from .plots import do_colorfill_plot, do_slice_viewer, do_1d_plot
 from .workspace_icons import IconLegend, get_icon
 
 Frame = Enum("Frame", {"None": 1000, "QSample": 1001, "QLab": 1002, "HKL": 1003})
@@ -654,6 +656,10 @@ def __init__(self, parent=None):
         self.save_script_callback = None
         self.plot_callback = None
 
+    def get_display_name_and_intensity_limits(self, selected_ws, ndims):
+        """get the display name and intensity limits"""
+        return self.parent().parent().get_plot_data(selected_ws, ndims)
+
     def contextMenu(self, pos):  # pylint: disable=invalid-name
         """right-click event handler"""
         selected_ws = self.itemAt(pos)
@@ -665,30 +671,42 @@ def contextMenu(self, pos):  # pylint: disable=invalid-name
 
         menu = QMenu(self)
 
+        log_scale = get_data("main_tab.plot", "logarithmic_intensity")
+        display_name, intensity_limits = self.get_display_name_and_intensity_limits(selected_ws, ndims)
+
         if ndims == 1:
             plot = QAction("Plot 1D")
-            plot.triggered.connect(partial(self.plot_1d, selected_ws, False, False))
+
+            plot.triggered.connect(
+                partial(self.plot_1d, selected_ws, display_name, intensity_limits, log_scale, False, False)
+            )
             menu.addAction(plot)
 
             overplot = QAction("Overplot 1D")
-            overplot.triggered.connect(partial(self.plot_1d, selected_ws, False, True))
+            overplot.triggered.connect(
+                partial(self.plot_1d, selected_ws, display_name, intensity_limits, log_scale, False, True)
+            )
             menu.addAction(overplot)
 
             plot_err = QAction("Plot 1D with errors")
-            plot_err.triggered.connect(partial(self.plot_1d, selected_ws, True, False))
+            plot_err.triggered.connect(
+                partial(self.plot_1d, selected_ws, display_name, intensity_limits, log_scale, True, False)
+            )
             menu.addAction(plot_err)
 
             overplot_err = QAction("Overplot 1D with errors")
-            overplot_err.triggered.connect(partial(self.plot_1d, selected_ws, True, True))
+            overplot_err.triggered.connect(
+                partial(self.plot_1d, selected_ws, display_name, intensity_limits, log_scale, True, True)
+            )
             menu.addAction(overplot_err)
         elif ndims == 2:
             colorfill = QAction("Plot colorfill")
-            colorfill.triggered.connect(partial(self.plot_2d, selected_ws))
+            colorfill.triggered.connect(partial(self.plot_2d, selected_ws, display_name, intensity_limits, log_scale))
             menu.addAction(colorfill)
 
         if ndims > 1:
             sliceviewer = QAction("Show Slice Viewer")
-            sliceviewer.triggered.connect(partial(self.plot_slice_viewer, selected_ws))
+            sliceviewer.triggered.connect(partial(self.plot_slice_viewer, selected_ws, intensity_limits, log_scale))
             menu.addAction(sliceviewer)
 
         menu.addSeparator()
@@ -720,17 +738,17 @@ def add_ws(self, name, ws_type, frame, ndims):
         if ws_type == self.ws_type and name != "None":
             self.addItem(QListWidgetItem(name, type=ndims))
 
-    def plot_1d(self, name, errors, overplot):
+    def plot_1d(self, name, display_name, intensity_limits, log_scale, errors, overplot):
         """method to do 1D plots"""
-        plot_md_ws_from_names([name], errors, overplot)
+        do_1d_plot([name], display_name, intensity_limits, log_scale, errors, overplot)
 
-    def plot_2d(self, name):
+    def plot_2d(self, name, display_name, intensity_limits, log_scale):
         """method to do 2D plots"""
-        do_colorfill_plot([name])
+        do_colorfill_plot([name], display_name, intensity_limits, log_scale)
 
-    def plot_slice_viewer(self, name):
+    def plot_slice_viewer(self, name, intensity_limits, log_scale):
         """method to open sliceviewer"""
-        do_slice_viewer([name], self)
+        do_slice_viewer([name], self, intensity_limits, log_scale)
 
     def save_script(self, name):
         """method to handle the saving of script"""

tests/views/test_mdh_workspaces.py

@@ -24,6 +24,11 @@ def test_mdh_workspaces_menu(qtbot):
     qtbot.addWidget(mdh_table)
     mdh_table.show()
 
+    def mock_call(ws_name, ndims):
+        return f"full {ws_name}: {ndims}", {"min": None, "max": None}
+
+    mdh_table.get_display_name_and_intensity_limits = mock_call
+
     mdh_table.add_ws("mdh1", "mdh", "HKL", 1)
     mdh_table.add_ws("mdh2", "mdh", "HKL", 2)
     mdh_table.add_ws("mdh3", "mdh", "HKL", 3)
@@ -139,6 +144,12 @@ def test_mdh_plotting_1d(qtbot):
     )
 
     mdh_table = MDHList()
+
+    def mock_call(ws_name, ndims):
+        return f"full {ws_name}: {ndims}", {"min": None, "max": None}
+
+    mdh_table.get_display_name_and_intensity_limits = mock_call
+
     qtbot.addWidget(mdh_table)
     mdh_table.show()
 
@@ -215,6 +226,12 @@ def test_mdh_plotting_2d(qtbot):
     )
 
     mdh_table = MDHList()
+
+    def mock_call(ws_name, ndims):
+        return f"full {ws_name}: {ndims}", {"min": None, "max": None}
+
+    mdh_table.get_display_name_and_intensity_limits = mock_call
+
     qtbot.addWidget(mdh_table)
     mdh_table.show()
 
@@ -270,6 +287,12 @@ def test_mdh_plotting_3d(qtbot):
     )
 
     mdh_table = MDHList()
+
+    def mock_call(ws_name, ndims):
+        return f"full {ws_name}: {ndims}", {"min": None, "max": None}
+
+    mdh_table.get_display_name_and_intensity_limits = mock_call
+
     qtbot.addWidget(mdh_table)
     mdh_table.show()