-
Notifications
You must be signed in to change notification settings - Fork 29
A few suggestions #120
Comments
Hi,
|
Thank you very much for your prompt answer. Regarding programs - my bad, I didn't checked the situation with the switched off appliance. However a configuration option may be suggested for the integration, and based on its value new appliances will be added with "standard" list (by the way, can we extend it with positions from my list?) or wait for the appliance to be online to add it with proper options and programs. Regarding status, am I correctly understand that I should fork your project, test it and then send you a pull request? I never used Python before, but it may be interesting. Regarding the meaningful events (like program finished, empty condensation container etc), with all my respect, I don't understand what you have suggested. I can see that you code receive the event, however I don't understand what exactly do you recommend. If I correctly understand an architecture of HA, such events should be implemented as integration-specific triggers. However it may be a more simple way, to make a sensor with last event status, and use a standard trigger for sensor value change. |
Hi David, it took me a few hours to learn basics of Python and HA architecture, and I think I've found a solution for 2) and 3) requests from my post. I've added a fourth sensor that translates BSH.Common.Status.OperationState into its value, and show all other pairs from appliance.status as its attributes. With such sensor, it is possible to create any automation that triggers from change of the sensor itself or its attributes, and act accordingly to program finish, for example. It looks like this (some enums are not translated): There is a lot of things to be improved, and I am afraid it will also require some changes in homeconnect module. However I hope to find a way not to touch the base api module and work only on component level. I also definitely need to learn Python better, last time I've programmed about 10 years ago and it was C-Sharp... |
David hi, are you ok if I fork your project in order to resolve your initial problem with program switches? I have both washer and dryer, and a lot of entities as a result :) Moreover, your initial suggestion that follow the programs list from API will be enough is not correct, my dryer has at least two programs that were not included in your list:
|
Hi, Santa! I have Bosch dishwasher and tried to find its speciffic programs from api, but without any luck... |
Hi Radoslav, what exactly you have tried?
|
Hi, I never replied to this question
but of course I'm totally happy if you fork - this integration is even part of HA Core, so you can also submit your improvements directly there. I currently don't have a lot of time to work on this (as you noticed I guess). |
Dear David, first of all let me show my gratitude for your great work.
I have three HomeConnect-enabled appliances, a Neff Hob, and Siemens Washer and Dryer (separate appliances). With all respect to the current versions, I'd like to specify a few things that may be easily improved.
An API request gives the following results for me, which totally match the listed programs on the machine control pane. May you amend your function DeviceWithPrograms.get_programs_available() to make a request to https://api.home-connect.com/api/homeappliances//programs and return the actual list?
Thank you one more time,
Dmitry
The text was updated successfully, but these errors were encountered: