Merge pull request #24 from iloveicedgreentea/new_commands

Adding content_type, hdr processing, theater optimizer Attributes. Library bump

custom_components/jvc_projectors/manifest.json

@@ -4,7 +4,7 @@
   "config_flow": false,
   "documentation": "https://www.home-assistant.io/integrations/jvc_projector",
   "requirements": [
-    "jvc-projector-remote-improved2==3.4.0"
+    "jvc-projector-remote-improved2==3.4.3"
   ],
   "ssdp": [],
   "zeroconf": [],
@@ -14,5 +14,5 @@
     "@iloveicedgreentea"
   ],
   "iot_class": "local_polling",
-  "version": "3.4.0"
+  "version": "3.4.3"
 }

custom_components/jvc_projectors/remote.py

@@ -53,6 +53,7 @@ def setup_platform(
         ]
     )
 
+
 class JVCRemote(RemoteEntity):
     """Implements the interface for JVC Remote in HA."""
 
@@ -75,6 +76,9 @@ def __init__(
         self._eshift = ""
         self._color_mode = ""
         self._input_level = ""
+        self._content_type = ""
+        self._hdr_processing = ""
+        self._theater_optimizer = ""
         # Because we can only have one connection at a time, we need to lock every command
         # otherwise JVC's server implementation will cancel the running command
         # and just confuse everything, then cause HA to freak out
@@ -84,9 +88,6 @@ def __init__(
     @property
     def should_poll(self):
         """Poll."""
-#       
-        # Polling is disabled as it is unreliable and will lock up commands at the moment
-        # Requires adding stronger locking and command buffering
         return True
 
     @property
@@ -105,15 +106,26 @@ def extra_state_attributes(self):
         # Useful for making sensors
         return {
             "power_state": self._state,
-            "installation_mode": self._installation_mode,
             "picture_mode": self._picture_mode,
+            "installation_mode": self._installation_mode,
+            "content_type": "Unsupported"
+            if "NX" in self._model_family
+            else self._content_type,
+            "hdr_processing": "Unsupported"
+            if "NX" in self._model_family
+            else self._hdr_processing,
+            "theater_optimizer": "Unsupported"
+            if "NX" in self._model_family
+            else "on" in self._theater_optimizer,
+            "low_latency": self._lowlatency_enabled,
             "input_mode": self._input_mode,
-            "laser_mode": self._laser_mode if "NZ" in self._model_family else "Unsupported",
-            "eshift": self._eshift if "NZ" in self._model_family else "Unsupported",
-            "color_mode": self._color_mode,
+            "laser_mode": self._laser_mode
+            if "NZ" in self._model_family
+            else "Unsupported",
             "input_level": self._input_level,
-            "low_latency": self._lowlatency_enabled,
-            "model_family": self._model_family
+            "color_mode": self._color_mode,
+            "eshift": self._eshift if "NZ" in self._model_family else "Unsupported",
+            "model_family": self._model_family,
         }
 
     @property
@@ -146,8 +158,15 @@ def update(self):
             self._input_level = self.jvc_client.get_input_level()
             self._picture_mode = self.jvc_client.get_picture_mode()
 
-            # Only look at laser for NZ
+            # NZ specifics
             if "NZ" in self._model_family:
+                self._content_type = self.jvc_client.get_content_type()
+                # only check HDR if the content type matches else timeout
+                if any(x in self._content_type for x in ["hdr", "hlg"]):
+                    self._hdr_processing = self.jvc_client.get_hdr_processing()
+                    self._theater_optimizer = (
+                        self.jvc_client.get_theater_optimizer_state()
+                    )
                 self._laser_mode = self.jvc_client.get_laser_mode()
                 # Some non-nz models support this but some don't, so easier to just not check
                 self._eshift = self.jvc_client.get_eshift_mode()