From fd7c9bd5e0298e9e17386302fe511aa89b1355bf Mon Sep 17 00:00:00 2001
From: James Briggs <james.briggs@hotmail.com>
Date: Mon, 11 Nov 2024 08:36:24 +0100
Subject: [PATCH] fix: make sync hash optional implementation

---
 semantic_router/index/base.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/semantic_router/index/base.py b/semantic_router/index/base.py
index 31ff24d8..6ddaf6ed 100644
--- a/semantic_router/index/base.py
+++ b/semantic_router/index/base.py
@@ -6,6 +6,7 @@ from pydantic.v1 import BaseModel
 
 from semantic_router.schema import ConfigParameter
 from semantic_router.route import Route
+from semantic_router.utils.logger import logger
 
 
 class BaseIndex(BaseModel):
@@ -183,7 +184,12 @@ class BaseIndex(BaseModel):
 
         This method should be implemented by subclasses.
         """
-        raise NotImplementedError("This method should be implemented by subclasses.")
+        logger.warning("This method should be implemented by subclasses.")
+        return ConfigParameter(
+            field="sr_hash",
+            value="",
+            namespace=self.namespace,
+        )
 
     def _write_config(self, config: ConfigParameter):
         """
@@ -191,7 +197,7 @@ class BaseIndex(BaseModel):
 
         This method should be implemented by subclasses.
         """
-        raise NotImplementedError("This method should be implemented by subclasses.")
+        logger.warning("This method should be implemented by subclasses.")
 
     def _get_all(self, prefix: Optional[str] = None, include_metadata: bool = False):
         """
-- 
GitLab