Changeset 8b21fa7 in sasview for src/sas/perspectives/pr/pr_widgets.py

Timestamp:

Mar 6, 2015 2:05:50 PM (10 years ago)

Author:

Doucet, Mathieu <doucetm@…>

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:

b1e609c

Parents:

c1c14ba

Message:

pylint fixes

File:

• src/sas/perspectives/pr/pr_widgets.py (modified) (12 diffs)

src/sas/perspectives/pr/pr_widgets.py

@@ -3 +3 @@
 #This software was developed by the University of Tennessee as part of the
 #Distributed Data Analysis of Neutron Scattering Experiments (DANSE)
-#project funded by the US National Science Foundation. 
+#project funded by the US National Science Foundation.
 #
 #See the license text in license.txt
@@ -26 +26 @@
         ScrolledPanel.__init__(self, *args, **kwds)
         self.SetupScrolling()
-        
-        
+
+
 class PrTextCtrl(wx.TextCtrl):
     """
@@ -34 +34 @@
     """
     def __init__(self, *args, **kwds):
-        
+
         wx.TextCtrl.__init__(self, *args, **kwds)
-        
+
         ## Set to True when the mouse is clicked while the whole string is selected
         self.full_selection = False
@@ -49 +49 @@
         Catch when the text control is set in focus to highlight the whole
         text if necessary
-        
+
         :param event: mouse event
-        
+
         """
         event.Skip()
         self.full_selection = True
-        
+
     def _highlight_text(self, event):
         """
         Highlight text of a TextCtrl only of no text has be selected
-        
+
         :param event: mouse event
-        
+
         """
         # Make sure the mouse event is available to other listeners
         event.Skip()
-        control  = event.GetEventObject()
+        control = event.GetEventObject()
         if self.full_selection:
             self.full_selection = False
             # Check that we have a TextCtrl
             if issubclass(control.__class__, wx.TextCtrl):
-                # Check whether text has been selected, 
+                # Check whether text has been selected,
                 # if not, select the whole string
                 (start, end) = control.GetSelection()
-                if start==end:
-                    control.SetSelection(-1,-1)
+                if start == end:
+                    control.SetSelection(-1, -1)
 
 class OutputTextCtrl(wx.TextCtrl):
     """
     Text control used to display outputs.
-    No editing allowed. The background is 
+    No editing allowed. The background is
     grayed out. User can't select text.
     """
@@ -88 +88 @@
         self.SetEditable(False)
         self.SetBackgroundColour(self.GetParent().GetBackgroundColour())
-        
+
         # Bind to mouse event to avoid text highlighting
         # The event will be skipped once the call-back
         # is called.
         self.Bind(wx.EVT_MOUSE_EVENTS, self._click)
-        
+
     def _click(self, event):
         """
         Prevent further handling of the mouse event
         by not calling Skip().
-        """ 
+        """
         pass
-        
+
 
 class DataFileTextCtrl(OutputTextCtrl):
@@ -106 +106 @@
     Text control used to display only the file name
     given a full path.
-     
+
     :TODO: now that we no longer choose the data file from the panel,
         it's no longer necessary to pass around the file path. That code
-        should be refactored away and simplified. 
+        should be refactored away and simplified.
     """
     def __init__(self, *args, **kwds):
@@ -116 +116 @@
         OutputTextCtrl.__init__(self, *args, **kwds)
         self._complete_path = None
-    
+
     def SetValue(self, value):
         """
@@ -122 +122 @@
         """
         self._complete_path = str(value)
-        file = os.path.basename(self._complete_path)
-        OutputTextCtrl.SetValue(self, file)
-        
+        file_path = os.path.basename(self._complete_path)
+        OutputTextCtrl.SetValue(self, file_path)
+
     def GetValue(self):
         """
@@ -130 +130 @@
         """
         return self._complete_path
-    
-
-            
-            
+
+
 def load_error(error=None):
     """
     Pop up an error message.
-    
+
     :param error: details error message to be displayed
     """
     message = "The data file you selected could not be loaded.\n"
     message += "Make sure the content of your file is properly formatted.\n\n"
-    
+
     if error is not None:
         message += "When contacting the DANSE team, mention the"
         message += " following:\n%s" % str(error)
-    
+
     dial = wx.MessageDialog(None, message, 'Error Loading File',
                             wx.OK | wx.ICON_EXCLAMATION)
-    dial.ShowModal()    
-
-    
+    dial.ShowModal()
+
+
 class DataDialog(wx.Dialog):
     """
@@ -166 +164 @@
         self._choice_sizer = wx.GridBagSizer(5, 5)
         self._panel = DialogPanel(self, style=wx.RAISED_BORDER,
-                               size=(WIDTH-20, HEIGHT/3))
-       
+                                  size=(WIDTH - 20, HEIGHT / 3))
+
         self.__do_layout(data_list, text=text)
-        
-        
+
     def __do_layout(self, data_list, text=''):
         """
         layout the dialog
         """
-        #if not data_list or len(data_list) <= 1:
-        #    return 
         #add text
         if text.strip() == "":
@@ -182 +177 @@
             text += "Please select only one Data.\n"
         text_ctrl = wx.TextCtrl(self, -1, str(text), style=wx.TE_MULTILINE,
-                                size=(-1, HEIGHT/3))
+                                size=(-1, HEIGHT / 3))
         text_ctrl.SetEditable(False)
-        self._sizer_txt.Add(text_ctrl , 1, wx.EXPAND|wx.ALL, 10)
+        self._sizer_txt.Add(text_ctrl, 1, wx.EXPAND | wx.ALL, 10)
         iy = 0
         ix = 0
-        rbox = wx.RadioButton(self._panel, -1, str(data_list[0].name), 
-                                  (10, 10), style= wx.RB_GROUP)
+        rbox = wx.RadioButton(self._panel, -1, str(data_list[0].name),
+                              (10, 10), style=wx.RB_GROUP)
         rbox.SetValue(True)
         self.list_of_ctrl.append((rbox, data_list[0]))
         self._choice_sizer.Add(rbox, (iy, ix), (1, 1),
-                         wx.LEFT|wx.EXPAND|wx.ADJUST_MINSIZE, 15)
+                               wx.LEFT | wx.EXPAND | wx.ADJUST_MINSIZE, 15)
         for i in range(1, len(data_list)):
             iy += 1
-            rbox = wx.RadioButton(self._panel, -1, 
+            rbox = wx.RadioButton(self._panel, -1,
                                   str(data_list[i].name), (10, 10))
             rbox.SetValue(False)
             self.list_of_ctrl.append((rbox, data_list[i]))
             self._choice_sizer.Add(rbox, (iy, ix),
-                           (1, 1), wx.LEFT|wx.EXPAND|wx.ADJUST_MINSIZE, 15)
+                                   (1, 1), wx.LEFT | wx.EXPAND | wx.ADJUST_MINSIZE, 15)
         self._panel.SetSizer(self._choice_sizer)
         #add sizer
-        self._sizer_button.Add((20, 20), 1, wx.EXPAND|wx.ADJUST_MINSIZE, 0)
+        self._sizer_button.Add((20, 20), 1, wx.EXPAND | wx.ADJUST_MINSIZE, 0)
         button_cancel = wx.Button(self, wx.ID_CANCEL, "Cancel")
         self._sizer_button.Add(button_cancel, 0,
-                          wx.LEFT|wx.RIGHT|wx.ADJUST_MINSIZE, 10)
-        button_OK = wx.Button(self, wx.ID_OK, "Ok")
-        button_OK.SetFocus()
-        self._sizer_button.Add(button_OK, 0,
-                                wx.LEFT|wx.RIGHT|wx.ADJUST_MINSIZE, 10)
+                               wx.LEFT | wx.RIGHT | wx.ADJUST_MINSIZE, 10)
+        button_ok = wx.Button(self, wx.ID_OK, "Ok")
+        button_ok.SetFocus()
+        self._sizer_button.Add(button_ok, 0,
+                               wx.LEFT | wx.RIGHT | wx.ADJUST_MINSIZE, 10)
         static_line = wx.StaticLine(self, -1)
-        
-        self._sizer_txt.Add(self._panel, 0, wx.EXPAND|wx.ALL, 5)
-        self._sizer_main.Add(self._sizer_txt, 0, wx.EXPAND|wx.ALL, 5)
+
+        self._sizer_txt.Add(self._panel, 0, wx.EXPAND | wx.ALL, 5)
+        self._sizer_main.Add(self._sizer_txt, 0, wx.EXPAND | wx.ALL, 5)
         self._sizer_main.Add(static_line, 0, wx.EXPAND, 0)
-        self._sizer_main.Add(self._sizer_button, 0, wx.EXPAND|wx.ALL, 10)
+        self._sizer_main.Add(self._sizer_button, 0, wx.EXPAND | wx.ALL, 10)
         self.SetSizer(self._sizer_main)
-        
+
     def get_data(self):
         """
@@ -226 +221 @@
             rbox, data = item
             if rbox.GetValue():
-                return data 
-
-    
-
- 
+                return data