Changeset 9863343 in sasview for src/sas/qtgui/Perspectives/Fitting/GPUOptions.py

Timestamp:

Oct 25, 2017 7:58:46 AM (7 years ago)

Author:

krzywon

Branches:

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

Children:

c31d41e7

Parents:

37be27f

Message:

Address issues outlined in SVCC-79.

File:

• src/sas/qtgui/Perspectives/Fitting/GPUOptions.py (modified) (9 diffs)

src/sas/qtgui/Perspectives/Fitting/GPUOptions.py

@@ -6 +6 @@
 import platform
 
+import sas.qtgui.Utilities.GuiUtils as GuiUtils
 from PyQt4 import QtGui, QtCore, QtWebKit
 from sas.qtgui.Perspectives.Fitting.UI.GPUOptionsUI import Ui_GPUOptions
@@ -38 +39 @@
         self.setupUi(self)
         self.addOpenCLOptions()
+        self.setFixedSize(self.size())
         self.createLinks()
 
@@ -61 +63 @@
             # Expand group and shift items down as more are added
             self.openCLCheckBoxGroup.resize(391, 60 + i)
-            self.label.setGeometry(QtCore.QRect(20, 90 + i, 391, 37))
+            self.warningMessage.setGeometry(QtCore.QRect(20, 90 + i, 391, 37))
             self.okButton.setGeometry(QtCore.QRect(20, 127 + i, 93, 28))
             self.resetButton.setGeometry(QtCore.QRect(120, 127 + i, 93, 28))
@@ -73 +75 @@
         Link user interactions to function calls
         """
-        self.testButton.clicked.connect(lambda: self.testButtonClicked())
-        self.helpButton.clicked.connect(lambda: self.helpButtonClicked())
+        self.testButton.clicked.connect(self.testButtonClicked)
+        self.helpButton.clicked.connect(self.helpButtonClicked)
         for item in self.openCLCheckBoxGroup.findChildren(QtGui.QCheckBox):
-            item.clicked.connect(lambda: self.checked())
+            item.clicked.connect(self.checked)
 
     def checked(self):
@@ -151 +153 @@
         }
 
-        msg_info = 'OpenCL tests results'
-
         msg = str(tests_completed) + ' tests completed.\n'
         if len(failures) > 0:
@@ -172 +172 @@
             msg += "\nOpenCL driver: "
             msg += json.dumps(info['opencl']) + "\n"
-        GPUTestResults(self, msg, msg_info)
+        GPUTestResults(self, msg)
 
     def helpButtonClicked(self):
@@ -178 +178 @@
         Open the help menu when the help button is clicked
         """
-        tree_location = "user/sasgui/perspectives/fitting/gpu_setup.html"
-        anchor = "#device-selection"
+        help_location = GuiUtils.HELP_DIRECTORY_LOCATION
+        help_location += "/user/sasgui/perspectives/fitting/gpu_setup.html"
+        help_location += "#device-selection"
         self.helpView = QtWebKit.QWebView()
-        help_location = tree_location + anchor
         self.helpView.load(QtCore.QUrl(help_location))
         self.helpView.show()
@@ -211 +211 @@
     OpenCL Dialog to modify the OpenCL options
     """
-    def __init__(self, parent, msg, title):
+    def __init__(self, parent, msg):
         super(GPUTestResults, self).__init__(parent)
         self.setupUi(self)
         self.resultsText.setText(_translate("GPUTestResults", msg, None))
+        self.setFixedSize(self.size())
         self.open()
 
@@ -223 +224 @@
     """
     clinfo = []
-    platforms = []
+    cl_platforms = []
     try:
         import pyopencl as cl
-        platforms = cl.get_platforms()
+        cl_platforms = cl.get_platforms()
     except ImportError:
         print("pyopencl import failed. Using only CPU computations")
 
     p_index = 0
-    for platform in platforms:
+    for cl_platform in cl_platforms:
         d_index = 0
-        devices = platform.get_devices()
-        for device in devices:
-            if len(devices) > 1 and len(platforms) > 1:
+        cl_platforms = cl_platform.get_devices()
+        for cl_platform in cl_platforms:
+            if len(cl_platforms) > 1 and len(cl_platforms) > 1:
                 combined_index = ":".join([str(p_index), str(d_index)])
-            elif len(platforms) > 1:
+            elif len(cl_platforms) > 1:
                 combined_index = str(p_index)
             else:
                 combined_index = str(d_index)
-            clinfo.append((combined_index, ":".join([platform.name,
-                                                     device.name])))
+            clinfo.append((combined_index, ":".join([cl_platform.name,
+                                                     cl_platform.name])))
             d_index += 1
         p_index += 1