devika icon indicating copy to clipboard operation
devika copied to clipboard

Update knowledge_base.py

Open ImSuvodeep opened this issue 11 months ago • 6 comments

Implemented BM25 search functionality for knowledge retrieval in KnowledgeBase class

ImSuvodeep avatar Mar 29 '24 16:03 ImSuvodeep

Looks good to me. It would be cool if you could add the BM25Okapi package to the 'requirements.txt'

juvi21 avatar Mar 29 '24 17:03 juvi21

Done

ImSuvodeep avatar Mar 29 '24 18:03 ImSuvodeep

Commit the updated requirements.txt to this pull request instead of opening a new one. I think you can close https://github.com/stitionai/devika/pull/254 :)

juvi21 avatar Mar 29 '24 18:03 juvi21

Exception in thread Thread-11: Traceback (most recent call last): File "C:\Users\pc\AppData\Local\Programs\Python\Python39\lib\threading.py", line 973, in _bootstrap_inner self.run() File "C:\Users\pc\AppData\Local\Programs\Python\Python39\lib\threading.py", line 910, in run self._target(*self._args, **self._kwargs) File "C:\Users\pc\Desktop\fork\devika\devika.py", line 97, in thread = Thread(target=lambda: agent.execute(message, project_name)) File "C:\Users\pc\Desktop\fork\devika\src\agents\agent.py", line 353, in execute search_results = self.search_queries(queries, project_name) File "C:\Users\pc\Desktop\fork\devika\src\agents\agent.py", line 72, in search_queries knowledge_base = KnowledgeBase() File "C:\Users\pc\Desktop\fork\devika\src\memory\knowledge_base.py", line 31, in init self.bm25 = BM25Okapi(self.tokenized_contents) File "C:\Users\pc\Desktop\devika.venv\lib\site-packages\rank_bm25.py", line 83, in init super().init(corpus, tokenizer) File "C:\Users\pc\Desktop\devika.venv\lib\site-packages\rank_bm25.py", line 27, in init nd = self._initialize(corpus) File "C:\Users\pc\Desktop\devika.venv\lib\site-packages\rank_bm25.py", line 52, in _initialize self.avgdl = num_doc / self.corpus_size ZeroDivisionError: division by zero

``

darrassi1 avatar Mar 31 '24 07:03 darrassi1

let me know if the new commit fix ur error

ImSuvodeep avatar Mar 31 '24 10:03 ImSuvodeep

let me know if the new commit fix ur error

the error is gone

darrassi1 avatar Apr 02 '24 13:04 darrassi1

Need testing

ARajgor avatar Apr 12 '24 03:04 ARajgor

Also add it to requirements

ARajgor avatar Apr 12 '24 03:04 ARajgor