Added Home-Assistant autodiscovery MQTT

[FastFrank]

Hello,

i added some code for HA autodiscovery so the Classic shows up as a MQTT device.
It also sends different icons for the chargestate and SOC.

Do you (eode) use HA?

Greetings!
(and thank you for fixing the python 3 stuff)

[eode]

Thanks for your changes! After review issues are addressed I'll do a PR upstream.

Yes, I use HA myself -- currently, I'm doing some of this info in HA templates (like the charge state text), but I think it's a decent idea to move it into MQTT.

[eode]

Sweet. Overall, it looks good enough at this point. But, I would like to lock these changes behind a CLI flag, like --homeassistant.

After that, I'm ok with it as-is.

[eode]

I can merge this now -- thanks for all of your work on it!

I do still have some small areas of contention, but you might have your reasons for them -- For example, I think the SOC should perhaps be HA's device_class battery, and its icon should then automatically reflect the state.. ..but if we want to refine that, we can do it in another PR.

Thanks for your contribution!

[FastFrank]

I can merge this now -- thanks for all of your work on it!

I do still have some small areas of contention, but you might have your reasons for them -- For example, I think the SOC should perhaps be HA's device_class battery, and its icon should then automatically reflect the state.. ..but if we want to refine that, we can do it in another PR.

Thanks for your contribution!

I am glad i can contribute...
I did not know that HA has a battery device_class... i am trying it now...
But... i did make it the way that if the Charge State is Charging the icon contains the lightning symbol and if the classic is not charging is does not contain the lightning symbol.

Maybe one day we can reach the point that it is possible to write to the register to change the settings of the Classic.

Thanks!

[FastFrank]

It turned green on the graph thing...