Changeset 00b3b40 in sasview for src/sas/qtgui/Perspectives/Fitting/FittingWidget.py

Timestamp:

May 15, 2017 10:57:34 AM (8 years ago)

Author:

Piotr Rozyczko <rozyczko@…>

Branches:

ESS_GUI, ESS_GUI_Docs, ESS_GUI_batch_fitting, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_iss879, ESS_GUI_iss959, ESS_GUI_opencl, ESS_GUI_ordering, ESS_GUI_sync_sascalc

Children:

2a432e7

Parents:

2241130

Message:

Improvements to view delegates and model updates.

File:

• src/sas/qtgui/Perspectives/Fitting/FittingWidget.py (modified) (8 diffs)

src/sas/qtgui/Perspectives/Fitting/FittingWidget.py

@@ -370 +370 @@
 
         # Update state stack
-        if self.undo_supported:
-            state = self.currentState()
-            self.pushFitPage(state)
+        self.updateUndo()
 
     def onSelectStructureFactor(self):
@@ -429 +427 @@
         name_index = self._poly_model.index(model_row, 0)
         # Extract changed value. Assumes proper validation by QValidator/Delegate
-        # Checkbox in column 0
+        # TODO: abstract away hardcoded column numbers
         if model_column == 0:
+            # Is the parameter checked for fitting?
             value = item.checkState()
+            # TODO: add the param to self.params_for_fitting
+        elif model_column == 6:
+            value = item.text()
+            # TODO: Modify Npts/Nsigs based on function choice
         else:
             try:
@@ -866 +869 @@
             return
         parameter_name = str(self._model_model.data(name_index).toPyObject()) # sld, background etc.
-        property_name = str(self._model_model.headerData(1, model_column).toPyObject()) # Value, min, max, etc.
-
+        property_index = self._model_model.headerData(1, model_column).toInt()[0]-1 # Value, min, max, etc.
+
+        # Update the parameter value - note: this supports +/-inf as well
         self.kernel_module.params[parameter_name] = value
 
-        # TODO: update min/max based on property_name
         # min/max to be changed in self.kernel_module.details[parameter_name] = ['Ang', 0.0, inf]
-        # magnetic params in self.kernel_module.details['M0:parameter_name'] = value
-        # multishell params in self.kernel_module.details[??] = value
+        self.kernel_module.details[parameter_name][property_index] = value
+
+        # TODO: magnetic params in self.kernel_module.details['M0:parameter_name'] = value
+        # TODO: multishell params in self.kernel_module.details[??] = value
 
         # Force the chart update when actual parameters changed
@@ -880 +885 @@
 
         # Update state stack
-        if self.undo_supported:
-            state = self.currentState()
-            self.pushFitPage(state)
+        self.updateUndo()
 
     def checkboxSelected(self, item):
@@ -943 +946 @@
         Update a QStandardModelIndex containing model data
         """
-        if fitted_data.name is None:
-            name = self.nameForFittedData(self.logic.data.filename)
-            fitted_data.title = name
-            fitted_data.name = name
-        else:
-            name = fitted_data.name
+        name = self.nameFromData(fitted_data)
         # Make this a line if no other defined
         if hasattr(fitted_data, 'symbol') and fitted_data.symbol is None:
@@ -959 +957 @@
         Create a QStandardModelIndex containing model data
         """
+        name = self.nameFromData(fitted_data)
+        # Notify the GUI manager so it can create the theory model in DataExplorer
+        new_item = GuiUtils.createModelItemWithPlot(QtCore.QVariant(fitted_data), name=name)
+        self.communicate.updateTheoryFromPerspectiveSignal.emit(new_item)
+
+    def nameFromData(self, fitted_data):
+        """
+        Return name for the dataset. Terribly impure function.
+        """
         if fitted_data.name is None:
-            name = self.nameForFittedData(self.kernel_module.name)
+            name = self.nameForFittedData(self.logic.data.filename)
             fitted_data.title = name
             fitted_data.name = name
@@ -966 +973 @@
         else:
             name = fitted_data.name
-        # Notify the GUI manager so it can create the theory model in DataExplorer
-        new_item = GuiUtils.createModelItemWithPlot(QtCore.QVariant(fitted_data), name=name)
-        self.communicate.updateTheoryFromPerspectiveSignal.emit(new_item)
+        return name
 
     def methodCalculateForData(self):
@@ -1258 +1263 @@
         # TODO: add polidyspersity and magnetism
 
+
+    def updateUndo(self):
+        """
+        Create a new state page and add it to the stack
+        """
+        if self.undo_supported:
+            self.pushFitPage(self.currentState())
+
     def currentState(self):
         """