Home Assistant - Auto register sensor

[hutten0]

First, thanks for a nice firmware! Just love to be able to log more than one endpoint!
It seems to be wrong device classes in the template for Home Assistant - Auto register sensor.
Both gravity and tilt has "temperature". I think they shouldn’t have any at all.
A couple of wishes:

• In tilt and angle, two more digits please, or at least in one of them.
  I’m planning to calibrate by fermenting a sugar wash and take measurements along the way.
• Add Battery % as in “device_class” “battery”

I never managed to succeed with Home Assistant - Auto register sensor, so I made a script in HA to do the job and it’s persistent to.

Cheers

[mp-se]

First, thanks for a nice firmware! Just love to be able to log more than one endpoint! It seems to be wrong device classes in the template for Home Assistant - Auto register sensor. Both gravity and tilt has "temperature". I think they shouldn’t have any at all. A couple of wishes:

• In tilt and angle, two more digits please, or at least in one of them.
  I’m planning to calibrate by fermenting a sugar wash and take measurements along the way.
• Add Battery % as in “device_class” “battery”

I never managed to succeed with Home Assistant - Auto register sensor, so I made a script in HA to do the job and it’s persistent to.

Cheers

Thanks for the feedback.

For the temp I can agree that one more digit would be good to have. I don't really see the need for having 3 digits on the temp since the max accuracy is 0.05C anyway when using 12 digits resolution.

On the gyro I believe the accuracy is 1% so more than 2 digits would not give you any better accuracy anyway.

On the device classes for HA I agree that that should be changed. I will add that to the backlog.

The way the auto registration works is that you need to run it once when in configuration mode for the entities to register correctly but they will be lost when you restart home assistant. During fermentation it only sends the data parts to reduce the time needed for writing to mqtt. I could add an option so that you can choose if the auto registration should be done at every data transmit. That would guarantee that the values are persisted every time. Let me know what you think ?

[mp-se]

I looked into the battery part and according to the HA documentation that value should be a percentage of the charge. I would suggest that this is added as a new parameter since the existing voltage is aligned with HA sensor types.

[hutten0]

Perfect, thanks!

[hutten0]

I was looking in the dev branch, and I think there is a typo in service.rst for gravity. retain: false should be true, right?

[mp-se]

Yes, that was a typo

[mp-se]

You can try the version in the dev branch if you want. I still have more tests to do before I push it to master branch.