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

Ability to override prompt config section with program specific vals #13

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

dmgerman
Copy link

@dmgerman dmgerman commented Dec 9, 2023

It is now possible to have system and context in a program section. For example:

[prompt-sqlite3]
system=You are an assistant who provides SQL commands for sqlite3. ... 
context=1

In addition, add verbose debugging errors and more importantly, if the config does not parse, output the tokens that are responsible for this problem.

It also adds a verbose message when the prompt is send to OpenAI. Other backends might need to be updated.

It is now possible to have system and context in a program section.
For example:

[prompt-sqlite3]
system=You are an assistant who provides SQL commands for sqlite3. ...
context=1

In addition, add verbose debugging errors and more importantly,
if the config does not parse, output the tokens that are responsible
for this problem.

It also adds a verbose message when the prompt is send to OpenAI.
Other backends might need to be updated.
@dmgerman
Copy link
Author

I think there is still a need for an ENV variable. I would like to pass the schema of the database, but that has to be done by the invocation of SQLIte (would be way easier, as ai-cli would not need to know anything about SQLite).

With this patch a config file can be replaced when invoking the program, and that config file can have different prompts...

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@dspinellis dspinellis merged commit e605c21 into dspinellis:main Dec 12, 2023
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.

None yet

3 participants