Ignore:
Timestamp:
May 11, 2017 2:53:22 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:
d60da0c
Parents:
672b8ab
Message:

Code review fixes SASVIEW-588
Pylint related fixes in Perspectives/Fitting?

File:
1 edited

Legend:

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

    r672b8ab r1bc27f1  
    1 import sys 
    21import json 
    32import os 
    4 import numpy as np 
    53from collections import defaultdict 
    64from itertools import izip 
     
    97import traceback 
    108from twisted.internet import threads 
     9import numpy as np 
    1110 
    1211from PyQt4 import QtGui 
     
    2827from sas.sasgui.perspectives.fitting.utils import get_weight 
    2928 
    30 from UI.FittingWidgetUI import Ui_FittingWidgetUI 
     29from sas.qtgui.Perspectives.Fitting.UI.FittingWidgetUI import Ui_FittingWidgetUI 
    3130from sas.qtgui.Perspectives.Fitting.FittingLogic import FittingLogic 
    3231from sas.qtgui.Perspectives.Fitting import FittingUtilities 
    33 from SmearingWidget import SmearingWidget 
    34 from OptionsWidget import OptionsWidget 
    35 from FitPage import FitPage 
     32from sas.qtgui.Perspectives.Fitting.SmearingWidget import SmearingWidget 
     33from sas.qtgui.Perspectives.Fitting.OptionsWidget import OptionsWidget 
     34from sas.qtgui.Perspectives.Fitting.FitPage import FitPage 
    3635 
    3736TAB_MAGNETISM = 4 
     
    4544    Main widget for selecting form and structure factor models 
    4645    """ 
    47     def __init__(self, parent=None, data=None, id=1): 
     46    def __init__(self, parent=None, data=None, tab_id=1): 
    4847 
    4948        super(FittingWidget, self).__init__() 
     
    8180 
    8281        # Which tab is this widget displayed in? 
    83         self.tab_id = id 
     82        self.tab_id = tab_id 
    8483 
    8584        # Which shell is being currently displayed? 
     
    9897        layout = QtGui.QGridLayout() 
    9998        self.options_widget = OptionsWidget(self, self.logic) 
    100         layout.addWidget(self.options_widget)  
     99        layout.addWidget(self.options_widget) 
    101100        self.tabOptions.setLayout(layout) 
    102101 
     
    104103        layout = QtGui.QGridLayout() 
    105104        self.smearing_widget = SmearingWidget(self) 
    106         layout.addWidget(self.smearing_widget)  
     105        layout.addWidget(self.smearing_widget) 
    107106        self.tabResolution.setLayout(layout) 
    108107 
    109108        # Define bold font for use in various controls 
    110         self.boldFont=QtGui.QFont() 
     109        self.boldFont = QtGui.QFont() 
    111110        self.boldFont.setBold(True) 
    112111 
     
    315314        """ 
    316315        msg = 'Model description:\n' 
    317         info = "Info" 
    318316        if self.kernel_module is not None: 
    319317            if str(self.kernel_module.description).rstrip().lstrip() == '': 
     
    358356            self.createDefaultDataset() 
    359357 
    360         state = self.currentState() 
     358        # TODO: update state stack 
     359        #state = self.currentState() 
    361360 
    362361    def onSelectStructureFactor(self): 
     
    403402        # Retrieve the list of models 
    404403        model_list = self.master_category_dict[category] 
    405         models = [] 
    406404        # Populate the models combobox 
    407405        self.cbModel.addItems(sorted([model for (model, _) in model_list])) 
     
    538536        if res.fitness is None or \ 
    539537            not np.isfinite(res.fitness) or \ 
    540             np.any(res.pvec == None) or \ 
     538            np.any(res.pvec is None) or \ 
    541539            not np.all(np.isfinite(res.pvec)): 
    542540            msg = "Fitting did not converge!!!" 
     
    638636        # Regardless of previous state, this should now be `plot show` functionality only 
    639637        self.cmdPlot.setText("Show Plot") 
     638        if not self.data_is_loaded: 
     639            self.recalculatePlotData() 
    640640        self.showPlot() 
    641641 
     
    690690        elif self.log_points: 
    691691            qmin = -10.0 if self.q_range_min < 1.e-10 else np.log10(self.q_range_min) 
    692             qmax =  10.0 if self.q_range_max > 1.e10 else np.log10(self.q_range_max) 
     692            qmax = 10.0 if self.q_range_max > 1.e10 else np.log10(self.q_range_max) 
    693693            interval = np.logspace(start=qmin, stop=qmax, num=self.npts, endpoint=True, base=10.0) 
    694694        else: 
    695695            interval = np.linspace(start=self.q_range_min, stop=self.q_range_max, 
    696                     num=self.npts, endpoint=True) 
     696                                   num=self.npts, endpoint=True) 
    697697        self.logic.createDefault1dData(interval, self.tab_id) 
    698698 
     
    754754        """ 
    755755        Adds weighting contribution to fitting data 
    756         #""" 
     756        """ 
    757757        # Send original data for weighting 
    758758        weight = get_weight(data=data, is2d=self.is2D, flag=self.weighting) 
     
    856856        self.kernel_module.params[parameter_name] = value 
    857857 
     858        # TODO: update min/max based on property_name 
    858859        # min/max to be changed in self.kernel_module.details[parameter_name] = ['Ang', 0.0, inf] 
    859860        # magnetic params in self.kernel_module.details['M0:parameter_name'] = value 
     
    882883        self._model_model.blockSignals(True) 
    883884        # Convert to proper indices and set requested enablement 
    884         items = [self._model_model.item(row, 0).setCheckState(status) for row in rows] 
     885        _ = [self._model_model.item(row, 0).setCheckState(status) for row in rows] 
    885886        self._model_model.blockSignals(False) 
    886887 
     
    966967        # Awful API to a backend method. 
    967968        method = self.methodCalculateForData()(data=self.data, 
    968                               model=self.kernel_module, 
    969                               page_id=0, 
    970                               qmin=self.q_range_min, 
    971                               qmax=self.q_range_max, 
    972                               smearer=None, 
    973                               state=None, 
    974                               weight=None, 
    975                               fid=None, 
    976                               toggle_mode_on=False, 
    977                               completefn=None, 
    978                               update_chisqr=True, 
    979                               exception_handler=self.calcException, 
    980                               source=None) 
     969                                               model=self.kernel_module, 
     970                                               page_id=0, 
     971                                               qmin=self.q_range_min, 
     972                                               qmax=self.q_range_max, 
     973                                               smearer=None, 
     974                                               state=None, 
     975                                               weight=None, 
     976                                               fid=None, 
     977                                               toggle_mode_on=False, 
     978                                               completefn=None, 
     979                                               update_chisqr=True, 
     980                                               exception_handler=self.calcException, 
     981                                               source=None) 
    981982 
    982983        calc_thread = threads.deferToThread(method.compute) 
Note: See TracChangeset for help on using the changeset viewer.