Deepmemory fixes #2675

adolkhan · 2023-10-30T09:21:22Z

🚀 🚀 Pull Request

Impact

Bug fix (non-breaking change which fixes expected existing functionality)
Enhancement/New feature (adds functionality without impacting existing logic)
Breaking change (fix or feature that would cause existing functionality to change)

Description

Changed default rate_limiter behavior, if it is not specified or partially specified we auto populate other parameters.
Fixed docs
Changed printings in deep_memory and made them more clear
Changed deep_memory search to be local (temporary change, while tensor_db is not fixed)

Things to be aware of

Things to worry about

Additional Context

codecov · 2023-10-30T10:12:34Z

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (95786a3) 84.31% compared to head (62f842b) 83.72%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2675      +/-   ##
==========================================
- Coverage   84.31%   83.72%   -0.60%     
==========================================
  Files         231      231              
  Lines       25843    25856      +13     
==========================================
- Hits        21789    21647     -142     
- Misses       4054     4209     +155

Flag	Coverage Δ
unittests	`83.72% <93.10%> (-0.60%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
deeplake/core/vectorstore/deep_memory.py	`94.30% <100.00%> (+0.12%)`	⬆️
deeplake/core/vectorstore/deeplake_vectorstore.py	`95.79% <ø> (-0.21%)`	⬇️
...eeplake/core/vectorstore/deepmemory_vectorstore.py	`100.00% <ø> (ø)`
...core/vectorstore/vector_search/dataset/__init__.py	`100.00% <ø> (ø)`
.../core/vectorstore/vector_search/dataset/dataset.py	`95.98% <94.11%> (-0.20%)`	⬇️
...ectorstore/vector_search/indra/search_algorithm.py	`86.00% <83.33%> (ø)`