esphome / firmware

Holds firmware configuration files for projects that the ESPHome team provides.
https://esphome.io/projects
Apache License 2.0
128 stars 93 forks source link

Images as subsitution, new handling screen update, removed click to converse #116

Closed jlpouffier closed 6 months ago

jlpouffier commented 6 months ago

Removed the click to converse mode from the boxes The top left button is now only used to do a factory reset (by pressing 10 seconds as before)

Display update is now in a single place I renamed the script to draw_display. It is now the only component that is updating the display. In order to achieve that, I created a global called voice_assistant_phase that is representing the state of the voice assistant so that the responsibilities are split.

The values of voice_assistant_phase are the following

The illustration are now defined as substitution So that users can change the illustration after adoption without having to copy paste the whole firmware.

pixeye33 commented 6 months ago

Hello,

Background color defined as text, with transparent png was kinda neat.

Full screen PNG are a plus for easy customisation, but it would be even nicer if they can still be transparent and the background color adjustable ?

as a side-note, i've shared on discord a demo of a automatic variation of backlight based on status, that could be great to implement the same way.

pseudo-code :

substitutions:
  loading_illustration_file: https://github.com/esphome/firmware/raw/main/voice-assistant/casita/loading_320_240.png
  loading_background_color: BLACK
  loading_backlight_level: 30

feel free to simply ignore this 👍

jlpouffier commented 6 months ago

Don't worry I am not ignoring you @pixeye33 :) I am keeping your idea for a better way to customize. I actually like the idea of having both transparent image AND colors as substitutions. I think it gives even more flexibility to our users. Right now I am focusing on making the first "official" release of the S3 Box with the goal of allowing our users to customize it easily. This PR is big enough as it is, but I'll bring this idea up internally afterwards! Thx a lot for your idea!

jlpouffier commented 6 months ago

@pixeye33 Ok so ... I did it :D ef3086c Thx for the idea, it's super cool