Cosmetics

pyblish_qml/host.py

@@ -111,17 +111,7 @@ def _foster_ninja(foster):
         return False
 
     value = os.environ.get("PYBLISH_QML_FOSTER_NINJA", "").lower()
-    if value in ("true", "yes", "1"):
-        return True
-
-    elif value in ("false", "no", "0"):
-        return False
-
-    else:
-        if QtCore.qVersion()[0] == "5":
-            return True
-        else:
-            return False
+    return value in ("true", "yes", "1") or QtCore.qVersion()[0] == "5"
 
 
 def show(parent=None, targets=[], modal=None, foster=None):
@@ -130,6 +120,12 @@ def show(parent=None, targets=[], modal=None, foster=None):
     Requires install() to have been run first, and
     a live instance of Pyblish QML in the background.
 
+    Arguments:
+        parent (None, optional): Deprecated
+        targets (list, optional): Publishing targets
+        modal (bool, optional): Block interactions to parent
+        foster (bool, optional): Become a real child of the parent process
+
     """
 
     # Get modal mode from environment

pyblish_qml/ipc/server.py

@@ -104,11 +104,12 @@ def show(self, settings=None):
             settings (optional, dict): Client settings
 
         """
-        if not self.ninja:
+        if self.ninja:
+            return self._dispatch("show", args=[settings or {},
+                                                self._winId,
+                                                self.ninja])
+        else:
             self.vessel.show()
-        return self._dispatch("show", args=[settings or {},
-                                            self._winId,
-                                            self.ninja])
 
     def hide(self):
         """Hide the GUI"""
@@ -224,6 +225,10 @@ class Server(object):
         service (service.Service): Dispatch requests to this service
         python (str, optional): Absolute path to Python executable
         pyqt5 (str, optional): Absolute path to PyQt5
+        targets (list, optional): Publishing targets, e.g. `ftrack`
+        modal (bool, optional): Block interactions to parent
+        foster (bool, optional): Become a real child of the parent process
+        ninja (bool, optional): ...?
 
     """