do not poll player when not needed

diff --git a/music_assistant/controllers/players.py b/music_assistant/controllers/players.py
index 2ac4b3719666b810e0e62e140a607bce35f92ef5..b7e6f3441e639a5918eea6e6ca4d7275371fa5dc 100755 (executable)
--- a/music_assistant/controllers/players.py
+++ b/music_assistant/controllers/players.py
@@ -96,9 +96,15 @@ class PlayerController:
         cur_tick = 0
         while True:
             for player in self.players:
-                if cur_tick == interval or player.state in (
-                    PlayerState.PLAYING,
-                    PlayerState.PAUSED,
+                if not player.available:
+                    continue
+                if cur_tick == interval or (
+                    player.active_queue.active
+                    and player.state
+                    in (
+                        PlayerState.PLAYING,
+                        PlayerState.PAUSED,
+                    )
                 ):
                     player.update_state()
             if cur_tick == interval:

diff --git a/music_assistant/models/player_queue.py b/music_assistant/models/player_queue.py
index 800b6facf474bdeb10def24f6d841cf4ba74c8f3..c8b21051c4a85863f06498197468e31b4055356c 100644 (file)
--- a/music_assistant/models/player_queue.py
+++ b/music_assistant/models/player_queue.py
@@ -105,10 +105,12 @@ class PlayerQueue:
 
     @property
     def active(self) -> bool:
-        """Return bool if the queue is currenty active on the player."""
-        if not self.player.current_url:
-            return False
+        """Return if the queue is currenty active."""
         if stream := self.stream:
+            if not self.stream.done.is_set():
+                return True
+            if not self.player.current_url:
+                return False
             return stream.stream_id in self.player.current_url
         return False