Changeset 65d1d04 in sasview for src/sas/sasgui/perspectives/fitting

Timestamp:

Jan 11, 2017 4:19:17 AM (8 years ago)

Author:

wojciech

Children:

1b061a31

Parents:

4320104

Message:

Minor code changes to make pylint happy

File:

• src/sas/sasgui/perspectives/fitting/gpu_options.py (modified) (12 diffs)

src/sas/sasgui/perspectives/fitting/gpu_options.py

@@ -17 +17 @@
 
 class CustomMessageBox(wx.Dialog):
+    """
+    Custom message box for OpenCL results
+    """
     def __init__(self, parent, msg, title):
 
@@ -26 +29 @@
 
         self.text = wx.TextCtrl(self, -1, size=(500, 400),
-                           style=wx.TE_MULTILINE|wx.TE_READONLY)
+                                style=wx.TE_MULTILINE|wx.TE_READONLY)
         self.text.SetValue(msg)
         self.text.SetBackgroundColour(self.GetBackgroundColour())
@@ -40 +43 @@
         self.vbox.Add(self.panel, 0, wx.ALL, 10)
 
-        self.message_text = wx.StaticText(self, -1,"If tests fail on OpenCL devices, "
-                                "please select No OpenCL option.\n\n"
-                                "In case of large number of failing tests, "
-                                "please consider sending\n"
-                                "above report to help@sasview.org.")
+        self.message_text = wx.StaticText(self, -1, "If tests fail on OpenCL devices, "
+                                                    "please select No OpenCL option.\n\n"
+                                                    "In case of large number of failing tests, "
+                                                    "please consider sending\n"
+                                                    "above report to help@sasview.org.")
 
         self.vbox.Add(self.message_text, 0, wx.LEFT|wx.EXPAND|wx.ADJUST_MINSIZE, 10)
@@ -87 +90 @@
         self.buttons = []
         #Check if SAS_OPENCL is already set as enviromental variable
-        self.sas_opencl = os.environ.get("SAS_OPENCL","")
+        self.sas_opencl = os.environ.get("SAS_OPENCL", "")
 
         for clopt in clinfo:
@@ -134 +137 @@
         self.Bind(wx.EVT_BUTTON, self.on_help, help_btn)
 
-        test_text = wx.StaticText(self, -1,"WARNING: Running tests can take a few minutes!")
+        test_text = wx.StaticText(self, -1, "WARNING: Running tests can take a few minutes!")
         self.vbox.Add(test_text, 0, wx.LEFT|wx.EXPAND|wx.ADJUST_MINSIZE, 10)
 
@@ -170 +173 @@
                 for device in devices:
                     if len(devices) > 1 and len(platforms) > 1:
-                        combined_index = ":".join([str(p_index),str(d_index)])
+                        combined_index = ":".join([str(p_index), str(d_index)])
                     elif len(platforms) > 1:
                         combined_index = str(p_index)
                     else:
                         combined_index = str(d_index)
-                    #combined_index = ":".join([str(p_index),str(d_index)]) \
-                    #    if len(platforms) > 1 else str(d_index)
-                    clinfo.append((combined_index, ":".join([platform.name,device.name])))
+                    clinfo.append((combined_index, ":".join([platform.name, device.name])))
                     d_index += 1
                 p_index += 1
@@ -183 +184 @@
             warnings.warn("pyopencl import failed. Using only CPU computations")
 
-        clinfo.append(("None","No OpenCL"))
+        clinfo.append(("None", "No OpenCL"))
         return clinfo
 
@@ -211 +212 @@
         else:
             if "SAS_OPENCL" in os.environ:
-                del(os.environ["SAS_OPENCL"])
+                del os.environ["SAS_OPENCL"]
 
         #Sasmodels kernelcl doesn't exist when initiated with None
@@ -227 +228 @@
             btn.SetValue(0)
 
-        self.sas_opencl=None
+        self.sas_opencl = None
 
     def on_test(self, event):
@@ -246 +247 @@
         else:
             if "SAS_OPENCL" in os.environ:
-                del(os.environ["SAS_OPENCL"])
+                del os.environ["SAS_OPENCL"]
 
         #Sasmodels kernelcl doesn't exist when initiated with None
@@ -263 +264 @@
             env = environment()
             clinfo = [(ctx.devices[0].platform.vendor,
-                      ctx.devices[0].platform.version,
-                      ctx.devices[0].vendor,
-                      ctx.devices[0].name,
-                      ctx.devices[0].version)
-                    for ctx in env.context]
+                       ctx.devices[0].platform.version,
+                       ctx.devices[0].vendor,
+                       ctx.devices[0].name,
+                       ctx.devices[0].version)
+                      for ctx in env.context]
         except ImportError:
             clinfo = None
@@ -295 +296 @@
             msg += 'Failing tests: '
             msg += json.dumps(info['failing tests'])
-            msg +="\n"
-        else:
-            msg+="All tests passed!\n"
-
-        msg +="\nPlatform Details:\n\n"
+            msg += "\n"
+        else:
+            msg += "All tests passed!\n"
+
+        msg += "\nPlatform Details:\n\n"
         msg += "Sasmodels version: "
         msg += info['version']+"\n"