Howie/fully customizable event handler #14
Conversation
src/quartapp/chat.py
Outdated
|
|
||
| if stream_data: | ||
| yield f"data: {stream_data}\n\n" | ||
| async for _, _, to_be_yield in stream: |
There was a problem hiding this comment.
I prefer you get the concept from my inline comment and variable names rather than answering here. Let me know if my new comment answer your question.
| self, message: "ThreadMessage" | ||
| ) -> Optional[str]: | ||
| if message.status == "completed": | ||
| annotations = [annotation.as_dict() for annotation in message.file_citation_annotations] |
There was a problem hiding this comment.
Are annotations always citations? Do they have any other additional information?
There was a problem hiding this comment.
Are annotations always citations?
No, a while ago there was issue that the delta text has annotation, but we got no citation. Backend claimed they have fixed it. To do our best, we collect all citation and convert the anntations into clickable links only if the corresponded citation exist.
Do they have any other additional information?
Yes, but for citation, I only care about file_citation_annotations.
|
Are PRs to this repo always into main? |
use the new version of event handler. Fixed citation.