Open-Assistant
Open-Assistant copied to clipboard
(WIP)Conversation messages refactor add msg ids
trafficstars
Pull Request for #360
- Still a WIP, still need to update main and potentially update ConversationMessage types to remove optional.
Here the current changes
- in oasst_shared/schemas/protocal.py I modified ConversationMessage adding the variable message_id and frontend_message_id
- in backend/oasst_backend/api/v1 I modified instances of
ConversationMessagein tasks.py and utils.py to include the new variable when instantiatingConversationMessage - Ran pre-commit to format
What still needs to be fixed:
- I'm still trying to understand how to fix what's going on in main.py any advice would be appreciated. @andreaskoepf mentioned using trace_conversation() in prompt_repository.