Feature/include vectors option document chunks

[emrgnt-cmplxty]

Important

Add include_vectors option to document chunk retrieval in CLI, API, SDKs, and update documentation accordingly.

• CLI:
  • Added --include-vectors option to document_chunks command in management.py.
• API:
  • Added include_vectors query parameter to document_chunks_app in management_router.py.
  • Updated document_chunks method in management_service.py to handle include_vectors.
• Database:
  • Modified get_document_chunks in vector.py to support include_vectors.
• SDKs:
  • Added include_vectors parameter to document_chunks method in management.py for Python and JS SDKs.
• Documentation:
  • Updated CLI, JS, and Python SDK documentation to include include_vectors option for document chunks.

^{This description was created by for 1c61f86. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 1c61f86 in 1 minute and 21 seconds

More details

• Looked at 306 lines of code in 10 files
• Skipped 0 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. py/core/providers/database/vector.py:508

• Draft comment:
  Consider adding a comment to explain the purpose of the include_vectors parameter and its impact on the query performance and data transfer.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The addition of the include_vectors parameter in the get_document_chunks method is a significant change. It allows users to optionally include vectors in the response, which can be useful for applications that need to process or analyze these vectors. However, the implementation should ensure that the vectors are only included when necessary to avoid unnecessary data transfer and processing.

2. py/core/main/api/management_router.py:370

• Draft comment:
  Ensure that the include_vectors parameter is consistently handled across all methods that support it, and update the documentation to reflect this new feature.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The include_vectors parameter is added to multiple methods across different files. This parameter allows users to include vectors in the response, which is a useful feature for applications that need to process these vectors. However, it's important to ensure that this parameter is consistently handled across all methods and that the documentation reflects this new feature.

3. py/sdk/management.py:268

• Draft comment:
  Ensure that the include_vectors parameter is consistently handled across all methods that support it, and update the documentation to reflect this new feature.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The include_vectors parameter is added to the document_chunks method in the SDK. This parameter allows users to include vectors in the response, which is a useful feature for applications that need to process these vectors. However, it's important to ensure that this parameter is consistently handled across all methods and that the documentation reflects this new feature.

Workflow ID: wflow_0EUrxLP7sHdZFd6l

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.