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

feat: Store Conversation History #77

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

dasmy
Copy link
Contributor

@dasmy dasmy commented Aug 3, 2023

With this change, we are storing the relevant parts of the full conversation history (instead of only keeping the user prompts). This makes full use of the chat-API and back-reference to earlier answers easier. With also including generated output or error messages from the code, it is possible to directly ask GPT to fix the error message, which partially solves #42.

I am reluctant to automatically ask for a fix in case of an error during execution as this might introduce endless loops if no proper fix is been generated.

… way, referring to previous code is simpler.
message_status for 'Kernel is ready' and the like
message_error for error messages/backtraces from execution
…allow for easier back-reference, e.g. when asking 'Please fix the code'
There is no real documentation on which difference this makes, but at least semantically, this is correct.
@dasmy dasmy changed the title feat: Store Full Conversation History feat: Store Conversation History Aug 3, 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

1 participant