RF: Title available in show mehtod.

- actors parameter is show method can accept a single object as well
- updated the title in the show manager to handle empty and none values.

Author: maharshi-gor

fury/tests/test_window.py

@@ -22,6 +22,7 @@
     Scene as GfxScene,
     ScreenCoordsCamera,
     Texture,
+    TrackballController,
     have_imgui_bundle,
 )
 from fury.ui import Rectangle2D, UIContext
@@ -222,7 +223,7 @@ def test_screen_initialization_default():
     assert screen.size == (640, 480)  # Default size of pygfx
     assert screen.position == (0, 0)  # Default position of pygfx
     assert isinstance(screen.camera, PerspectiveCamera)
-    assert isinstance(screen.controller, OrbitController)
+    assert isinstance(screen.controller, TrackballController)
 
     assert (
         len(screen.scene.main_scene.children) == 3
@@ -522,11 +523,40 @@ def test_show_manager_with_empty_config():
     assert len(show_m.screens) == 1
 
 
+def test_show_manager_with_empty_title():
+    """Test initialization with empty screen config."""
+    show_m = ShowManager(window_type="offscreen", title=None)
+    assert show_m._title == "FURY 2.0"
+    show_m = ShowManager(window_type="offscreen", title="")
+    assert show_m._title == "FURY 2.0"
+
+
 def test_display_default(sample_actor):
     """Test the display function with default parameters."""
     with patch("fury.window.ShowManager") as mock_show_manager:
-        show([sample_actor])
+        show(sample_actor)
         mock_show_manager.assert_called_once()
+        kwargs = mock_show_manager.call_args.kwargs
+        assert kwargs["window_type"] == "default"
+        assert kwargs["title"] == "FURY 2.0"
+        assert sample_actor in kwargs["scene"].main_scene.children
+        mock_show_manager.return_value.start.assert_called_once_with()
+
+
+def test_display_accepts_iterable_actors(sample_actor):
+    """Test the display function with a non-list iterable of actors."""
+    second_actor = sphere(np.array([[1, 0, 0]]), material="basic", impostor=False)
+    actors = (item for item in (sample_actor, second_actor))
+
+    with patch("fury.window.ShowManager") as mock_show_manager:
+        show(actors, window_type="offscreen", title="Iterable actors")
+
+        kwargs = mock_show_manager.call_args.kwargs
+        assert kwargs["window_type"] == "offscreen"
+        assert kwargs["title"] == "Iterable actors"
+        assert sample_actor in kwargs["scene"].main_scene.children
+        assert second_actor in kwargs["scene"].main_scene.children
+        mock_show_manager.return_value.start.assert_called_once_with()
 
 
 def test_add_remove_ui_to_from_scene(sample_actor):

fury/window.py

@@ -11,6 +11,7 @@
 import logging
 import os
 import sys
+from typing import Iterable
 
 from PIL.Image import fromarray as image_from_array
 import numpy as np
@@ -690,6 +691,8 @@ def __init__(
         event handling.
         """
         self._size = size
+        if not title:
+            title = "FURY 2.0"
         self._title = title
         self._is_qt = False
         self._qt_app = qt_app
@@ -1743,7 +1746,12 @@ def __str__(self):
     return report
 
 
-def show(actors, *, window_type="default"):
+def show(
+    actors,
+    *,
+    window_type="default",
+    title="FURY 2.0",
+):
     """
     Display one or more actors in a new window quickly.
 
@@ -1756,9 +1764,14 @@ def show(actors, *, window_type="default"):
         The PyGfx actor(s) to display.
     window_type : str, optional
         The type of window canvas to create ('default', 'glfw', 'qt',
-        'jupyter', 'offscreen'). Defaults to 'default'.
+        'jupyter', 'offscreen').
+    title : str, optional
+        The title for the window.
     """
     scene = Scene()
-    scene.add(*actors)
-    show_m = ShowManager(scene=scene, window_type=window_type)
+    if isinstance(actors, Iterable):
+        scene.add(*actors)
+    else:
+        scene.add(actors)
+    show_m = ShowManager(scene=scene, window_type=window_type, title=title)
     show_m.start()