tomwer 1.3.27__py3-none-any.whl → 1.4.0__py3-none-any.whl

orangecontrib/tomwer/widgets/reconstruction/AxisOW.py

@@ -1,31 +1,5 @@
 # coding: utf-8
-# /*##########################################################################
-#
-# Copyright (c) 2016-2017 European Synchrotron Radiation Facility
-#
-# Permission is hereby granted, free of charge, to any person obtaining a copy
-# of this software and associated documentation files (the "Software"), to deal
-# in the Software without restriction, including without limitation the rights
-# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-# copies of the Software, and to permit persons to whom the Software is
-# furnished to do so, subject to the following conditions:
-#
-# The above copyright notice and this permission notice shall be included in
-# all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-# THE SOFTWARE.
-#
-# ###########################################################################*/
-
-__authors__ = ["H. Payno"]
-__license__ = "MIT"
-__date__ = "04/03/2019"
+from __future__ import annotations
 
 import copy
 import functools
@@ -50,7 +24,7 @@ from tomwer.core.process.reconstruction.axis import AxisTask
 from tomwer.core.process.reconstruction.axis.mode import AxisMode
 from tomwer.core.scan.nxtomoscan import NXtomoScan
 from tomwer.core.scan.scanbase import TomwerScanBase, _TomwerBaseDock
-from tomwer.gui.reconstruction.axis import AxisWindow
+from tomwer.gui.reconstruction.axis import AxisMainWindow
 from tomwer.synctools.axis import QAxisRP
 from tomwer.synctools.stacks.reconstruction.axis import AxisProcessStack
 
@@ -64,7 +38,7 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
     Widget used to defined the center of rotation axis to be used for a
     reconstruction.
 
-    :param bool _connect_handler: True if we want to store the modifications
+    :param _connect_handler: True if we want to store the modifications
                       on the setting. Need for unit test since
                       keep alive qt widgets.
     """
@@ -112,12 +86,12 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
     class Outputs:
         data = Output(name="data", type=TomwerScanBase, doc="one scan to be process")
 
-    def __init__(self, parent=None, axis_params=None):
+    def __init__(self, parent=None, axis_params: QAxisRP | None = None):
         """
 
         :param parent: QWidget parent or None
         :param _connect_handler: used for CI, because if connected fails CI
-        :param QAxisRP axis_params: reconstruction parameters
+        :param axis_params: reconstruction parameters
         """
         if axis_params is not None:
             if not isinstance(axis_params, QAxisRP):
@@ -160,23 +134,13 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
             axis_params=self._axis_params, process_id=self.process_id
         )
 
-        self._widget = AxisWindow(parent=self, axis_params=self._axis_params)
+        self._widget = AxisMainWindow(parent=self, axis_params=self._axis_params)
 
         self._layout = gui.vBox(self.mainArea, self.name).layout()
         self._layout.setContentsMargins(0, 0, 0, 0)
 
         self._layout.addWidget(self._widget)
 
-        # expose API
-        self.setMode = self._widget.setMode
-        self.getRadioMode = self._widget.getRadioMode
-        self.getEstimatedCor = self._widget.getEstimatedCor
-        self.setEstimatedCor = self._widget.setEstimatedCor
-        self.getAxis = self._widget.getAxis
-        self.setValueLock = self._widget.setValueLock
-        self._setModeLockFrmSettings = self._widget._setModeLockFrmSettings
-        self._setValueLockFrmSettings = self._widget._setValueLockFrmSettings
-
         # load settings
         try:
             if "mode_is_lock" in gui_settings:
@@ -184,7 +148,7 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
                 # if the mode is manual or read ignore lock
                 if not (
                     mode_lock is True
-                    and self.getRadioMode() in (AxisMode.manual, AxisMode.read)
+                    and self.getMode() in (AxisMode.manual, AxisMode.read)
                 ):
                     self._setModeLockFrmSettings(mode_lock)
 
@@ -196,12 +160,13 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
                 "auto_update_estimated_cor", True
             )
             self._widget.setAutoUpdateEstimatedCor(auto_update_estimated_cor)
+            self._widget.setYAxisInverted(gui_settings.get("y_axis_inverted", False))
 
         except Exception as e:
             logger.warning(f"Fail to load settings. Error is {str(e)}")
 
         # expose API
-        self._applyBut = self._widget._axisWidget._controlWidget._applyBut
+        self._applyBut = self._widget._controlWidget._applyBut
         # connect Signal / Slot
         self._widget.sigComputationRequested.connect(self.__compute)
         self._widget.sigApply.connect(self.__validate)
@@ -210,16 +175,19 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
         self._processingStack.sigComputationEnded.connect(self._scanProcessed)
 
         self._axis_params.sigChanged.connect(self._updateSettingsVals)
+        self._widget._axisWidget._settingsWidget._mainWidget._calculationWidget.sigUpdateXRotAxisPixelPosOnNewScan.connect(
+            self._updateSettingsVals
+        )
         self._widget.sigAxisEditionLocked.connect(self._updateSettingsVals)
-        self._widget.sigMethodChanged.connect(self._updateSettingsVals)
+        self._widget.sigModeChanged.connect(self._updateSettingsVals)
         self._widget.sigLockModeChanged.connect(self._updateSettingsVals)
 
-        self._widget._axisWidget._radioAxis._controlWidget._mainWidget._calculationWidget._nearValueCB.toggled.connect(
+        self._widget._axisWidget._settingsWidget._mainWidget._calculationWidget._estimatedCorWidget.sigValueChanged.connect(
             self._updateSettingsVals
         )
 
         # handle special case of the manual mode
-        # force axis to manual because not handled by the widget setup dircetly
+        # force axis to manual because not handled by the widget setup directly
         if original_mode in (
             AxisMode.manual,
             AxisMode.read,
@@ -230,11 +198,11 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
 
             self.setMode(original_mode)
             # force gui update
-            self._widget.sigMethodChanged.emit("manual")
+            self._widget.sigModeChanged.emit("manual")
             # hard fix, but this widget has to be redone and rework anyway
-            self._widget._axisWidget._radioAxis._controlWidget._mainWidget._calculationWidget._modeChanged()
+            self._widget._axisWidget._settingsWidget._mainWidget._calculationWidget._modeChanged()
             if original_cor is not None:
-                self._widget._axisWidget._controlWidget._positionInfo.setPosition(
+                self._widget._controlWidget._positionInfo.setPosition(
                     relative_cor=original_cor, abs_cor=None
                 )
 
@@ -316,7 +284,9 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
         The Axis will process the next scan in the stack.
         """
         if self.__scan:
-            if self.getRadioMode() is AxisMode.manual:
+            if self.getMode() is AxisMode.manual:
+                # when value is set from manual mode we need to retrieve it from the GUI `_axis_param`. Else it has already been set
+                # to the scan `_axis_params`
                 if (
                     self.__scan._axis_params.frame_width is None
                     and self.__scan.dim_1 is not None
@@ -376,7 +346,8 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
             "gui": {
                 "mode_is_lock": self.isModeLocked(),
                 "value_is_lock": self.isValueLock(),
-                "auto_update_estimated_cor": self._widget._axisWidget.getAutoUpdateEstimatedCor(),
+                "auto_update_estimated_cor": self._widget.getAutoUpdateEstimatedCor(),
+                "y_axis_inverted": self._widget.isYAxisInverted(),
             },
         }
 
@@ -392,7 +363,7 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
     def _lock_axis_controls(self, lock):
         """
 
-        :param bool lock: lock the axis controls to avoid modification of the
+        :param lock: lock the axis controls to avoid modification of the
                           requested options, method... of the axis calculation
                           when this value is under calculation.
         """
@@ -482,9 +453,6 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
                         "serialize_output_data": False,
                     }
                 )
-                for patch in self._patches:
-                    mode, function = patch
-                    processing_class._CALCULATIONS_METHODS[mode] = function
                 processing_class.run()
             self._updatePosition(scan=self.__scan)
             if self.isModeLocked() or self.isValueLock():
@@ -533,9 +501,7 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
 
     def _updatePosition(self, scan):
         if scan.axis_params is not None:
-            self._widget.setPosition(
-                frm=None, value=scan.axis_params.relative_cor_value
-            )
+            self._widget.setPosition(relative_value=scan.axis_params.relative_cor_value)
 
     @Inputs.data_recompute_axis
     def reprocess(self, scan):
@@ -559,9 +525,34 @@ class AxisOW(SuperviseOW, WidgetLongProcessing):
 
     def keyPressEvent(self, event):
         """The event has to be filtered since we have some children
-         that can be edited using the 'enter' key as defining the cor manually
+        that can be edited using the 'enter' key as defining the cor manually
         (see #481)). As we are in a dialog this automatically trigger
         'accepted'. See https://forum.qt.io/topic/5080/preventing-enter-key-from-triggering-ok-in-qbuttonbox-in-particular-qlineedit-qbuttonbox/5
         """
         if event.key() != qt.Qt.Key_Enter:
             super().keyPressEvent(event)
+
+    # expose API
+    def setMode(self, mode):
+        self._widget.setMode(mode=mode)
+
+    def getMode(self):
+        return self._widget.getMode()
+
+    def getEstimatedCor(self):
+        return self._widget.getEstimatedCor()
+
+    def setEstimatedCor(self, value):
+        self._widget.setEstimatedCor(value=value)
+
+    def getAxisParams(self):
+        return self._widget.getAxisParams()
+
+    def setValueLock(self, lock: bool):
+        self._widget.setValueLock(lock=lock)
+
+    def _setModeLockFrmSettings(self, lock: bool):
+        self._widget._setModeLockFrmSettings(lock=lock)
+
+    def _setValueLockFrmSettings(self, lock: bool):
+        self._widget._setValueLockFrmSettings(lock=lock)

orangecontrib/tomwer/widgets/reconstruction/CastNabuVolumeOW.py

@@ -1,50 +1,21 @@
-# coding: utf-8
-# /*##########################################################################
-#
-# Copyright (c) 2016-2017 European Synchrotron Radiation Facility
-#
-# Permission is hereby granted, free of charge, to any person obtaining a copy
-# of this software and associated documentation files (the "Software"), to deal
-# in the Software without restriction, including without limitation the rights
-# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-# copies of the Software, and to permit persons to whom the Software is
-# furnished to do so, subject to the following conditions:
-#
-# The above copyright notice and this permission notice shall be included in
-# all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-# THE SOFTWARE.
-#
-# ###########################################################################*/
-
-__authors__ = ["H. Payno"]
-__license__ = "MIT"
-__date__ = "15/12/2021"
+from __future__ import annotations
 
 import logging
-from typing import Optional
 
 from orangewidget import gui
 from orangewidget.settings import Setting
 from orangewidget.widget import Input, Output
-from processview.core.manager.manager import DatasetState, ProcessManager
 from silx.gui import qt
 
 import tomwer.core.process.reconstruction.nabu.castvolume
 from orangecontrib.tomwer.widgets.utils import WidgetLongProcessing
-from tomwer.core import settings
 from tomwer.core.cluster.cluster import SlurmClusterConfiguration
 from tomwer.core.futureobject import FutureTomwerObject
 from tomwer.core.scan.scanbase import TomwerScanBase
-from tomwer.core.utils.lbsram import is_low_on_memory
 from tomwer.core.volume.volumebase import TomwerVolumeBase
 from tomwer.gui.reconstruction.nabu.castvolume import CastVolumeWidget
+from tomwer.gui.utils.qt_utils import block_signals
+
 from tomwer.synctools.stacks.reconstruction.castvolume import CastVolumeProcessStack
 from tomwer.utils import docstring
 
@@ -59,7 +30,7 @@ class CastNabuVolumeOW(WidgetLongProcessing, SuperviseOW):
 
     This is done in a separate process because:
 
-    * this is done in cpu when nabu reconstruct on GPU and this should free sooner GPU ressources with the current architecture.
+    * this is done in cpu when nabu reconstruct on GPU and this should free sooner GPU resources with the current architecture.
     * this is not included in nabu but also done as post processing.
     * limitation is that not having computed the histogram during volume construction will slow down the cast
 
@@ -164,61 +135,43 @@ class CastNabuVolumeOW(WidgetLongProcessing, SuperviseOW):
         return super().__new__(cls, *args, **kwargs)
 
     @Inputs.data
-    def process_data(self, scan: Optional[TomwerScanBase]):
-        self._process_data(scan)
+    def process_data(self, scan: TomwerScanBase | None):
+        self._process(scan=scan, volume=None)
 
-    def _process_data(self, scan: Optional[TomwerScanBase]):
-        if scan is None:
+    def _process(self, scan: TomwerScanBase | None, volume: TomwerVolumeBase | None):
+        if scan is None and volume is None:
             return
-        if not isinstance(scan, TomwerScanBase):
+        # check types
+        if scan is not None and not isinstance(scan, TomwerScanBase):
             raise TypeError(
                 f"scan is expected to be an instance of {TomwerScanBase} not {type(scan)}"
             )
-        if (
-            settings.isOnLbsram(scan)
-            and is_low_on_memory(settings.get_lbsram_path()) is True
-        ):
-            ProcessManager().notify_dataset_state(
-                dataset=scan,
-                process=self,
-                state=DatasetState.SKIPPED,
-            )
-            _logger.processSkipped(
-                f"skip volume cast for {str(scan)} because lbsram is almost full. Try to free it"
+        if volume is not None and not isinstance(volume, TomwerVolumeBase):
+            raise TypeError(
+                f"volume is expected to be an instance of {TomwerVolumeBase} not {type(volume)}"
             )
-            self.Outputs.data.send(scan)
-        else:
-            self._processingStack.add(scan, configuration=self.getConfiguration())
+
+        self._processingStack.add(scan or volume, configuration=self.getConfiguration())
 
     @docstring
     def reprocess(self, dataset):
         if isinstance(dataset, TomwerScanBase):
-            self._process_data(dataset)
+            self._process(scan=dataset, volume=None)
         elif isinstance(dataset, TomwerVolumeBase):
-            self._process_volume(dataset)
+            self._process(scan=None, volume=dataset)
         else:
             raise TypeError("dataset is expected to be an instance of ''")
 
     @Inputs.cluster_in
-    def setCluster(self, slurm_cluster: Optional[SlurmClusterConfiguration]):
+    def setCluster(self, slurm_cluster: SlurmClusterConfiguration | None):
         assert isinstance(
             slurm_cluster, (type(None), SlurmClusterConfiguration)
         ), f"Expect None of SlurmClusterConfiguration. Not {type(slurm_cluster)}"
         self._slurmCluster = slurm_cluster
 
     @Inputs.volume
-    def process_volume(self, volume: Optional[TomwerVolumeBase]):
-        self._process_volume(volume)
-
-    def _process_volume(self, volume: Optional[TomwerVolumeBase]):
-        if volume is None:
-            return
-        if not isinstance(volume, TomwerVolumeBase):
-            raise TypeError(
-                f"volume is expected to be an instance of {TomwerVolumeBase} not {type(volume)}"
-            )
-        else:
-            self._processingStack.add(volume, configuration=self.getConfiguration())
+    def process_volume(self, volume: TomwerVolumeBase | None):
+        self._process(scan=None, volume=volume)
 
     def getConfiguration(self) -> dict:
         config = self._window.getConfiguration()
@@ -226,7 +179,8 @@ class CastNabuVolumeOW(WidgetLongProcessing, SuperviseOW):
         return config
 
     def setConfiguration(self, configuration: dict) -> None:
-        self._window.setConfiguration(configuration)
+        with block_signals(self._window):
+            self._window.setConfiguration(configuration)
 
     def _updateConfig(self):
         self._ewoks_default_inputs = {

orangecontrib/tomwer/widgets/reconstruction/DarkRefAndCopyOW.py

@@ -1,32 +1,5 @@
 # coding: utf-8
-# /*##########################################################################
-#
-# Copyright (c) 2016-2017 European Synchrotron Radiation Facility
-#
-# Permission is hereby granted, free of charge, to any person obtaining a copy
-# of this software and associated documentation files (the "Software"), to deal
-# in the Software without restriction, including without limitation the rights
-# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-# copies of the Software, and to permit persons to whom the Software is
-# furnished to do so, subject to the following conditions:
-#
-# The above copyright notice and this permission notice shall be included in
-# all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-# THE SOFTWARE.
-#
-# ###########################################################################*/
-
-__authors__ = ["C. Nemoz", "H. Payno"]
-__license__ = "MIT"
-__date__ = "10/01/2018"
-
+from __future__ import annotations
 
 import copy
 import logging
@@ -113,10 +86,10 @@ class DarkRefAndCopyOW(SuperviseOW, WidgetLongProcessing):
             doc="DataUrl of the reduced flat. Key is the relative position, value is the reduced flat",
         )
 
-    def __init__(self, parent=None, reconsparams=None):
+    def __init__(self, parent=None, reconsparams: QDKRFRP | None = None):
         """
 
-        :param QKRFRP reconsparams: reconstruction parameters
+        :param reconsparams: reconstruction parameters
         """
         SuperviseOW.__init__(self, parent)
         WidgetLongProcessing.__init__(self)

orangecontrib/tomwer/widgets/reconstruction/NabuHelicalPrepareWeightsDoubleOW.py

@@ -1,3 +1,5 @@
+# from __future__ import annotations
+
 # import logging
 
 # from orangewidget import gui
@@ -159,26 +161,26 @@
 #         except Exception:
 #             pass
 
-#     def _notify_state(self):
-#         try:
-#             task_executor = self.sender()
-#             task_suceeded = task_executor.succeeded
-#             scan = task_executor.current_task.outputs.data
-#             if task_suceeded:
-#                 self.notify_succeed(scan=scan)
-#             else:
-#                 self.notify_failed(scan=scan)
-#         except Exception as e:
-#             _logger.error(f"failed to handle task finished callback. Raiseon is {e}")
-
-#     @Inputs.data
-#     def process_data(self, scan: Optional[TomwerScanBase]):
-#         if scan is None:
-#             return
+# def _notify_state(self):
+#     try:
+#         task_executor = self.sender()
+#         task_suceeded = task_executor.succeeded
+#         scan = task_executor.current_task.outputs.data
+#         if task_suceeded:
+#             self.notify_succeed(scan=scan)
 #         else:
-#             self.__scan = scan
-#             self.notify_pending(scan=scan)
-#             self.execute_ewoks_task()
+#             self.notify_failed(scan=scan)
+#     except Exception as e:
+#         _logger.error(f"failed to handle task finished callback. Reason is {e}")
+
+# @Inputs.data
+# def process_data(self, scan: TomwerScanBase | None):
+#     if scan is None:
+#         return
+#     else:
+#         self.__scan = scan
+#         self.notify_pending(scan=scan)
+#         self.execute_ewoks_task()
 
 #     def sizeHint(self) -> qt.QSize:
 #         return qt.QSize(650, 60)

orangecontrib/tomwer/widgets/reconstruction/NabuOW.py

@@ -1,38 +1,10 @@
 # coding: utf-8
-# /*##########################################################################
-#
-# Copyright (c) 2016-2020 European Synchrotron Radiation Facility
-#
-# Permission is hereby granted, free of charge, to any person obtaining a copy
-# of this software and associated documentation files (the "Software"), to deal
-# in the Software without restriction, including without limitation the rights
-# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-# copies of the Software, and to permit persons to whom the Software is
-# furnished to do so, subject to the following conditions:
-#
-# The above copyright notice and this permission notice shall be included in
-# all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-# THE SOFTWARE.
-#
-# ###########################################################################*/
-
-__authors__ = ["C. Nemoz", "H. Payno"]
-__license__ = "MIT"
-__date__ = "14/02/2020"
-
-
-import os
+
+from __future__ import annotations
+
 import copy
 import logging
 from contextlib import AbstractContextManager
-from typing import Optional
 
 from orangewidget import gui
 from orangewidget.settings import Setting
@@ -45,10 +17,7 @@ from tomwer.core.futureobject import FutureTomwerObject
 from tomwer.core.process.reconstruction.nabu.nabuslices import NabuSlicesTask
 from tomwer.core.scan.scanbase import TomwerScanBase, _TomwerBaseDock
 from tomwer.core.volume.volumefactory import VolumeFactory
-from tomwer.core.process.icat.gallery import (
-    IcatScreenshots,
-    deduce_dataset_gallery_location,
-)
+
 from tomwer.gui.reconstruction.nabu.slices import NabuWindow
 from tomwer.synctools.stacks.reconstruction.nabu import NabuSliceProcessStack
 
@@ -114,12 +83,6 @@ class NabuOW(WidgetLongProcessing, SuperviseOW):
 
         slice_urls = Output(name="slice urls", type=tuple, doc="tuple of urls created")
 
-        screenshots = Output(
-            name="screenshots",
-            type=IcatScreenshots,
-            doc="screenshots that can be saved",
-        )
-
     class DialogCM(AbstractContextManager):
         """Simple context manager to hide / show button dialogs"""
 
@@ -133,17 +96,6 @@ class NabuOW(WidgetLongProcessing, SuperviseOW):
             self._dialogButtonsBox.hide()
 
     def __init__(self, parent=None):
-        """
-        Widget which read the .hdf5 generated by octave and modify it.
-        Then run a subprocess to call octave and run ftseries
-
-        :param bool _connect_handler: True if we want to store the modifications
-                                      on the setting. Need for unit test since
-                                      keep alive qt widgets.
-        :param recons_params: reconsparameter to be used by the FTWidget.
-                              If None, some will be created.
-        :type: :class:`QReconsParams`
-        """
         SuperviseOW.__init__(self, parent)
         WidgetLongProcessing.__init__(self)
         self._slurmCluster = None
@@ -220,11 +172,7 @@ class NabuOW(WidgetLongProcessing, SuperviseOW):
                 from tomwer.synctools.axis import QAxisRP
 
                 scan.axis_params = QAxisRP()
-            scan.axis_params.set_relative_value(float(r_cor) - scan.dim_1 / 2.0)
-            try:
-                scan.axis_params.set_relative_value(float(r_cor) - scan.dim_1 / 2.0)
-            except Exception:
-                pass
+            scan.axis_params.set_relative_value(float(r_cor))
 
         self.show()
         with NabuOW.DialogCM(self._buttons):
@@ -254,7 +202,7 @@ class NabuOW(WidgetLongProcessing, SuperviseOW):
                 self._processingStack._process_next()
 
     @Inputs.cluster_in
-    def setCluster(self, slurm_cluster: Optional[SlurmClusterConfiguration]):
+    def setCluster(self, slurm_cluster: SlurmClusterConfiguration | None):
         assert isinstance(
             slurm_cluster, (type(None), SlurmClusterConfiguration)
         ), f"Expect None of SlurmClusterConfiguration. Not {type(slurm_cluster)}"
@@ -275,17 +223,6 @@ class NabuOW(WidgetLongProcessing, SuperviseOW):
                 slice_urls = tuple(slice_urls)
                 # provide list of reconstructed slices
                 self.Outputs.slice_urls.send(slice_urls)
-                # provide list of screenshots
-                self.Outputs.screenshots.send(
-                    IcatScreenshots(
-                        data_dir=deduce_dataset_gallery_location(scan),
-                        screenshots={
-                            os.path.splitext(os.path.basename(url.file_path()))[0]: url
-                            for url in slice_urls
-                        },
-                        scan=scan,
-                    )
-                )
 
             self.sigScanReady.emit(scan)
         if future_tomo_obj is not None:
@@ -322,7 +259,7 @@ class NabuOW(WidgetLongProcessing, SuperviseOW):
 
     def keyPressEvent(self, event):
         """The event has to be filtered since we have some children
-         that can be edited using the 'enter' key as defining the cor manually
+        that can be edited using the 'enter' key as defining the cor manually
         (see #481)). As we are in a dialog this automatically trigger
         'accepted'. See https://forum.qt.io/topic/5080/preventing-enter-key-from-triggering-ok-in-qbuttonbox-in-particular-qlineedit-qbuttonbox/5
         """