Do not always set UNKNOWN if flag is empty

sdss · Aug 31, 2023 · e38b905 · e38b905
1 parent d8c0226
commit e38b905
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/python/lvmecp/dome.py b/python/lvmecp/dome.py
@@ -65,6 +65,9 @@ async def _update_internal(self):
         else:
             new_status |= self.flag.POSITION_UNKNOWN
 
+        if new_status.value == 0:
+            new_status = self.flag.__unknown__
+
         return new_status
 
     async def set_direction(self, open: bool):

diff --git a/python/lvmecp/module.py b/python/lvmecp/module.py
@@ -87,9 +87,6 @@ async def update(self):
             log.warning(f"{self.name}: failed updating status: {err}")
             new_status = self.flag(self.flag.__unknown__)
 
-        if new_status.value == 0:
-            new_status = self.flag(self.flag.__unknown__)
-
         # Only notify if the status has changed.
         if new_status != self.status:
             await self.notify_status(new_status)

diff --git a/python/lvmecp/safety.py b/python/lvmecp/safety.py
@@ -38,6 +38,9 @@ async def _update_internal(self):
         if safety_status.local:
             new_status |= self.flag.LOCAL
 
+        if new_status.value == 0:
+            new_status = self.flag.__unknown__
+
         return new_status
 
     async def is_remote(self):