C-Loftus / talon-ai-tools

Query LLMs and AI tools with voice commands
http://colton.place/talon-ai-tools/
MIT License
46 stars 17 forks source link

Adjust verbosity of model window commands #132

Closed jaresty closed 1 month ago

jaresty commented 1 month ago
C-Loftus commented 1 month ago

The reason I have this more verbose is given the fact that it could conflict with other user commands related to pasting since it is pretty generic.

But I think this is fine, and probably not too likely to conflict. Ok with merging if you want.

jaresty commented 1 month ago

That's a good point-I followed the pattern of the homophones window to enable it only when the window is open now.

jaresty commented 1 month ago

I'd like to also add context/thread to this.

C-Loftus commented 1 month ago

Looks fine but a bit worried that 6 buttons on one window is a bit much. Let's keep it for now but might want to refactor this later or remove buttons. Will check if there is a way to condense things

jaresty commented 1 month ago

It is a lot of buttons so I definitely would support a way to condense it if it's possible-however I'm pretty glad that there is some way to interactively select how you want to handle the model response. It's pretty powerful!

jaresty commented 1 month ago

Let me know when you're okay to merge this-we can always adjust later!

C-Loftus commented 1 month ago

Talon currently can't do horizontal buttons or columns in imgui. So I guess for right now we can just merge this since there isn't a better option. I am not sure if model chain is needed as a separate button since it is just a helper and doesn't add any behavior really, but we can keep it

jaresty commented 1 month ago

Personally I really like model chain-I think it's really handy to have it in the window UI especially since you're pausing for a moment to think about the response you're more likely to know that you want to select it when you input it as well. I'm gonna merge this for now and we can think about it some more-thanks!