Changeset 6d48919 in sasview for calculatorview/perspectives


Ignore:
Timestamp:
Oct 5, 2010 9:38:03 AM (14 years ago)
Author:
Gervaise Alina <gervyh@…>
Branches:
master, 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, costrafo411, magnetic_scatt, release-4.1.1, release-4.1.2, release-4.2.2, release_4.0.1, ticket-1009, ticket-1094-headless, ticket-1242-2d-resolution, ticket-1243, ticket-1249, ticket885, unittest-saveload
Children:
35aface
Parents:
ec1c554
Message:

working on pylint warnings

Location:
calculatorview/perspectives/calculator
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • calculatorview/perspectives/calculator/__init__.py

    r85a24d9 r6d48919  
    55from distutils.filelist import findall 
    66 
     7N_DIR = 12 
    78def get_data_path(media): 
    89    """ 
    … …  
    1920    path = os.path.dirname(__file__) 
    2021    #Look for maximum n_dir up of the current dir to find media 
    21     n_dir = 12 
    22     for i in range(n_dir): 
     22    
     23    #for i in range(n_dir): 
     24    i = 0 
     25    while(i < N_DIR): 
    2326        path, _ = os.path.split(path) 
    2427        media_path = os.path.join(path, media) 
    … …  
    2831                 return module_media_path 
    2932             return media_path 
     33        i += 1 
    3034    
    3135    raise RuntimeError('Could not find calculator media files') 
    … …  
    3943 
    4044    """ 
    41     data_files =[] 
     45    data_files = [] 
    4246    path = get_data_path(media="media") 
    4347    for f in findall(path): 

  • calculatorview/perspectives/calculator/aperture_editor.py

    r6137b150 r6d48919  
    1919     
    2020class ApertureDialog(wx.Dialog): 
    21     def __init__(self, parent=None, manager=None,aperture=None, *args, **kwds): 
     21    def __init__(self, parent=None, manager=None, aperture=None, *args, **kwds): 
    2222        """ 
    2323        Dialog allows to enter values for aperture 
    2424        """ 
    25         kwds['size'] =(PANEL_WIDTH, PANEL_HEIGHT) 
     25        kwds['size'] = (PANEL_WIDTH, PANEL_HEIGHT) 
    2626        kwds['title'] = "Aperture Editor" 
    2727        wx.Dialog.__init__(self, parent=parent, *args, **kwds) 
    … …  
    4040        """ 
    4141        self.main_sizer = wx.BoxSizer(wx.VERTICAL) 
    42         self.box_aperture = wx.StaticBox(self, -1,str("Aperture")) 
    43         self.boxsizer_aperture = wx.StaticBoxSizer(self.box_aperture, wx.VERTICAL) 
     42        self.box_aperture = wx.StaticBox(self, -1, str("Aperture")) 
     43        self.boxsizer_aperture = wx.StaticBoxSizer(self.box_aperture,  
     44                                                   wx.VERTICAL) 
    4445        
    4546        self.name_sizer = wx.BoxSizer(wx.HORIZONTAL) 
    … …  
    5657        #Aperture name [string] 
    5758        aperture_name_txt = wx.StaticText(self, -1, 'Aperture Name : ')   
    58         self.aperture_name_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH*5, 20), style=0)  
     59        self.aperture_name_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH*5, 20), 
     60                                              style=0)  
    5961        self.name_sizer.AddMany([(aperture_name_txt, 0, wx.LEFT|wx.RIGHT, 10), 
    6062                                       (self.aperture_name_tcl, 0, wx.EXPAND)]) 
    … …  
    7577        #Aperture distance [float] 
    7678        distance_txt = wx.StaticText(self, -1, 'Distance:')  
    77         self.distance_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH, 20), style=0)    
     79        self.distance_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH, 20),  
     80                                        style=0)    
    7881        distance_unit_txt = wx.StaticText(self, -1, 'Unit: ')  
    79         self.distance_unit_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH, 20),style=0)   
     82        self.distance_unit_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH, 20), 
     83                                             style=0)   
    8084        self.distance_sizer.AddMany([(distance_txt, 0, wx.LEFT|wx.RIGHT, 10), 
    8185                                     (self.distance_tcl, 0, wx.LEFT, 10), 
    82                                      (distance_unit_txt, 0, wx.LEFT|wx.RIGHT, 10), 
     86                                (distance_unit_txt, 0, wx.LEFT|wx.RIGHT, 10), 
    8387                                     (self.distance_unit_tcl, 0, wx.EXPAND)])   
    8488    def _layout_size_name(self): 
    … …  
    8892        # Size name [string] 
    8993        size_name_txt = wx.StaticText(self, -1, 'Size Name : ')   
    90         self.size_name_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH*5, 20), style=0)  
     94        self.size_name_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH*5, 20), 
     95                                          style=0)  
    9196        self.size_name_sizer.AddMany([(size_name_txt, 0, wx.LEFT|wx.RIGHT, 10), 
    9297                                       (self.size_name_tcl, 0, wx.EXPAND)]) 
    … …  
    99104        aperture_size_txt = wx.StaticText(self, -1, 'Size:')  
    100105        x_aperture_size_txt = wx.StaticText(self, -1, 'x = ')   
    101         self.x_aperture_size_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH,20), style=0)  
     106        self.x_aperture_size_tcl = wx.TextCtrl(self, -1,  
     107                                               size=(_BOX_WIDTH,20), style=0)  
    102108        y_aperture_size_txt = wx.StaticText(self, -1, 'y = ')   
    103         self.y_aperture_size_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH,20), style=0)  
     109        self.y_aperture_size_tcl = wx.TextCtrl(self, -1, 
     110                                                size=(_BOX_WIDTH,20), style=0)  
    104111        z_aperture_size_txt = wx.StaticText(self, -1, 'z = ')   
    105         self.z_aperture_size_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH,20), style=0)   
     112        self.z_aperture_size_tcl = wx.TextCtrl(self, -1, 
     113                                                size=(_BOX_WIDTH,20), style=0)   
    106114        aperture_size_unit_txt = wx.StaticText(self, -1, 'Unit: ')  
    107         self.aperture_size_unit_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH,20), style=0)   
    108         self.aperture_size_sizer.AddMany([(aperture_size_txt, 0, wx.LEFT|wx.RIGHT, 10), 
     115        self.aperture_size_unit_tcl = wx.TextCtrl(self, -1,  
     116                                                size=(_BOX_WIDTH,20), style=0)   
     117        self.aperture_size_sizer.AddMany([(aperture_size_txt, 
     118                                         0, wx.LEFT|wx.RIGHT, 10), 
    109119                                     (x_aperture_size_txt, 0, wx.LEFT, 17), 
    110                                      (self.x_aperture_size_tcl, 0, wx.RIGHT, 10), 
     120                                (self.x_aperture_size_tcl, 0, wx.RIGHT, 10), 
    111121                                     (y_aperture_size_txt, 0, wx.EXPAND), 
    112                                      (self.y_aperture_size_tcl, 0, wx.RIGHT, 10), 
     122                                (self.y_aperture_size_tcl, 0, wx.RIGHT, 10), 
    113123                                     (z_aperture_size_txt, 0, wx.EXPAND), 
    114                                      (self.z_aperture_size_tcl, 0, wx.RIGHT, 10), 
     124                                (self.z_aperture_size_tcl, 0, wx.RIGHT, 10), 
    115125                                     (aperture_size_unit_txt, 0, wx.EXPAND), 
    116                                      (self.aperture_size_unit_tcl, 0, wx.RIGHT, 10)]) 
     126                            (self.aperture_size_unit_tcl, 0, wx.RIGHT, 10)]) 
    117127         
    118128    def _layout_button(self):   
    … …  
    132142                                   (self.bt_close, 0, wx.LEFT, 10)]) 
    133143         
    134     def _do_layout(self, data=None): 
     144    def _do_layout(self ):#, data=None): 
    135145        """ 
    136146        Draw the current panel 
    … …  
    224234        if self._aperture.name != name: 
    225235            self._notes += "Change sample 's " 
    226             self._notes += "name from %s to %s \n"%(self._aperture.name, name) 
     236            self._notes += "name from %s to %s \n"% (self._aperture.name, name) 
    227237            self._aperture.name = name 
    228238             
    … …  
    235245        self._aperture.type = type 
    236246        self._notes += " Change type from" 
    237         self._notes += " %s to %s \n"%(self._aperture.type, type) 
     247        self._notes += " %s to %s \n"% (self._aperture.type, type) 
    238248         
    239249    def on_change_distance(self): 
    … …  
    250260                if self._aperture.distance != float(distance): 
    251261                    self._notes += "Change distance from " 
    252                     self._notes += "%s to %s \n"%(self._aperture.distance, distance) 
     262                    self._notes += "%s to %s \n"% (self._aperture.distance,  
     263                                                  distance) 
    253264                    self._aperture.distance  = float(distance) 
    254265            else: 
    255266                self._notes += "Error: Expected a float for distance  " 
    256267                self._notes += "won't changes distance from " 
    257                 self._notes += "%s to %s"%(self._aperture.distance, distance) 
     268                self._notes += "%s to %s"% (self._aperture.distance, distance) 
    258269        #change the distance unit 
    259270        unit = self.distance_unit_tcl.GetValue().lstrip().rstrip() 
    260271        if self._aperture.distance_unit != unit: 
    261272            self._notes += " Change distance 's unit from " 
    262             self._notes += "%s to %s"%(self._aperture.distance_unit, unit) 
     273            self._notes += "%s to %s"% (self._aperture.distance_unit, unit) 
    263274         
    264275    def on_change_size_name(self): 
    … …  
    270281        self._aperture.size_name = size_name 
    271282        self._notes += " Change size name from" 
    272         self._notes += " %s to %s \n"%(self._aperture.size_name, size_name) 
     283        self._notes += " %s to %s \n"% (self._aperture.size_name, size_name) 
    273284    
    274285    def on_change_size(self): 
    … …  
    284295                if self._aperture.size.x != float(x_aperture_size) : 
    285296                    self._notes += "Change x of aperture size from " 
    286                     self._notes += "%s to %s \n"%(self._aperture.size.x, x_aperture_size) 
     297                    self._notes += "%s to %s \n"% (self._aperture.size.x, 
     298                                                   x_aperture_size) 
    287299                    self._aperture.aperture_size.x  = float(x_aperture_size) 
    288300            else: 
    289                 self._notes += "Error: Expected a float for the aperture size 's x " 
     301                self._notes += "Error: Expected a" 
     302                self._notes += " float for the aperture size 's x " 
    290303                self._notes += "won't changes x aperture size from " 
    291                 self._notes += "%s to %s"%(self._aperture.size.x, x_aperture_size) 
     304                self._notes += "%s to %s"% (self._aperture.size.x,  
     305                                           x_aperture_size) 
    292306        #Change y coordinate 
    293307        y_aperture_size = self.y_aperture_size_tcl.GetValue().lstrip().rstrip() 
    … …  
    299313                if self._aperture.size.y != float(y_aperture_size): 
    300314                    self._notes += "Change y of aperture size from " 
    301                     self._notes += "%s to %s \n"%(self._aperture.size.y, y_aperture_size) 
     315                    self._notes += "%s to %s \n"% (self._aperture.size.y, 
     316                                                   y_aperture_size) 
    302317                    self._aperture.size.y  = float(y_aperture_size) 
    303318            else: 
    304                 self._notes += "Error: Expected a float for the aperture size's y " 
     319                self._notes += "Error: Expected a float for the" 
     320                self._notes += " aperture size's y " 
    305321                self._notes += "won't changes y aperture size from " 
    306                 self._notes += "%s to %s"%(self._aperture.size.y, y_aperture_size) 
     322                self._notes += "%s to %s"% (self._aperture.size.y, 
     323                                            y_aperture_size) 
    307324        #Change z coordinate 
    308325        z_aperture_size = self.z_aperture_size_tcl.GetValue().lstrip().rstrip() 
    … …  
    314331                if self._aperture.size.z != float(z_aperture_size): 
    315332                    self._notes += "Change z of aperture size from " 
    316                     self._notes += "%s to %s \n"%(self._aperture.size.z, 
     333                    self._notes += "%s to %s \n"% (self._aperture.size.z, 
    317334                                                   z_aperture_size) 
    318335                    self._aperture.size.z  = float(z_aperture_size) 
    … …  
    320337                self._notes += "Error: Expected a float for the offset 's x " 
    321338                self._notes += "won't changes z aperture size from " 
    322                 self._notes += "%s to %s"%(self._aperture.size.z, z_aperture_size) 
     339                self._notes += "%s to %s"% (self._aperture.size.z, 
     340                                            z_aperture_size) 
    323341        #change the aperture center unit 
    324342        unit = self.aperture_size_unit_tcl.GetValue().lstrip().rstrip() 
    325343        if self._aperture.size_unit != unit: 
    326344            self._notes += " Change aperture size's unit from " 
    327             self._notes += "%s to %s"%(self._aperture.size_unit, unit) 
     345            self._notes += "%s to %s"% (self._aperture.size_unit, unit) 
    328346            self._aperture.size_unit = unit 
    329347                  
    … …  
    340358        if self.manager is not None: 
    341359            self.manager.set_aperture(self._aperture) 
    342          
     360        if event is not None: 
     361            event.Skip() 
     362             
    343363    def on_click_cancel(self, event): 
    344364        """ 
    … …  
    349369        if self.manager is not None: 
    350370             self.manager.set_aperture(self._aperture) 
     371        if event is not None: 
     372            event.Skip() 
    351373  
    352 if __name__ =="__main__": 
     374if __name__ == "__main__": 
    353375    
    354376    app  = wx.App() 
    … …  
    357379    # Load data  
    358380    from DataLoader.data_info import Aperture 
    359     aperture = Aperture() 
    360     dlg = ApertureDialog(aperture=aperture) 
     381    ap = Aperture() 
     382    dlg = ApertureDialog(aperture=ap) 
    361383    dlg.ShowModal() 
    362384    app.MainLoop() 
Note: See TracChangeset for help on using the changeset viewer.