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

feat: add component clickhouse for vector database #3657

Merged
merged 25 commits into from
Sep 4, 2024

Conversation

MarceloNunesAlves
Copy link
Contributor

This PR is about including the Clickhouse component as a database for embedding.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Sep 2, 2024
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Sep 2, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-3657.dmtpw4p5recq1.amplifyapp.com

@MarceloNunesAlves
Copy link
Contributor Author

Hi @ogabrielluiz,
I included a new component, and for it to work I needed to add a library to poetry but there was an error in the "Check Poetry lock" validation but I don't know what it is.

@ogabrielluiz
Copy link
Contributor

Hey @MarceloNunesAlves

Thanks for this component!

This is probably happening because you are creating a PR from your main branch

@MarceloNunesAlves
Copy link
Contributor Author

Hey @MarceloNunesAlves

Thanks for this component!

This is probably happening because you are creating a PR from your main branch

Do you think I should create a new PR?

The error is about an error 403 HTTP.

@MarceloNunesAlves
Copy link
Contributor Author

Hi @ogabrielluiz,

I understood what happened (autofix.ci / Check Poetry lock), the problem occurs because I don't have write access to the repository, and since this PR came from a fork it ended up giving this error when adding a comment (Posting diff to new comment.).
I believe that if you run that part it will work correctly.

Thank you so much!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 3, 2024
@ogabrielluiz ogabrielluiz merged commit d93c907 into langflow-ai:main Sep 4, 2024
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants