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

Langchain::Tool::Database#describe_table with COMMENT #891

Merged

Conversation

Bahanix
Copy link
Contributor

@Bahanix Bahanix commented Dec 1, 2024

Allows LLM to make use of columns' comments.
Possible since Sequel 5.87.0 (PR).

Example of usage:

Given this structure:

CREATE TABLE users(
  id integer PRIMARY KEY,
  name string COMMENT 'Internally known as 'foobar''
);

I can know ask things like Give me the last registered user's foobar to the LLM.

@andreibondarev
Copy link
Collaborator

@Bahanix Thank you for the PR. Pretty sure you need to update the version in the gemspec file as well.

@Bahanix Bahanix force-pushed the database-with-comments branch from fb18048 to 5f2d964 Compare December 2, 2024 19:37
@Bahanix
Copy link
Contributor Author

Bahanix commented Dec 2, 2024

@andreibondarev the problem was in the Gemfile.lock, this commit should fix the build 🤞

@Bahanix Bahanix force-pushed the database-with-comments branch from 5f2d964 to b4eede7 Compare December 2, 2024 19:40
Allows LLM to make use of columns' comments.
Possible since Sequel 5.87.0.
@Bahanix Bahanix force-pushed the database-with-comments branch from 2cc34e3 to 1bad4a5 Compare December 3, 2024 09:42
@andreibondarev andreibondarev merged commit b16ba86 into patterns-ai-core:main Dec 3, 2024
5 checks passed
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