Add note about get_ntp

Author: marcelveldt

music_assistant/providers/airplay/protocols/_protocol.py

@@ -83,6 +83,8 @@ def running(self) -> bool:
     @abstractmethod
     async def get_ntp(self) -> int:
         """Get current NTP timestamp from the CLI binary."""
+        # this can probably be removed now that we already get the ntp
+        # in python (within the stream session start)
 
     @abstractmethod
     async def start(self, start_ntp: int, skip: int = 0) -> None:

music_assistant/providers/airplay/protocols/airplay2.py

@@ -35,7 +35,8 @@ class AirPlay2Stream(AirPlayProtocol):
 
     async def get_ntp(self) -> int:
         """Get current NTP timestamp."""
-        # TODO!
+        # this can probably be removed now that we already get the ntp
+        # in python (within the stream session start)
         return get_ntp_timestamp()
 
     @property

music_assistant/providers/airplay/protocols/raop.py

@@ -45,6 +45,8 @@ def running(self) -> bool:
 
     async def get_ntp(self) -> int:
         """Get current NTP timestamp from the CLI binary."""
+        # this can probably be removed now that we already get the ntp
+        # in python (within the stream session start)
         cli_binary = await get_cli_binary(self.player.protocol)
         # TODO: we can potentially also just generate this ourselves?
         self.prov.logger.debug("Getting NTP timestamp from %s CLI binary", self.player.protocol)