Willow-Systems / pebble-home-assistant

A pretty Pebble app to control Home Assistant
MIT License
32 stars 3 forks source link

initial flow between pebble dictation and HA conversation #12

Closed cgbaker closed 1 year ago

cgbaker commented 1 year ago

This PR seeks to address #11 . It uses Pebble built-in dictation support and the Home Assistant Conversation API. I'm creating it as a draft for feedback. There's a few things:

So, that's not helpful. I'll keep looking.

cgbaker commented 1 year ago

@Willow-Systems , let me know if you have any feedback. I'll try to find some time to get this working the rest of the way and document any changes needed to my HA configuration.

cgbaker commented 1 year ago

Okay, even though I didn't have a Voice Assistant pipeline configured for the OpenAI integration, it was still using that for some reason? As soon as I disabled the integration, the conversation API started working. I'm going to open this PR in case you want to merge it.

Willow-Systems commented 1 year ago

Hey! This looks great! I'll have a play with this this week and see if we can get it merged! Doesn't look like there's much work on my end

Time to upgrade my HA instance again first 😬

cgbaker commented 1 year ago

Apparently the conversation system (and presumably it's API?) were introduced in HA 0.7: https://www.home-assistant.io/integrations/conversation/

So maybe hit that endpoint on your HA instance see before attempting the upgrade?

Willow-Systems commented 1 year ago

I played around with a more animated UI:

But I've landed on something more useful:

pebble_screenshot_2023-06-06_16-16-26

Willow-Systems commented 1 year ago

But other than UI changes I made basically no changes to your code @cgbaker. Thanks so much for adding this. I'll merge the new voice branch into master later today and update the app on the Rebble appstore :rocket:

I'll be sure to credit you on the appstore listing <3

Willow-Systems commented 1 year ago

Closing as it's now superseded by #13