Skip to content

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

Merged
jczhong84 merged 1 commit into
pinterest:masterfrom
jczhong84:chore/llm
Sep 20, 2023
Merged

jczhong84 merged 1 commit into
pinterest:masterfrom
jczhong84:chore/llm

Conversation

@jczhong84

@jczhong84 jczhong84 commented Sep 20, 2023 •

Copy link
Copy Markdown
Collaborator
  • 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
aidenprice pushed a commit to arrowtail-precision/querybook that referenced this pull request Jan 3, 2024
@jczhong84
jczhong84 deleted the chore/llm branch March 6, 2025 07:49
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