From ebe67079f38ee18e671e7fc0528ac0381f66d291 Mon Sep 17 00:00:00 2001 From: wyzula-jan Date: Fri, 16 Oct 2026 09:26:07 +0000 Subject: [PATCH] refactor(multi_waveform): remove unused MultiWaveformSelectionToolbarBundle --- .../toolbar_components/monitor_selection.py | 65 +------------------ 1 file changed, 2 insertions(+), 63 deletions(-) diff --git a/bec_widgets/widgets/plots/multi_waveform/toolbar_components/monitor_selection.py b/bec_widgets/widgets/plots/multi_waveform/toolbar_components/monitor_selection.py index d8ed8b5e..0f27b196 100644 --- a/bec_widgets/widgets/plots/multi_waveform/toolbar_components/monitor_selection.py +++ b/bec_widgets/widgets/plots/multi_waveform/toolbar_components/monitor_selection.py @@ -1,27 +1,13 @@ from bec_lib.device import ReadoutPriority -from qtpy.QtCore import Qt -from qtpy.QtWidgets import QStyledItemDelegate, QWidget +from qtpy.QtWidgets import QWidget -from bec_widgets.utils.error_popups import SafeSlot from bec_widgets.utils.toolbars.actions import DeviceComboBoxAction, WidgetAction from bec_widgets.utils.toolbars.bundles import ToolbarComponents from bec_widgets.utils.toolbars.toolbar import ToolbarBundle -from bec_widgets.widgets.control.device_input.device_combobox.device_combobox import ( - BECDeviceFilter, - DeviceComboBox, -) +from bec_widgets.widgets.control.device_input.device_combobox.device_combobox import BECDeviceFilter from bec_widgets.widgets.utility.visual.colormap_widget.colormap_widget import BECColorMapWidget -class NoCheckDelegate(QStyledItemDelegate): - """To reduce space in combo boxes by removing the checkmark.""" - - def initStyleOption(self, option, index): - super().initStyleOption(option, index) - # Remove any check indicator - option.checkState = Qt.Unchecked - - def monitor_selection_bundle( components: ToolbarComponents, target_widget: QWidget ) -> ToolbarBundle: @@ -51,50 +37,3 @@ def monitor_selection_bundle( bundle.add_action("monitor_selection") bundle.add_action("color_map") return bundle - - -class MultiWaveformSelectionToolbarBundle(ToolbarBundle): - """ - A bundle of actions for a toolbar that selects motors. - """ - - def __init__(self, bundle_id="monitor_selection", target_widget=None, **kwargs): - super().__init__(bundle_id=bundle_id, actions=[], **kwargs) - self.target_widget = target_widget - - # Monitor Selection - self.monitor = DeviceComboBox( - device_filter=BECDeviceFilter.DEVICE, readout_priority_filter=ReadoutPriority.ASYNC - ) - self.monitor.addItem("", None) - self.monitor.setCurrentText("") - self.monitor.setToolTip("Select Monitor") - self.monitor.setItemDelegate(NoCheckDelegate(self.monitor)) - self.add_action("monitor", WidgetAction(widget=self.monitor, adjust_size=False)) - - # Colormap Selection - self.colormap_widget = BECColorMapWidget(cmap="plasma") - self.add_action("color_map", WidgetAction(widget=self.colormap_widget, adjust_size=False)) - - # Connect slots, a device will be connected upon change of any combobox - self.monitor.currentTextChanged.connect(lambda: self.connect()) - self.colormap_widget.colormap_changed_signal.connect(self.change_colormap) - - @SafeSlot() - def connect(self): - monitor = self.monitor.currentText() - - if monitor != "": - if monitor != self.target_widget.config.monitor: - self.target_widget.monitor = monitor - - @SafeSlot(str) - def change_colormap(self, colormap: str): - self.target_widget.color_palette = colormap - - def cleanup(self): - """ - Cleanup the toolbar bundle. - """ - self.monitor.close() - self.monitor.deleteLater()