[WIP] try to fix for use with PyQt5

examples/simple/__main__.py

@@ -15,10 +15,12 @@
 class ExampleSynopticWidget(SynopticWidget):
 
     "A custom subclass of the synoptic widget."
-
-    def on_click(self, kind, name):
-        # Overriding the click event handler to print information
-        print "click", kind, name
+    def __init__(self, url=None, parent=None, *args, **kwargs):
+        super(ExampleSynopticWidget, self).__init__(url=None, parent=None, *args, **kwargs)
+
+    #def on_click(self, kind, name):
+    #    # Overriding the click event handler to print information
+    #    print("click", kind, name)
 
 
 def main():

examples/simple_tango/models.json

@@ -0,0 +1,5 @@
+{
+  "value1" 	:  "sys/tg_test/1/double_scalar",
+  "value2"      :  "sys/tg_test/1/ampli",
+  "value3"      :  "sys/tg_test/1/ampli"
+}

setup.cfg

@@ -8,4 +8,4 @@ requires = python-fandango
            python-pytango
            qtwebkit
 
-release = 1%{?dist}.maxlab
+release = maxlab

svgsynoptic2/caseless.py

@@ -44,8 +44,8 @@ class CaselessDictionary(MutableMapping):
     def __init__(self, *args, **kwargs):
         self.__dict__["_dict"] = {}
         temp_dict = dict(*args, **kwargs)
-        for key, value in temp_dict.iteritems():
-            if isinstance(key, basestring):
+        for key, value in temp_dict.items():
+            if isinstance(key, str):
                 key = CaselessString.make_caseless(key)
             self._dict[key] = value
 
@@ -90,7 +90,7 @@ def __cmp__(self, other):
 
     @classmethod
     def make_caseless(cls, string):
-        if isinstance(string, unicode):
+        if isinstance(string, str):
             return CaselessUnicode(string)
         return CaselessStr(string)
 
@@ -99,5 +99,5 @@ class CaselessStr(CaselessString, str):
     pass
 
 
-class CaselessUnicode(CaselessString, unicode):
+class CaselessUnicode(CaselessString, str):
     pass

svgsynoptic2/jsinterface.py

@@ -1,6 +1,6 @@
 from threading import Lock
 
-from PyQt4 import QtCore
+from PyQt5 import QtCore
 
 
 class JSInterface(QtCore.QObject):
@@ -65,7 +65,7 @@ def setup(self):
 
     @QtCore.pyqtSlot(str, str, str)
     def run_plugin_command(self, plugin, cmd, args):
-        print "run_plugin_command", plugin, cmd, args
+        print("run_plugin_command", plugin, cmd, args)
         # Note: since we're using signals to loosely connect with the widget
         # it's not possible to get a return value. But we probably don't
         # want that anyway since it might block..?

svgsynoptic2/synopticwidget.py

@@ -8,10 +8,11 @@
 import os
 import json
 
-from PyQt4 import Qt, QtCore, QtGui
-from PyQt4.QtWebKit import QWebPage, QWebView, QWebSettings, QWebInspector
+from PyQt5 import Qt, QtCore, QtGui, QtWidgets
+from PyQt5.QtWebKitWidgets import  QWebView, QWebPage, QWebInspector
+from PyQt5.QtWebKit  import QWebSettings
 
-from jsinterface import JSInterface
+from .jsinterface import JSInterface
 
 
 class LoggingWebPage(QWebPage):
@@ -27,10 +28,10 @@ def __init__(self, logger=None, parent=None):
 
     def javaScriptConsoleMessage(self, msg, lineNumber, sourceID):
         # don't use the logger for now; too verbose :)
-        print "JsConsole(%s:%d):\n\t%s" % (sourceID, lineNumber, msg)
+        print("JsConsole(%s:%d):\n\t%s" % (sourceID, lineNumber, msg))
 
 
-class SynopticWidget(QtGui.QWidget):
+class SynopticWidget(QtWidgets.QWidget):
 
     """
     A Qt widget displaying a SVG synoptic in a webview.
@@ -50,7 +51,7 @@ def __init__(self, url=None, parent=None, *args, **kwargs):
         self._modelNames = None
 
     def _setup_ui(self, url=None, section=None):
-        self.hbox = hbox = QtGui.QHBoxLayout(self)
+        self.hbox = hbox = QtWidgets.QHBoxLayout(self)
         self.hbox.setContentsMargins(0, 0, 0, 0)
         self.hbox.layout().setContentsMargins(0, 0, 0, 0)
         self.setLayout(self.hbox)
@@ -60,13 +61,13 @@ def _setup_ui(self, url=None, section=None):
     def set_url(self, url, section=None):
         # TODO: probably breaks things if the url is already set
         self._url = url
-        self.splitter = QtGui.QSplitter(self)
+        self.splitter = QtWidgets.QSplitter(self)
         self.splitter.setOrientation(QtCore.Qt.Vertical)
         self.hbox.addWidget(self.splitter)
         view = self._create_view(url, section)
         self._setup_inspector(view)
         self.splitter.addWidget(view)
-        print "set_url", url
+        print("set_url", url)
 
     def setConfig(self, configFile):
         abspath = os.path.dirname(os.path.abspath(configFile))
@@ -110,7 +111,8 @@ def _create_view(self, html=None, section=None):
         # Inject JSInterface into the JS global namespace as "Backend"
         def addBackend():
             frame.addToJavaScriptWindowObject('QtBackend', self.js)
-        view.connect(frame, QtCore.SIGNAL("javaScriptWindowObjectCleared()"), addBackend)
+        #view.connect(frame, QtCore.SIGNAL("javaScriptWindowObjectCleared()"), addBackend)
+        frame.javaScriptWindowObjectCleared.connect(addBackend)
 
         # load the page
         # need to set the "base URL" for the webview to find the
@@ -152,7 +154,7 @@ def toggle_inspector():
                 self._inspector.setPage(page)
                 self.splitter.addWidget(self._inspector)
 
-        shortcut = QtGui.QShortcut(self)
+        shortcut = QtWidgets.QShortcut(self)
         shortcut.setKey(QtCore.Qt.Key_F12)
         shortcut.activated.connect(toggle_inspector)
 
@@ -218,7 +220,7 @@ def select(self, kind, names, replace=True):
         """Set a list of items as 'selected'. By default unselects all
         previously selected things first.
         """
-        print "select", kind, names
+        print("select", kind, names)
         if replace:
             self.js.evaluate("synoptic.unselectAll()")
         if names:
@@ -234,7 +236,7 @@ def unselect_all(self):
 
 if __name__ == '__main__':
     import sys
-    print sys.argv[1]
+    print(sys.argv[1])
     qapp = Qt.QApplication([])
     sw = SynopticWidget(sys.argv[1])
     sw.show()

svgsynoptic2/taurusregistry.py

@@ -1,7 +1,7 @@
 from threading import Lock, Event
 from time import sleep
 
-from PyQt4 import QtCore
+from PyQt5 import QtCore
 from taurus import Attribute
 from taurus.core import TaurusException
 try:
@@ -18,9 +18,9 @@
 
 # We can't use pytango's CaselessDict since it does not keep the original
 # case of the keys :(
-from caseless import CaselessDictionary as CaselessDict
+from .caseless import CaselessDictionary as CaselessDict
 
-from ttldict import TTLDict
+from .ttldict import TTLDict
 
 
 class Registry(QtCore.QThread):
@@ -77,19 +77,19 @@ def subscribe(self, models=[]):
                     try:
                         taurusattrs[modelstate] = Attribute(modelstate)
                     except TaurusException as e:
-                        print "Failed to create Taurus Attribute for model %s! %s" % (model, e)
+                        print("Failed to create Taurus Attribute for model %s! %s" % (model, e))
             elif (self.attribute_validator.isValid(model) or
                     self.eval_validator.isValid(model)):
                 attrs[model] = True
                 if model not in taurusattrs.keys():
                     try:
                         taurusattrs[model] = Attribute(model)
                     except TaurusException as e:
-                        print "Failed to create Taurus Attribute for model %s! %s" % (model, e)
+                        print("Failed to create Taurus Attribute for model %s! %s" % (model, e))
                     except Exception as e:
-                        print "Failed to create Taurus Attribute for model %s!" % (model)
+                        print("Failed to create Taurus Attribute for model %s!" % (model))
             else:
-                print "Invalid Taurus model %s!?" % model
+                print("Invalid Taurus model %s!?" % model)
         self._attributes = attrs
         self._taurus_attributes = taurusattrs
 
@@ -129,7 +129,7 @@ def _update(self, attributes=CaselessDict()):
             try:
                 self._add_listener(attr)
             except (TypeError, PyTango.DevFailed) as e:
-                print "Failed to setup listener for", attr, e
+                print("Failed to setup listener for", attr, e)
 
         self.unsubscribe_callback(old_attrs)
 
@@ -147,9 +147,9 @@ def _add_listener(self, model):
             listener.addListener(self.handle_event)
             return listener
         except (TaurusException, AttributeError) as e:
-            print "Failed to subscribe to model %s! %s" % (model, e)
+            print("Failed to subscribe to model %s! %s" % (model, e))
         except Exception:
-            print "Failed to subscribe to model %s!" % (model)
+            print("Failed to subscribe to model %s!" % (model))
 
     def _remove_listener(self, model):
         listener = self.listeners.pop(model)

svgsynoptic2/taurussynopticwidget.py

@@ -6,9 +6,9 @@
 import json
 
 import numpy as np
-from PyQt4 import QtCore
+from PyQt5 import QtCore
 import PyTango
-from synopticwidget import SynopticWidget
+from .synopticwidget import SynopticWidget
 from taurus import Attribute, Manager, Release
 from taurus.core.taurusbasetypes import (
     AttrQuality, TaurusEventType, TaurusSerializationMode)
@@ -47,7 +47,7 @@ def run(self):
                                   % value.value)
             self.finished.emit(self.model, html)
         except PyTango.DevFailed as e:
-            print e
+            print(e)
 
 
 def getStateClasses(state=None):
@@ -110,13 +110,13 @@ def run_plugin_command(self, plugin, cmd, args):
             plugins = __import__("plugins.%s" % plugin, globals(),
                                  locals(), [cmd], -1)
         except ImportError as e:
-            print "Could not initialize plugin '%s'!" % plugin
-            print e
+            print("Could not initialize plugin '%s'!" % plugin)
+            print(e)
             return ""
         return getattr(plugins, cmd)(self, args)
 
     def handle_subscriptions(self, models=[]):
-        print "handle_subscriptions ", models
+        print("handle_subscriptions ", models)
         if self.registry:
             self.registry.subscribe(models)
 
@@ -246,7 +246,7 @@ def on_click(self, kind, name):
         clicked section.  Override this function if you need something
         else.
         """
-        print "on_click", kind, name
+        print("on_click", kind, name)
         if kind == "model" and self.registry.device_validator.isValid(name):
             self.select(kind, [name])
             self.emit(Qt.SIGNAL("graphicItemSelected(QString)"), name)
@@ -269,7 +269,7 @@ def on_rightclick(self, kind, name):
             if name.lower() in self._panels:
 
                 widget = self._panels[name.lower()]
-                print "Found existing panel for %s:" % name, widget
+                print("Found existing panel for %s:" % name, widget)
                 if not widget.isVisible():
                     widget.show()
                 widget.activateWindow()
@@ -305,10 +305,10 @@ def _cleanup_panel(self, w):
         become pretty bogged down."""
         if self.registry:
             with self.registry.lock:
-                print "cleaning up panel for", w.getModel(), "..."
+                print("cleaning up panel for", w.getModel(), "...")
                 self._panels.pop(str(w.getModel()).lower(), None)
                 w.setModel(None)
-                print "done!"
+                print("done!")
 
     # Note: the tooltip stuff is broken and not currently in use.
     # Currently there is only the default tooltip which displays the
@@ -365,13 +365,13 @@ def closeEvent(self, event):
         # not exit cleanly.
         super(TaurusSynopticWidget, self).closeEvent(event)
         for model, panel in self._panels.items():
-            print "closing panel for", model
+            print("closing panel for", model)
             panel.close()
 
 
 if __name__ == '__main__':
     import sys
-    print sys.argv[1]
+    print(sys.argv[1])
     # qapp = Qt.QApplication([])
     app = TaurusApplication()
     sw = TaurusSynopticWidget()