-
Notifications
You must be signed in to change notification settings - Fork 34
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
allow conversations to continue with %%assist
#66
Conversation
This relies on previous `GenaiMarkdown` displays to form the basis of past assistant communication.
Interesting little prompt response I just got:
|
genai/display.py
Outdated
self._message: str = message | ||
self._display_id: str = hexlify(os.urandom(8)).decode('ascii') | ||
self._stage: Optional[Stage] = stage | ||
|
||
if execution_count: | ||
self.assists[execution_count] = self |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe instead of including this in GenaiMarkdown
as a side effect, there should be a separate execution_count
-> assist
singleton for lookup.
This captures past assistance as well as errors so that ChatGPT has access to the full context of:
This PR closes #63 since now the
%%assist
magic uses Markdown output instead ofset_next_input
. It also closes #56 since the progress display is no more.Bonus: hot swapping the models
Think a little more on how best to capture past assistant messages (does it belong inCreated a separateGenaiMarkdown
?)PastAssistant
singleton