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

fix: some small changes and fixes for AI assistant #1333

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

jczhong84
Copy link
Collaborator

@jczhong84 jczhong84 commented Sep 20, 2023

  • move the "from logic import vector_store as vs_logic" as delayed import to fix the issue of can't find langchain module when ai assistant is not enabled
  • when the response format of auto fix is not as expected, show the unformatted response anyway
  • Try to not include table/column description or other metadadata if prompt is too long
  • Pass prompt length to _get_llm, so it can be used to determine which llm model to use
  • Fix for some cases the table name in the vector store is not a full table name.
  • update the docs of installing langchain package

@jczhong84 jczhong84 merged commit 21c3fd4 into pinterest:master Sep 20, 2023
5 checks passed
aidenprice pushed a commit to arrowtail-precision/querybook that referenced this pull request Jan 3, 2024
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