Changeset b0ab6cb in sasview for calculatorview/perspectives/calculator/collimation_editor.py

Timestamp:

Oct 27, 2010 4:41:30 PM (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:

6cda91f

Parents:

c4d6900

Message:

working with pylint

File:

• calculatorview/perspectives/calculator/collimation_editor.py (modified) (11 diffs)

calculatorview/perspectives/calculator/collimation_editor.py

@@ -1 @@
-
+"""
+"""
 import wx
 import sys
@@ -28 +29 @@
         """
         """
-        kwds['size'] =(PANEL_WIDTH, PANEL_HEIGHT)
+        kwds['size'] = (PANEL_WIDTH, PANEL_HEIGHT)
         kwds['title'] = "Collimation Editor"
         wx.Dialog.__init__(self, parent=parent, *args, **kwds)
@@ -37 +38 @@
         self._notes = ""
         self._description = "Edit collimation"
+        #layout attributes
+        self.main_sizer = None
+        self.box_collimation = None
+        self.boxsizer_collimation = None
+        
+        
         self._do_layout()
         self.set_values()
@@ -45 +52 @@
         """
         self.main_sizer = wx.BoxSizer(wx.VERTICAL)
-        self.box_collimation = wx.StaticBox(self, -1,str("Edit Selected Collimation"))
-        self.boxsizer_collimation = wx.StaticBoxSizer(self.box_collimation, wx.VERTICAL)
+        self.box_collimation = wx.StaticBox(self, -1,
+                                            str("Edit Selected Collimation"))
+        self.boxsizer_collimation = wx.StaticBoxSizer(self.box_collimation,
+                                                       wx.VERTICAL)
         
         collimation_box = wx.StaticBox(self, -1, "Edit Number of Collimations")
@@ -80 +89 @@
         
         self.bt_remove_collimation = wx.Button(self, -1, "Remove")
-        self.bt_remove_collimation.SetToolTipString("Remove data's collimation.")
+        hint = "Remove data's collimation."
+        self.bt_remove_collimation.SetToolTipString(hint)
         self.bt_remove_collimation.Bind(wx.EVT_BUTTON, self.remove_collimation)
       
-        self.collimation_button_sizer.AddMany([(collimation_name_txt, 0, wx.LEFT, 15),
+        self.collimation_button_sizer.AddMany([(collimation_name_txt, 0,
+                                                 wx.LEFT, 15),
                                      (self.collimation_cbox, 0, wx.LEFT, 5),
                                      (self.bt_add_collimation, 0, wx.LEFT, 10),
-                                     (self.bt_remove_collimation, 0, wx.LEFT, 5)])
+                                     (self.bt_remove_collimation,
+                                       0, wx.LEFT, 5)])
         collimation_hint_txt = 'No collimation is available for this data.'
         self.collimation_txt = wx.StaticText(self, -1, collimation_hint_txt) 
         self.collimation_hint_sizer.Add(self.collimation_txt, 0, wx.LEFT, 10)
-        self.collimation_sizer.AddMany([(self.collimation_button_sizer, 0, wx.ALL, 10),
-                                     (self.collimation_hint_sizer, 0, wx.ALL, 10)])
+        self.collimation_sizer.AddMany([(self.collimation_button_sizer,
+                                          0, wx.ALL, 10),
+                                     (self.collimation_hint_sizer,
+                                       0, wx.ALL, 10)])
      
         self.fill_collimation_combox()
@@ -115 +129 @@
         self.length_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH, 20), style=0)   
         length_unit_txt = wx.StaticText(self, -1, 'Unit: ') 
-        self.length_unit_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH, 20),style=0)  
+        self.length_unit_tcl = wx.TextCtrl(self, -1, size=(_BOX_WIDTH, 20),
+                                           style=0)  
         self.length_sizer.AddMany([(length_txt, 0, wx.LEFT|wx.RIGHT, 10),
                                      (self.length_tcl, 0, wx.LEFT, 12),
@@ -165 +180 @@
         self.aperture_txt = wx.StaticText(self, -1, aperture_hint_txt) 
         self.aperture_hint_sizer.Add(self.aperture_txt, 0, wx.LEFT, 10)
-        self.aperture_sizer.AddMany([(self.aperture_button_sizer, 0, wx.ALL, 10),
+        self.aperture_sizer.AddMany([(self.aperture_button_sizer,
+                                       0, wx.ALL, 10),
                                      (self.aperture_hint_sizer, 0, wx.ALL, 10)])
         self.fill_aperture_combox()
         self.enable_aperture()
     
-    def _do_layout(self, data=None):
+    def _do_layout(self):
         """
         Draw the current panel
@@ -265 +281 @@
     def on_select_collimation(self, event):
         """
-        fill the control on the panel according to the current  selected collimation 
+        fill the control on the panel according to 
+        the current  selected collimation 
         """
         self.set_values()
@@ -275 +292 @@
         Enable /disable widgets related to collimation
         """
-        if self._collimation is not None and self.collimation_cbox.GetCount() > 0:
+        if self._collimation is not None and \
+            self.collimation_cbox.GetCount() > 0:
             self.collimation_cbox.Enable()
             self.bt_remove_collimation.Enable()
             n_collimation = self.collimation_cbox.GetCount()
-            collimation_hint_txt = 'collimations available: %s '%str(n_collimation)
+            collimation_hint_txt = "collimations"
+            collimation_hint_txt += " available: %s "% str(n_collimation)
             if len(self._collimation) > 0:
                 self.bt_remove_collimation.Enable()
@@ -491 +510 @@
                     collimation.length  = float(length)
             else:
-                self._notes += "Error: Expected a float for collimation length "
-                self._notes += "won't changes length from "
+                self._notes += "Error: Expected a float for collimation length"
+                self._notes += " won't changes length from "
                 self._notes += "%s to %s"%(collimation.length, length)
         #change length  unit
@@ -518 +537 @@
         self.set_values()
         if self.manager is not None:
-             self.manager.set_collimation(self._collimation)
+            self.manager.set_collimation(self._collimation)
        
  
-if __name__ =="__main__":
+if __name__ == "__main__":
 
     app  = wx.App()