Set VectorStore.search embedding argument type

[adolkhan]

🚀 🚀 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

There were some users that didn't know what should they be putting inside of embed_data parameter ins Vectorstore's search method. Intuitively most of them expect it to support direct strings. In the future we are also planning to add batch query support. So decided to add make embed_data parameter accept both string and List of strings.

Things to be aware of

Things to worry about

Additional Context

[codecov]

Codecov Report

Patch coverage: 36.36% and project coverage change: -0.02% ⚠️

Comparison is base (8749a4d) 84.92% compared to head (4d7bfa8) 84.90%.
Report is 16 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2538      +/-   ##
==========================================
- Coverage   84.92%   84.90%   -0.02%     
==========================================
  Files         328      328              
  Lines       39035    39043       +8     
==========================================
  Hits        33149    33149              
- Misses       5886     5894       +8     

Flag	Coverage Δ
unittests	84.90% <36.36%> (-0.02%)	⬇️

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

Files Changed	Coverage Δ
deeplake/core/vectorstore/deeplake_vectorstore.py	88.88% <ø> (-0.27%)	⬇️
...lake/core/vectorstore/test_deeplake_vectorstore.py	59.66% <14.28%> (-0.77%)	⬇️
.../core/vectorstore/vector_search/dataset/dataset.py	95.78% <75.00%> (-0.45%)	⬇️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.