fix: Revert refactor commits

Revert "fix: Sys deps"

This reverts commit aba32e4.

Revert "fix: Minor details"

This reverts commit 42a3459.

Revert "fix: Depenency pinning, remove sys deps"

This reverts commit e688320.

Revert "fix: Refactor to have independent backend"

This reverts commit 508b625.

Author: AlejandroFernandezLuces

.github/workflows/ci_cd.yml

@@ -69,8 +69,10 @@ jobs:
   docs-build:
     name: Documentation Build
     runs-on: ubuntu-latest
-    needs: [docs-style]
+    # needs: [docs-style]
     steps:
+    - name: Install system dependencies
+      run: sudo apt install -qq libglu1-mesa-dev libx11-xcb-dev '^libxcb*'
     - name: Setup headless display
       uses: pyvista/setup-headless-display-action@v2
 
@@ -102,9 +104,11 @@ jobs:
 
   testing:
     name: Run Unit Tests
-    needs: [ smoke-tests ]
+    # needs: [ smoke-tests ]
     runs-on: ubuntu-latest
     steps:
+      - name: Install system dependencies
+        run: sudo apt install libegl1 libxcb-cursor0 libsm6 libxext6 libxcb-xinerama0 -y
       - name: Restore images cache
         uses: actions/cache@v4
         with:

examples/00-basic-pyvista-examples/qt_backend.py

@@ -37,17 +37,15 @@
 import pyvista as pv
 
 from ansys.tools.visualization_interface import Plotter
-from ansys.tools.visualization_interface.backends.pyvista.pyvistaqt_interface import (
-    PyVistaQtBackend,
-)
+from ansys.tools.visualization_interface.backends.pyvista import PyVistaBackend
 
 #########################
 # Open a pyvistaqt window
 # =======================
 # .. code-block:: python
 #
 #    cube = pv.Cube()
-#    pv_backend = PyVistaQtBackend()
+#    pv_backend = PyVistaBackend(use_qt=True)
 #    pl = Plotter(backend=pv_backend)
 #    pl.plot(cube)
 #    pl.show()

pyproject.toml

@@ -40,7 +40,7 @@ tests = [
     "pytest-cov==6.0.0",
     "pyside6==6.7.3",
     "pyvistaqt==0.11.1,<1",
-    "pytest-qt==4.4.0"
+    "pytest-qt"
 ]
 
 doc = [
@@ -49,6 +49,8 @@ doc = [
     "jupytext==1.16.4",
     "nbsphinx==0.9.5",
     "numpydoc==1.8.0",
+    "pyside6==6.7.3",
+    "pyvistaqt==0.11.1,<1",
     "sphinx==8.1.3",
     "sphinx-autoapi==3.3.3",
     "sphinx-copybutton==0.5.2",

src/ansys/tools/visualization_interface/backends/pyvista/pyvista.py

@@ -88,6 +88,8 @@ class PyVistaBackendInterface(BaseBackend):
         Whether to plot the names of the picked objects.
     show_plane : Optional[bool], default: False
         Whether to show the plane in the plotter.
+    use_qt : Optional[bool], default: False
+        Whether to use the Qt backend for the plotter.
     """
 
     def __init__(
@@ -97,12 +99,14 @@ def __init__(
         allow_hovering: Optional[bool] = False,
         plot_picked_names: Optional[bool] = False,
         show_plane: Optional[bool] = False,
+        use_qt: Optional[bool] = False,
         **plotter_kwargs,
     ) -> None:
         """Initialize the ``use_trame`` parameter and save the current ``pv.OFF_SCREEN`` value."""
         # Check if the use of trame was requested
         if use_trame is None:
             use_trame = ansys.tools.visualization_interface.USE_TRAME
+        self._use_qt = use_qt
         self._use_trame = use_trame
         self._allow_picking = allow_picking
         self._allow_hovering = allow_hovering
@@ -149,7 +153,7 @@ def __init__(
             logger.warning(warn_msg)
             self._pl = PyVistaInterface(show_plane=show_plane)
         else:
-            self._pl = PyVistaInterface(show_plane=show_plane, **plotter_kwargs)
+            self._pl = PyVistaInterface(show_plane=show_plane, use_qt=use_qt, **plotter_kwargs)
 
         self._enable_widgets = self._pl._enable_widgets
 
@@ -178,7 +182,8 @@ def enable_widgets(self):
             ]
             self._widgets.append(MeasureWidget(self))
             self._widgets.append(ScreenshotButton(self))
-            self._widgets.append(MeshSliderWidget(self))
+            if not self._use_qt:
+                self._widgets.append(MeshSliderWidget(self))
             self._widgets.append(HideButton(self))
 
     def add_widget(self, widget: Union[PlotterWidget, List[PlotterWidget]]):
@@ -544,10 +549,11 @@ def __init__(
         use_trame: Optional[bool] = None,
         allow_picking: Optional[bool] = False,
         allow_hovering: Optional[bool] = False,
-        plot_picked_names: Optional[bool] = True
+        plot_picked_names: Optional[bool] = True,
+        use_qt: Optional[bool] = False
     ) -> None:
         """Initialize the generic plotter."""
-        super().__init__(use_trame, allow_picking, allow_hovering, plot_picked_names)
+        super().__init__(use_trame, allow_picking, allow_hovering, plot_picked_names, use_qt=use_qt)
 
     def plot_iter(
         self,
@@ -593,3 +599,8 @@ def plot(self, plottable_object: Any, name_filter: str = None, **plotting_option
             self.pv_interface.plot_iter(plottable_object, name_filter, **plotting_options)
         else:
             self.pv_interface.plot(plottable_object, name_filter, **plotting_options)
+
+    def close(self):
+        """Close the plotter for PyVistaQT."""
+        if self._use_qt:
+            self.pv_interface.scene.close()

src/ansys/tools/visualization_interface/backends/pyvista/pyvista_interface.py

@@ -20,11 +20,13 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 """Provides plotting for various PyAnsys objects."""
+import importlib
 import re
 from typing import Any, Dict, List, Optional, Union
 
 import pyvista as pv
 from pyvista.plotting.plotter import Plotter as PyVistaPlotter
+import pyvistaqt
 
 import ansys.tools.visualization_interface as viz_interface
 from ansys.tools.visualization_interface.types.edge_plot import EdgePlot
@@ -33,6 +35,8 @@
 from ansys.tools.visualization_interface.utils.color import Color
 from ansys.tools.visualization_interface.utils.logger import logger
 
+_HAS_PYVISTAQT = importlib.util.find_spec("pyvistaqt")
+
 
 class PyVistaInterface:
     """Provides the middle class between PyVista plotting operations and PyAnsys objects.
@@ -58,6 +62,9 @@ class PyVistaInterface:
         for visualization.
     show_plane : bool, default: False
         Whether to show the XY plane in the plotter window.
+    use_qt : bool, default: False
+        Whether to use the Qt backend for the plotter window.
+
     """
 
     def __init__(
@@ -67,16 +74,27 @@ def __init__(
         num_points: int = 100,
         enable_widgets: bool = True,
         show_plane: bool = False,
+        use_qt: bool = False,
         **plotter_kwargs,
     ) -> None:
         """Initialize the plotter."""
         # Generate custom scene if ``None`` is provided
         if scene is None:
             if viz_interface.TESTING_MODE:
-                scene = pv.Plotter(off_screen=True, **plotter_kwargs)
+                if use_qt and _HAS_PYVISTAQT:
+                    scene = pyvistaqt.BackgroundPlotter(off_screen=True)
+                else:
+                    if use_qt and not _HAS_PYVISTAQT:
+                        message = "PyVistaQt dependency is not installed. Install it with " + \
+                                  "`pip install ansys-tools-visualization-interface[pyvistaqt]`."
+                        logger.warning(message)
+                    scene = pv.Plotter(off_screen=True, **plotter_kwargs)
+            elif use_qt:
+                scene = pyvistaqt.BackgroundPlotter()
             else:
                 scene = pv.Plotter(**plotter_kwargs)
 
+        self._use_qt = use_qt
         # If required, use a white background with no gradient
         if not color_opts:
             color_opts = dict(color="white")
@@ -91,7 +109,9 @@ def __init__(
 
         # Show the XY plane
         self._show_plane = show_plane
-        self.scene.add_axes(interactive=False)
+        # if (not DOCUMENTATION_BUILD) or (DOCUMENTATION_BUILD and not use_qt) or (TESTING_MODE and not use_qt):
+        if not use_qt:
+            self.scene.add_axes(interactive=False)
 
         # objects to actors mapping
         self._object_to_actors_map = {}
@@ -335,7 +355,10 @@ def show(
         if jupyter_backend:
             self.scene.show(jupyter_backend=jupyter_backend, **kwargs)
         else:
-            self.scene.show(**kwargs)
+            if self._use_qt:
+                self.scene.show()
+            else:
+                self.scene.show(**kwargs)
 
     def set_add_mesh_defaults(self, plotting_options: Optional[Dict]) -> None:
         """Set the default values for the plotting options.