Changeset a758043 in sasview


Ignore:
Timestamp:
Sep 7, 2018 5:13:39 AM (11 days ago)
Author:
Torin Cooper-Bennun <torin.cooper-bennun@…>
Branches:
ESS_GUI, ESS_GUI_Pr_fixes, ESS_GUI_iss1032, ESS_GUI_iss1033, ESS_GUI_iss1034, ESS_GUI_iss1052, ESS_GUI_poly_plot2
Children:
bc7371fd
Parents:
88ada06
git-author:
Torin Cooper-Bennun <torin.cooper-bennun@…> (09/07/18 05:13:25)
git-committer:
Torin Cooper-Bennun <torin.cooper-bennun@…> (09/07/18 05:13:39)
Message:

function signatures and return values now maintain backwards compatibility, and do not break tests

Location:
src/sas/qtgui/Perspectives/Fitting
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • src/sas/qtgui/Perspectives/Fitting/FittingUtilities.py

    r88ada06 ra758043  
    9191    return cbox 
    9292 
    93 def addParametersToModel(model, view, parameters, kernel_module, is2D): 
    94     """ 
    95     Update local ModelModel with sasmodel parameters 
     93def addParametersToModel(parameters, kernel_module, is2D, model=None, view=None): 
     94    """ 
     95    Update local ModelModel with sasmodel parameters. 
     96    Actually appends to model, if model and view params are not None. 
     97    Always returns list of lists of QStandardItems. 
    9698    """ 
    9799    multishell_parameters = getIterParams(parameters) 
     
    103105        params = parameters.iq_parameters 
    104106 
     107    rows = [] 
    105108    for param in params: 
    106109        # don't include shell parameters 
     
    156159 
    157160        # Append to the model and use the combobox, if required 
    158         model.appendRow(row) 
    159         if cbox is not None: 
    160             view.setIndexWidget(item2.index(), cbox) 
    161  
    162 def addSimpleParametersToModel(model, view, parameters, is2D): 
     161        if None not in (model, view): 
     162            model.appendRow(row) 
     163            if cbox: 
     164                view.setIndexWidget(item2.index(), cbox) 
     165        rows.append(row) 
     166 
     167    return rows 
     168 
     169def addSimpleParametersToModel(parameters, is2D, model=None, view=None): 
    163170    """ 
    164171    Update local ModelModel with sasmodel parameters (non-dispersed, non-magnetic) 
     
    169176        params = parameters.iq_parameters 
    170177 
     178    rows = [] 
    171179    for param in params: 
    172180        # Create the top level, checkable item 
     
    189197 
    190198        # Append to the model and use the combobox, if required 
    191         model.appendRow(row) 
    192         if cbox is not None: 
    193             view.setIndexWidget(item2.index(), cbox) 
     199        if None not in (model, view): 
     200            model.appendRow(row) 
     201            if cbox: 
     202                view.setIndexWidget(item2.index(), cbox) 
     203        rows.append(row) 
     204 
     205    return rows 
    194206 
    195207def markParameterDisabled(model, row): 
     
    271283    model.header_tooltips = copy.copy(poly_header_error_tooltips) 
    272284 
    273 def addShellsToModel(parameters, model, view, index): 
     285def addShellsToModel(parameters, model, index, view=None): 
    274286    """ 
    275287    Find out multishell parameters and update the model with the requested number of them 
     
    277289    multishell_parameters = getIterParams(parameters) 
    278290 
     291    rows = [] 
    279292    for i in range(index): 
    280293        for par in multishell_parameters: 
     
    309322            cbox = createFixedChoiceComboBox(par, row) 
    310323 
    311             # Append to the model and use the combobox, if required 
     324            # Always append to the model 
    312325            model.appendRow(row) 
    313             if cbox is not None: 
     326 
     327            # Apply combobox if required 
     328            if None not in (view, cbox): 
    314329                view.setIndexWidget(item2.index(), cbox) 
     330 
     331            rows.append(row) 
     332 
     333    return rows 
    315334 
    316335def calculateChi2(reference_data, current_data): 
  • src/sas/qtgui/Perspectives/Fitting/FittingWidget.py

    r88ada06 ra758043  
    20512051        # Update the QModel 
    20522052        FittingUtilities.addParametersToModel( 
    2053                 self._model_model, 
    2054                 self.lstParams, 
    20552053                self.model_parameters, 
    20562054                self.kernel_module, 
    2057                 self.is2D) 
     2055                self.is2D, 
     2056                self._model_model, 
     2057                self.lstParams) 
    20582058 
    20592059        # Update the counter used for multishell display 
     
    20762076        # Update the QModel 
    20772077        FittingUtilities.addSimpleParametersToModel( 
     2078                structure_parameters, 
     2079                self.is2D, 
    20782080                self._model_model, 
    2079                 self.lstParams, 
    2080                 structure_parameters, 
    2081                 self.is2D) 
     2081                self.lstParams) 
    20822082 
    20832083        # Any parameters removed from the structure factor when producing the product model, e.g. radius_effective, must 
     
    27122712                self.model_parameters, 
    27132713                self._model_model, 
    2714                 self.lstParams, 
    2715                 index) 
     2714                index, 
     2715                self.lstParams) 
    27162716 
    27172717        self.current_shell_displayed = index 
Note: See TracChangeset for help on using the changeset viewer.