Skip to content

Commit f2fadf0

Browse files
authored
[ENH]: Schema types (#5509)
## Description of changes _Summarize the changes made by this PR._ - Improvements & Bug fixes - Introduces a `Schema` type and provides builder for it so that users can construct it intuitively - Introduces config for various index types. Added type validation for embedding function for vector index - Introduces an `InternalSchema` type that gets inited from `Schema` type. This type is serialized to a json and sent over the wire to the FE. Also added deserialized method to convert from json to this type - Added 47 tests for various cases related to these types. - New functionality - ... ## Test plan _How are these changes tested?_ Plenty of unit-tests added - [x] Tests pass locally with `pytest` for python, `yarn test` for js, `cargo test` for rust ## Migration plan None ## Observability plan CI green ## Documentation Changes Not yet
1 parent 0460b9b commit f2fadf0

File tree

3 files changed

+1717
-5
lines changed

3 files changed

+1717
-5
lines changed

chromadb/api/client.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,13 +79,13 @@ def __init__(
7979
user_provided_tenant=tenant,
8080
user_provided_database=database,
8181
)
82-
82+
8383
# this should not happen unless types are invalidated
8484
if maybe_tenant is None and tenant is None:
8585
raise ChromaAuthError("Could not determine a tenant from the current authentication method. Please provide a tenant.")
8686
if maybe_database is None and database is None:
8787
raise ChromaAuthError("Could not determine a database name from the current authentication method. Please provide a database name.")
88-
88+
8989
if maybe_tenant:
9090
self.tenant = maybe_tenant
9191
if maybe_database:

0 commit comments

Comments
 (0)