Open-Assistant icon indicating copy to clipboard operation
Open-Assistant copied to clipboard

(WIP)Conversation messages refactor add msg ids

Open danielpatrickhug opened this issue 2 years ago • 0 comments
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 ConversationMessage in tasks.py and utils.py to include the new variable when instantiating ConversationMessage
  • 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.

danielpatrickhug avatar Jan 06 '23 15:01 danielpatrickhug