Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lift default chunk size to 50; bump to 1.2.1 #283

Merged
merged 5 commits into from
Jun 5, 2024

Conversation

hemidactylus
Copy link
Collaborator

@hemidactylus hemidactylus commented Jun 5, 2024

Along with this raise, also some docstring and sample-code improvements came along, esp. around the get_database vs get_database_by_api_endpoint freedom;

also the docstrings have been updated to reflect the latest Data API behaviour with respect to the default projection and the independency of $vector and other special fields from the rest.

This comes with improvements and fixes to the tests:

  • splitting in the insert many failure tests to prevent async spilling between functionalities
  • adapt to the latest default-projection behaviour also in the "core" testing directory
  • adapt to the latest vectorize things from the Data API, namely the expectations on the return value from findEmbeddingProviders call.

@hemidactylus hemidactylus requested review from cbornet and erichare and removed request for cbornet June 5, 2024 23:13
@hemidactylus hemidactylus merged commit 670326f into master Jun 5, 2024
2 checks passed
@hemidactylus hemidactylus deleted the SL-post-1.2-improvements branch June 5, 2024 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants