Conversation
- Wrote a README to setup the tool. - Wrote test cases. - Added Couchbase as an optional dependency in the project.
…how to create a vector search index.
Added Couchbase as a CrewAI tool
|
@lucasgomide this PR is ready for review! |
|
@capemox Hey! I started reviewing yesterday, but the setup is a bit extensive, so it’s taking me a bit longer to get everything ready. |
|
@lucasgomide Please let me know if I can help in any way, would be happy to help |
|
@capemox Sorry for the delay I haven’t had time to review it yet, but it’s on my radar. |
tks for the update! |
|
@lucasgomide no worries! |
|
@capemox could you share a video executing this tool? I believe would be better for reviewing |
|
@lucasgomide sure. I'll send over a video of how you can set up your cluster, credentials, etc. and use the couchbase SDK to create the tool, and what the output looks like. Is that alright? |
|
A video using and testing the tool is enough |
|
@lucasgomide sorry for the late response! Here's a loom where I'm calling the tool. |
|
@lucasgomide just a reminder that the video and PR is ready |
lucasgomide
left a comment
lucasgomide
left a comment
There was a problem hiding this comment.
Hey @capemox! Welcome, and thanks for your collaboration!
I’ve reviewed your work and watched the Loom video. Regarding the video, I have a couple of points:
1. Would you mind recording another one using an Agent, Tasks, and Crew? I’d love to see that setup in action.
2. You’ve assigned several variables from the OS environment/ is that all really necessary? In most cases, having the environment variable defined once should be enough
|
Hi @lucasgomide, thanks for the reviews! I'll update the code, and I'll send an updated loom video. |
- Removed unnecessary ImportError for missing 'couchbase' package. - Changed response format from a concatenated string to a JSON array for search results. - Updated error handling to return error messages instead of raising exceptions in certain cases. - Adjusted tests to reflect changes in response format and error handling.
|
@lucasgomide here is the updated loom video |
|
Would you mind sync with main again? |
- Changed pydantic version from 2.6.1 to 2.10.6 in both pyproject.toml and uv.lock. - Updated crewai-tools version from 0.42.2 to 0.42.3 in uv.lock. - Adjusted pydantic-core version from 2.33.1 to 2.27.2 in uv.lock, reflecting the new pydantic version.
|
@lucasgomide please have a look now! |
|
Hi @lucasgomide, sorry about the delay. I've resolved the pydantic version issue. |
|
@capemox can you resolve conflicts? |
|
@lorenzejay I've resolved the conflicts! |
|
Hi @lorenzejay all conflicts are resolved now |
|
Any updates on this one? |
|
Hi @lorenzejay @lucasgomide, please have a look at this PR, all conflicts are resolved now |
|
Hi @lucasgomide, these tests run when run locally, but some fail here. What do you suggest I do? |
|
@lucasgomide |
|
@lucasgomide |
i will review again today later |
* - Added CouchbaseFTSVectorStore as a CrewAI tool. - Wrote a README to setup the tool. - Wrote test cases. - Added Couchbase as an optional dependency in the project. * Fixed naming in some places. Added docstrings. Added instructions on how to create a vector search index. * Fixed pyproject.toml * error handling and response format - Removed unnecessary ImportError for missing 'couchbase' package. - Changed response format from a concatenated string to a JSON array for search results. - Updated error handling to return error messages instead of raising exceptions in certain cases. - Adjusted tests to reflect changes in response format and error handling. * Update dependencies in pyproject.toml and uv.lock - Changed pydantic version from 2.6.1 to 2.10.6 in both pyproject.toml and uv.lock. - Updated crewai-tools version from 0.42.2 to 0.42.3 in uv.lock. - Adjusted pydantic-core version from 2.33.1 to 2.27.2 in uv.lock, reflecting the new pydantic version. * Removed restrictive pydantic version and updated uv.lock * synced lockfile * regenerated lockfile * updated lockfile * regenerated lockfile * Update tool specifications for * Fix test cases --------- Co-authored-by: AayushTyagi1 <tyagiaayush5@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
5 participants
This PR adds Couchbase as a tool to perform semantic search over docs in a Couchbase cluster. Have also added tests and a readme. Have also added couchbase as an optional dependency.