benleb / sureha

SureHA 🐾 monitor & control your Sure Petcare devices via Home Assistant
Apache License 2.0
21 stars 12 forks source link

Felaqua doesn't update if no pet detected. #6

Closed blair287 closed 3 years ago

blair287 commented 3 years ago

Think this might be a bug

But sometimes the felaqua says XX water has been removed because it didn't see a microchip.

The Dev in chrome shows a log with a new actual weight and change however the HA entity doesn't update.

Screenshot_20210510-180919 Screenshot_20210510-180941~2 PXL_20210510_170935387~2

benleb commented 3 years ago

hm how is this related to #4? or did you forget about? :D :D I have a fix in the pipeline to use the actual_weight field if available - would that be correct from your view?

blair287 commented 3 years ago

hm how is this related to #4? or did you forget about? :D :D I have a fix in the pipeline to use the actual_weight field if available - would that be correct from your view?

Ignore 4 I think might just been a funny 5 minutes.

Current weight is there all the time from what I can see if no pet or pet, current weight is correct

benleb commented 3 years ago

update pushed, but with the actual_weight... if actual_weight is wrong, tell me and we switch to the current_weight ⚖️ (but not today ✌️ editor closed for today :D)

blair287 commented 3 years ago

update pushed, but with the actual_weight... if actual_weight is wrong, tell me and we switch to the current_weight ⚖️ (but not today ✌️ editor closed for today :D)

updated doesnt load integration has an error.

`Logger: homeassistant.setup Source: setup.py:310 First occurred: 20:05:20 (1 occurrences) Last logged: 20:05:20

Unable to prepare setup for platform surepetcarebeta.sensor: Platform not found (cannot import name 'FeederBowl' from 'surepy.entities.devices' (/usr/local/lib/python3.8/site-packages/surepy/entities/devices.py)).`

No problem good work :) Looks like error maybe need surepy updated but no idea how to update that in HA.

benleb commented 3 years ago

ah fuck, forgot to change the required version of surepy in the manifest, should work now :)

blair287 commented 3 years ago

ah fuck, forgot to change the required version of surepy in the manifest, should work now :)

Yep working will monitor next time it doesn't know which pet drank if it still goes down.

Cheers

blair287 commented 3 years ago

ah fuck, forgot to change the required version of surepy in the manifest, should work now :)

Still not updating when it doesn't see pet.

Had 19ml removed at 19:19 but HA shows no change for 7 hours.

blair287 commented 3 years ago

Tested a few more times working perfect when it knows who drank but doesn't update for when it says so much has been removed.

Any way to get round this? The notifications I see in the Dev tools for chrome shows it generates a notification and current weight updates.

benleb commented 3 years ago

Im on it :) cant promise I will finish it today but it will come really soon :)

benleb commented 3 years ago

@blair287, I think it should show always the correct values ;)

but! :D I also implemented this config flow/click-to-config **** so gl updating 😅 maybe you have to delete the config enter from your configuration.yaml and re-add it via the "integrations" menu... (hint: top field is sure petcare username/mail and the lower one the password :D)

blair287 commented 3 years ago

@blair287, I think it should show always the correct values ;)

but! :D I also implemented this config flow/click-to-config **** so gl updating 😅 maybe you have to delete the config enter from your configuration.yaml and re-add it via the "integrations" menu... (hint: top field is sure petcare username/mail and the lower one the password :D)

I managed to update, awesome work on making it able to be added via the UI! Was gonna be my next suggestion 🤣

I'll keep an eye out and see if it works with no pet detected.

Looks really good now would you be able to use custom icons if I sent you the SVG images for the entities like feeder, flap, hub and felaqua?

blair287 commented 3 years ago

@benleb

It's working! It saw the change even with no pet identified! Great work 👌

benleb commented 3 years ago

2021-05-16 09 15 49