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

ShareGPT for index and code #470

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

Conversation

kepler452b123
Copy link

Added Share Button functionality to code and index chat. The logic for the buttons are the same as for text_service_cog (converse chat). Had to find a way to store all prompts/replies for index and code chat. Did this by making a new dictionary instance in index_service_cog.py and by adding to the converser.cog.full_conversation_history instance variable in code_interpreter_service_cog.py. Moved end message logic from index_model.py to index_service_cog.py for consistency and simplicity of integrating the Share Button (since the conversation history was stored in index_service_cog.py and not index_model.py).

Stored conversation history by appending prompts and responses
to self.converser_cog's instance of full_conversation_history.
The logic for ShareView and ShareButton is exactly the same as index
text_service_cog.py. The view is sent along with the link to the
sandbox in code_interpreter_service_cog, however.
Made a new instance variable in index_service_cog.py,
full_conversation_history (similar to the one in text_service_cog.py)
Adds all index chat prompts and responses to full_conversation_history.
Changed the end/stop conversation handler to index_service_cog.py from
index_model.py. Sends the ShareView when it detects the end/stop,
same as in text_service_cog.py and code_interpreter_service_cog.py.
Typecasted Response object to string so that it could be serialized
into JSON and sent to ShareGPT.
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.

1 participant