TillFleisch / ESPHome-Philips-Smart-Coffee

ESPHome components which implement a Philips Series 2200/3200 Coffee Machine into HomeAssistant. Capable of brewing automatic coffee.
Other
162 stars 25 forks source link

Fix missing idle state on some EP2220 machines #45

Closed TillFleisch closed 6 months ago

TillFleisch commented 6 months ago

This PR relaxes the conditions for idle state detection w.r.t. different led states for the Steam option on some machines.

Try this PR using:

external_components:
  - source: github://TillFleisch/ESPHome-Philips-Smart-Coffee@fix_missing_idle_state
    refresh: 30min

close #41

TillFleisch commented 6 months ago

@davidepanato can you check if this resolves your issue regarding #41?

davidepanato commented 6 months ago

Just tested and now it works perfectly! I can see at power on all the states, preparing, cleaning and then idle. It also works when shutting down. Good work

davidepanato commented 6 months ago

is it possible that same issue is present in the bean and coffee size? I always see the values as NA

TillFleisch commented 6 months ago

What does the status sensor report when you have coffee selected? It should report 'Coffee selected'. Note that the beverage settings (number component) are only available/working when the provided source is selected. If source: COFFEE is used in the configuration file for the number component, it should only be available when coffee is currently selected. When using source: ANY the number component should be available on all beverages (but not in the Idle state). Do logs provide any useful information?

davidepanato commented 6 months ago

Correct till, if I set source: ANY it works, previously I had it configured as COFFEE but I usually prepare espresso. I think that this modification is ok!

mikulik86 commented 6 months ago

Thanks @TillFleisch, your work is greatly appreciated!