search

ofalvai / home-assistant-candy

Unofficial Candy/Haier appliance integration for Home Assistant
129 stars 29 forks source link

Remaining minutes issue #252

Closed Tassel8062 closed 1 year ago

Tassel8062 commented 1 year ago

Anyone know why my sensor.wash_cycle_remaining_time only displays the value 1 or 2 ? But on my washer it shows like this

PXL_20230129_091117747 MP

pyzimmer commented 1 year ago

Hello, #51 , can you close ?

Tassel8062 commented 1 year ago

Not sure #51 solves my issue. HA shows the senor value in minute(s) with either a value of 1 min or 2 min, how can I create a template sensor to show the actual mins

image

gee-jay-bee commented 1 year ago

Agreed. I had to remove the /60 on candy/client/model.py:66 to make my washing machine report the correct time:

            remaining_minutes=int(json["RemTime"]),

The value returned above really has precision of minutes so 1 hour 40 is reported as 100 minutes.

I think a template sensor might help if the washing machine is reporting remaining time as hours but we lose precision with the current implementation where machines really report minutes.

glebsterx commented 1 year ago

Hello! I have the same problem and the same solution -- remove / 60 in the code of the model.py file image Now the time is displayed normally

I already wanted to create an issue, but I saw that this one already exists =) I could make a pull request, but I believe that the author had reasons to do exactly as it is done now and if he told us it would be cool =) I made a PR, hope it helps someone =)

P.S. I have a Candy CVFTGP384TMH-07 washing machine

Tassel8062 commented 1 year ago

Agreed. I had to remove the /60 on candy/client/model.py:66 to make my washing machine report the correct time:

            remaining_minutes=int(json["RemTime"]),

The value returned above really has precision of minutes so 1 hour 40 is reported as 100 minutes.

I think a template sensor might help if the washing machine is reporting remaining time as hours but we lose precision with the current implementation where machines really report minutes.

This fixed it for me ! Thanks