Update to improve error logging around failures to parse the metadata…

… string.

model/model_updater.py

@@ -33,7 +33,9 @@ async def sync_model(self, hotkey: str) -> bool:
         metadata = await self._get_metadata(hotkey)
 
         if not metadata:
-            bt.logging.trace(f"No metadata found on the chain for hotkey {hotkey}")
+            bt.logging.trace(
+                f"No valid metadata found on the chain for hotkey {hotkey}"
+            )
             return False
 
         # Check what model id the model tracker currently has for this hotkey.

model/storage/chain/chain_model_metadata_store.py

@@ -46,6 +46,7 @@ async def retrieve_model_metadata(self, hotkey: str) -> Optional[ModelMetadata]:
         partial = functools.partial(
             bt.extrinsics.serving.get_metadata, self.subtensor, self.subnet_uid, hotkey
         )
+
         metadata = utils.run_in_subprocess(partial, 60)
 
         if not metadata:
@@ -55,7 +56,16 @@ async def retrieve_model_metadata(self, hotkey: str) -> Optional[ModelMetadata]:
         hex_data = commitment[list(commitment.keys())[0]][2:]
 
         chain_str = bytes.fromhex(hex_data).decode()
-        model_id = ModelId.from_compressed_str(chain_str)
+
+        try:
+            model_id = ModelId.from_compressed_str(chain_str)
+        except:
+            # If the metadata format is not correct on the chain then we return None.
+            bt.logging.trace(
+                f"Failed to parse the metadata on the chain for hotkey {hotkey}."
+            )
+            return None
+
         model_metadata = ModelMetadata(id=model_id, block=metadata["block"])
 
         return model_metadata