-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Upgrade to Chroma Vector database API v2 #2685
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
Upgrade to Chroma Vector database API v2 #2685
Conversation
Thank you very much! |
f7840db
to
452dba6
Compare
@ilayaperumalg I have completed the rebase to resolve the conflict. |
recently, testcontainers-java 1.21.0 is released By upgrading the version of testcontainers-java, we can remove the temporal overridden wait strategy in this pr: static {
chromaContainer.waitingFor(new AbstractWaitStrategy() {
@Override
protected void waitUntilReady() {
Wait.forHttp("/api/v2/heartbeat");
}
});
} If you want, I can update the PR. But it’s fine to leave it as is. |
I'm having these issues as well. Any idea when this issue will be merged? |
@dev-jonghoonpark Please go ahead and update the PR with your suggestion. |
@ilayaperumalg Thank you so much! |
452dba6
to
2d3c26c
Compare
Signed-off-by: jonghoonpark <dev@jonghoonpark.com>
2d3c26c
to
201433e
Compare
|
@dev-jonghoonpark Thank you once again for the PR upgrading Chroma vector database to use v2 API. Rebased and merged as 09a6a6e |
ChromaDB 1.0.0 has recently been released.
With this release, the v1 APIs have been completely deprecated.
To support the new v2 APIs, we need to upgrade API implementation.
related issues