Changeset d4d4c8a in sasview for sansguiframe

Timestamp:

Jul 12, 2012 2:19:08 PM (12 years ago)

Author:

Jae Cho <jhjcho@…>

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:

4cc25e9

Parents:

5e5704d

Message:

datapanel: second clean up for pylint

File:

• sansguiframe/src/sans/guiframe/data_panel.py (modified) (15 diffs)

sansguiframe/src/sans/guiframe/data_panel.py

@@ -100 +100 @@
         """
         # Get the item data
-        data1 = self.GetItemText(item1)
-        data2 = self.GetItemText(item2)
+        data_1 = self.GetItemText(item1)
+        data_2 = self.GetItemText(item2)
         # Compare the item data
-        if data1 < data2:
+        if data_1 < data_2:
             return -1
-        elif data1 > data2:
+        elif data_1 > data_2:
             return 1
         else:
@@ -239 +239 @@
         wx.EVT_COMBOBOX(self.selection_cbox, -1, self._on_selection_type)
         self.sizer5.AddMany([(select_txt, 0, wx.ALL,5), 
-                            (self.selection_cbox,0, wx.ALL,5)])
+                            (self.selection_cbox, 0, wx.ALL,5)])
         self.enable_selection()
         
@@ -403 +403 @@
         """
         data = None
-        selection = event.GetSelection()
+        #selection = event.GetSelection()
         id, _, _ = self.FindFocus().GetSelection().GetData()
         data_list, theory_list = \
@@ -467 +467 @@
         """
         data = self._get_data_selection(event)
-        path = None
+        #path = None
         default_name = data.name
         if default_name.count('.') > 0:
@@ -541 +541 @@
             id, data_class_name, _ = \
                             self.tree_ctrl_theory.GetSelection().GetData()
-            data_list, _ = \
-                            self.parent.get_data_manager().get_by_id(id_list=[id])
+            _, _ = self.parent.get_data_manager().get_by_id(id_list=[id])
         except:
             return
@@ -667 +666 @@
                         data_ctrl_list =  self.list_cb_data[state_id]
                         #This state is already display replace it contains
-                        data_c, d_i_c, d_t_c, r_n_c,  i_c_c, p_c_c, d_p_c, t_c \
+                        data_c, d_i_c, d_t_c, r_n_c,  i_c_c, p_c_c, d_p_c, _ \
                                 = data_ctrl_list
                         self.tree_ctrl.SetItemText(data_c, data_name) 
@@ -774 +773 @@
             theory_list_ctrl = {}
             for theory_id, item in theory_list.iteritems():
-                theory_data, theory_state = item
+                theory_data, _ = item
                 if theory_data is not None:
                     name = theory_data.name
@@ -897 +896 @@
         #remove theory  references independently of data
         for key in theory_key:
-            for t_key, theory_dict in self.list_cb_theory.iteritems():
+            for _, theory_dict in self.list_cb_theory.iteritems():
                 if key in theory_dict:
                     for  key, value in theory_dict.iteritems():
@@ -1119 +1118 @@
     def check_theory_to_freeze(self):
         """
+        Check_theory_to_freeze
         """
     def enable_freeze(self):
@@ -1288 +1288 @@
    
     
-from sans.guiframe.dataFitting import Data1D
-from sans.guiframe.dataFitting import Data2D, Theory1D
+from sans.guiframe.dataFitting import Theory1D
 from sans.guiframe.data_state import DataState
 
@@ -1332 +1331 @@
         path = "path1"
         state1 = State()
-        data_list1['1'] = set_data_state(data1, path,theory, state1)
+        data_list1['1'] = set_data_state(data1, path, theory, state1)
         #state 2
         data1 = Data2D()
@@ -1343 +1342 @@
         #state3
         state1 = State()
-        data_list1['2'] = set_data_state(data1, path,theory, state1)
+        data_list1['2'] = set_data_state(data1, path, theory, state1)
         data1 = Data1D()
         data1.id = 3
@@ -1357 +1356 @@
         process1 = data1.process[len(data1.process)-1]
         process1.data = "07/22/2010"
-        data_list1['4'] = set_data_state(data1, path,theory, state1)
+        data_list1['4'] = set_data_state(data1, path, theory, state1)
         #state 4
         temp_data_list = {}
         data1.name = "data5 erasing data2"
-        temp_data_list['4'] = set_data_state(data1, path,theory, state1)
+        temp_data_list['4'] = set_data_state(data1, path, theory, state1)
         #state 5
         data1 = Data2D()
@@ -1367 +1366 @@
         data1.id = 5
         data1.append_empty_process()
-        process1 = data.process[len(data1.process)-1]
+        process1 = data1.process[len(data1.process)-1]
         process1.data = "07/01/2010"
         theory = Theory1D()
@@ -1373 +1372 @@
         path = "path2"
         state1 = State()
-        dstate= set_data_state(data, path,theory, state1)
+        dstate1= set_data_state(data1, path, theory, state1)
         theory = Theory1D()
         theory.id = 6
         theory.name = "CoreShell"
-        dstate.set_theory(theory)
+        dstate1.set_theory(theory)
         theory = Theory1D()
         theory.id = 6
         theory.name = "CoreShell replacing coreshell in data3"
-        dstate.set_theory(theory)
-        data_list1['3'] = dstate
+        dstate1.set_theory(theory)
+        data_list1['3'] = dstate1
         #state 6
         data_list1['6'] = set_data_state(None, path, theory, state1)