Changeset 144ec831 in sasview for src/sas/qtgui/Perspectives/Fitting

Timestamp:

Jun 14, 2017 5:43:14 AM (7 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:

b0c5e8c

Parents:

1643d8ed

Message:

A few more minor fixes after code review SASVIEW-594

File:

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

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

@@ -1296 +1296 @@
         if '[' in param.name:
             return
-        # Values from the sasmodel
-        width = self.kernel_module.getParam(param.name+'.width')
-        npts = self.kernel_module.getParam(param.name+'.npts')
-        nsigs = self.kernel_module.getParam(param.name+'.nsigmas')
-        # Potential multishell params
-        checked_list = ["Distribution of "+param.name, str(width),
+        # Polydisp. values from the sasmodel
+        width = self.kernel_module.getParam(param.name + '.width')
+        npts = self.kernel_module.getParam(param.name + '.npts')
+        nsigs = self.kernel_module.getParam(param.name + '.nsigmas')
+
+        # Construct a row with polydisp. related variable.
+        # This will get added to the polydisp. model
+        # Note: last argument needs extra space padding for decent display of the control
+        checked_list = ["Distribution of " + param.name, str(width),
                         str(param.limits[0]), str(param.limits[1]),
                         str(npts), str(nsigs), "gaussian      "]
@@ -1309 +1312 @@
         func = QtGui.QComboBox()
         func.addItems([str(name_disp) for name_disp in POLYDISPERSITY_MODELS.iterkeys()])
-        # Default index
+        # set the default index
         func.setCurrentIndex(func.findText(DEFAULT_POLYDISP_FUNCTION))
-        # Index in the view
-        #func_index = self.lstPoly.model().index(row, 6)
 
     def onPolyComboIndexChange(self, combo_string, row_index):
@@ -1318 +1319 @@
         Modify polydisp. defaults on function choice
         """
-        # get npts/nsigs for current selection
+        # Get npts/nsigs for current selection
         param = self.model_parameters.form_volume_parameters[row_index]
 
@@ -1326 +1327 @@
             if param_name !=  param.name:
                 return
-            # modify the param value
+            # Modify the param value
             self._model_model.item(row, 0).child(0).child(0,4).setText(combo_string)