这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@homanp
Copy link
Collaborator

@homanp homanp commented Feb 4, 2025

This pull request includes several changes to the python/src/reag/client.py file to improve the parallel processing of document batches and minor adjustments in the python/tests/test_client.py file. The most important changes include adding asynchronous processing for document batches and reorganizing the return statement for clarity.

Improvements to parallel processing:

  • python/src/reag/client.py: Added tasks for parallel processing within the batch and used asyncio.gather to process all documents in the batch concurrently. This change improves the efficiency of handling document batches.

Code organization and readability:

  • python/src/reag/client.py: Moved the return statement outside the batch loop to ensure all results are returned after processing all batches. This change clarifies the flow of the function.

Minor code adjustments:

  • python/tests/test_client.py: Fixed the import statement to correctly import ReagClient, Document, and QueryResult from reag.client instead of src.reag.client.

@homanp homanp added the bug Something isn't working label Feb 4, 2025
@homanp homanp self-assigned this Feb 4, 2025
@homanp homanp merged commit 396e784 into main Feb 4, 2025
@homanp homanp linked an issue Feb 4, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Python impl bugs

2 participants