-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MiScale 1 doesn't report Lbs #28
Comments
Thanks I'm on holiday at the moment so limited time available. Can you confirm how you run the script? |
I just updated the post to the issue template - Sorry about that, it's actually my first time every reporting a bug on github Enjoy your holiday! When I get the BLE dongle in, I'll take some screen shots of the KG reporting, the JIN, reporting, and then the LBS not reporting. Thanks! |
do you get any error messages when the slider is set to Lbs? |
great stuff. |
something doesn't add up. Just checked the data you sent, and I can clearly see 92Kg on one side, and 203 Lbs on the other side:
Are you sure nothing is posted on MQTT? Have you tried something like MQTT explorer to confirm nothing is published? |
When I listen in on MQTT, only the KG messages show up: Here's the sensor config:
Also...I was able to get the error message: This happened after switching to Lbs from KGs Anything else I should provide? |
I'm not sure what's going on. |
Thanks. Can you try this updated add-on and confirm whether it works pls? |
Thanks, that's what it does indeed, but it is published on MQTT only when the weight changes. you have a special version that logs all loops so I could understand what was failing.
thanks I'd completely overlooked non Kg units for the body metrics. I've made a small adjustment which should cater for it. |
Thanks for being my own personal Guinea pig 👍 |
No Problems - Glad to be able to help with it! One thing more... the MQTT data is malformed when Lbs are used:
Its missing a comma between the weight and the weight unit. Other than that, I think it's all good! |
yes thanks. Forgot to update the file after noticing. If that's ok with you I'll close this issue now. |
Sweet! It all looks good |
A couple of small items, while you're doing the update The MQTT topic that you publish when using Lbs has the "S" as a lower case in "MiScale" The User weight limits are still functioning in KG - which I think is fine, but might be worth mentioning in the readme |
Thanks for your comments.
There was a typo difference indeed between the discovery and the manual entry, especially when left blank. I've just checked the code again and there is no hard coded miScale. I have however updated the README example for consistency
Thanks. I've already updated the README I've also added the weight unit as an attribute as I was originally hoping it would be possible to use a template to set the |
Describe the bug
MiScale 1 does not deliver mqtt messaging to Home Assistant when LBS are selected as the designated unit
To Reproduce
Move the slider on the mi scale 1 to the far left, so that it shows the weight as lbs. It will then no longer report the resulting weight to Home Assistant
Expected behaviour
The weight indicated on the scale would be reported in the MQTT message sent to the MQTT broker.
Screenshots
Scale (please complete the following information):
Desktop/Server (please complete the following information):
Additional context
The text was updated successfully, but these errors were encountered: