Changeset 7fb68060 in sasview for src

Timestamp:

Oct 29, 2018 7:54:42 AM (6 years ago)

Author:

piotr

Branches:

ESS_GUI, ESS_GUI_batch_fitting, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_opencl, ESS_GUI_ordering, ESS_GUI_sync_sascalc

Children:

6216eb75

Parents:

ec4a143

Message:

Minor fixes to the combobox enablement and multiplicity choice. SASVIEW-1153

File:

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

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

@@ -293 +293 @@
         self.has_magnet_error_column = False
 
+        # Enablement of comboboxes
+        self.enabled_cbmodel = False
+        self.enabled_sfmodel = False
+
         # If the widget generated theory item, save it
         self.theory_item = None
@@ -459 +463 @@
         self.cbModel.setEnabled(False)
         self.lblModel.setEnabled(False)
+        self.enabled_cbmodel = False
 
     def enableModelCombo(self):
@@ -464 +469 @@
         self.cbModel.setEnabled(True)
         self.lblModel.setEnabled(True)
+        self.enabled_cbmodel = True
 
     def disableStructureCombo(self):
@@ -469 +475 @@
         self.cbStructureFactor.setEnabled(False)
         self.lblStructure.setEnabled(False)
+        self.enabled_sfmodel = False
 
     def enableStructureCombo(self):
@@ -474 +481 @@
         self.cbStructureFactor.setEnabled(True)
         self.lblStructure.setEnabled(True)
+        self.enabled_sfmodel = True
 
     def togglePoly(self, isChecked):
@@ -3100 +3108 @@
         shell_par = None
         for par in kernel_pars:
-            if par.name == param_name:
+            parname = par.name
+            if '[' in parname:
+                 parname = parname[:parname.index('[')]
+            if parname == param_name:
                 shell_par = par
                 break
-        if not shell_par:
+        if shell_par is None:
             logger.error("Could not find %s in kernel parameters.", param_name)
+            return
         default_shell_count = shell_par.default
         shell_min = 0
@@ -3114 +3126 @@
             # no info about limits
             pass
+        except Exception as ex:
+            logging.error("Badly defined multiplicity: "+ str(ex))
+            return
         # don't update the kernel here - this data is display only
         self._model_model.blockSignals(True)
@@ -3120 +3135 @@
         self._model_model.blockSignals(False)
 
+        ## Respond to index change
+        #func.currentTextChanged.connect(self.modifyShellsInList)
+
+        # Respond to button press
+        button.clicked.connect(self.onShowSLDProfile)
+
+        # Available range of shells displayed in the combobox
+        func.addItems([str(i) for i in range(shell_min, shell_max+1)])
+
         # Respond to index change
         func.currentTextChanged.connect(self.modifyShellsInList)
 
-        # Respond to button press
-        button.clicked.connect(self.onShowSLDProfile)
-
-        # Available range of shells displayed in the combobox
-        func.addItems([str(i) for i in range(shell_min, shell_max+1)])
-
         # Add default number of shells to the model
         func.setCurrentText(str(default_shell_count))
+        self.modifyShellsInList(str(default_shell_count))
 
     def modifyShellsInList(self, text):
@@ -3178 +3197 @@
         """
         # get profile data
-        x, y = self.kernel_module.getProfile()
+        try:
+            x, y = self.kernel_module.getProfile()
+        except TypeError:
+            msg = "SLD profile calculation failed."
+            logging.error(msg)
+            return
+
         y *= 1.0e6
         profile_data = Data1D(x=x, y=y)
@@ -3211 +3236 @@
 
         self.cbCategory.setEnabled(enabled)
-        self.cbModel.setEnabled(enabled)
+
+        if enabled:
+            # worry about original enablement of model and SF
+            self.cbModel.setEnabled(self.enabled_cbmodel)
+            self.cbStructureFactor.setEnabled(self.enabled_sfmodel)
+        else:
+            self.cbModel.setEnabled(enabled)
+            self.cbStructureFactor.setEnabled(enabled)
+
         self.cmdPlot.setEnabled(enabled)