Closed C-Loftus closed 1 month ago
Have not tested this yet. Feel free to test if you want. Going to take a break for a bit.
Believe this should be good now. Tested with threading / context / images etc, and doesn't seem to have any regressions. The threading is very cool thank you for your work on this. I find it works well. I will add funcitonality to have the imgui window auto-update with a new thread response once this merged.
I'll use this branch for a while. Looks good though!
Kept same behavior the same as far as I can tell.
any
as a key or value. Try to useGPTMessage
orGPTMessageItem
as the type as much as possible. Let me know if there is a name you prefer instead of these. I wasn't sure what to name them.standard
type checking{user.model} **start** thread:
since it is one syllable instead ofenable
which is 3 syllables.contextual_user_context
to begpt_additional_user_context
since it is a globally exposed talon python function, we want to prefix it withgpt
. I personally findadditional_user_context
to be more clear thancontextual_user_context
which strikes me as a bit confusing.I am using
TypeDict
so we keep all the methods within the standarddict
class. Was considering creating a super class for the message and inheriting from it but I think this abstraction does not actually benefit and adds boilerplate.