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

New parameter: allow_llm_to_see_data #219

Closed
zainhoda opened this issue Jan 31, 2024 · 1 comment · Fixed by #271
Closed

New parameter: allow_llm_to_see_data #219

zainhoda opened this issue Jan 31, 2024 · 1 comment · Fixed by #271

Comments

@zainhoda
Copy link
Contributor

As we work on new functionality like summarization that requires the ability of the LLM to see the data, we need to introduce a parameter to both vn.ask and the flask run function. This will allow the user to be explicit about whether or not they will allow any part of the database contents to be sent to the LLM.

By default, this parameter should be False

@zainhoda zainhoda linked a pull request Mar 2, 2024 that will close this issue
@CNouraAraujo
Copy link

Is it possible to add this parameter (allow_llm_to_see_data) to an application developed with Vanna AI and Chainlit?

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 a pull request may close this issue.

2 participants