Patrick762 / hassio-bluetti-bt

Bluetti Integration for Home Assistant
MIT License
30 stars 9 forks source link

AC500+2B300S difference between 0.0.16 and 0.0.20 #76

Closed ItsMe00007 closed 1 month ago

ItsMe00007 commented 3 months ago

There are quite some sensors which don't work anymore or show wrong values after updating from 0.0.16 to 0.0.20. I thought you were just adding new Powerstations and the working ones don't get changed, but it seems you changed a lot.

First: how it looked with 0.0.16:

Result1 AC500 and 2xB300S Bluetti BT 0 0 16 Result2 AC500 and 2xB300S Bluetti BT 0 0 16

And with 0.0.20:

  1. AC DC output controls: you can't really see if they are on or off, the 0.0.16 way showed it clearly
  2. AC Input Frequency is now "unknown"
  3. AC Output mode only shows numbers, no text.
  4. Battery Range start/End is missing.
  5. Grid charge is now "unknown"
  6. internal AC Frequency is 500Hz instead of 50Hz
  7. internal AC Voltage is 2288V instead of 228,8V
  8. Split phase machine type ist "unknown"
  9. Time Control is "unknown"
  10. UPS Working mode is "unknown"
  11. Battery Pack 1 and 2 Infos are "unavailable"
Result1 AC500 and 2xB300S Bluetti BT 0 0 20 Result2 AC500 and 2xB300S Bluetti BT 0 0 20
Patrick762 commented 3 months ago

I changed the base for the integration so it has a smaller footprint in total, but might have missed some parts. I'll look into it.

I have only tested the EP600 and EB3A with the changes.

Patrick762 commented 3 months ago

FYI those are the things I missed:

  1. Added register polling for those addresses
  2. Added register polling for this address
  3. Not changed, should already work
  4. Added register polling for those addresses
  5. Added register polling for this address
  6. Changed scaling
  7. Changed scaling
  8. Added register polling for this address
  9. Added register polling for this address
  10. Added register polling for this address
  11. Removed addresses from normal polling (now it's pack polling only)
Patrick762 commented 3 months ago

Changes are now in Pull Request #77

Patrick762 commented 3 months ago

Looks like Enums now work as expected

Screenshot 2024-03-16 at 17-47-47 Einstellungen – Home Assistant

Patrick762 commented 3 months ago

@ItsMe00007 New version is now available: 0.0.21

ItsMe00007 commented 3 months ago

Hi Patrick, wow, that was really fast !!!! Looks much better now.

Only the Battery Pack 1 and 2 are "unavailable". They were working in 0.0.16.

Bildschirmfoto 2024-03-16 um 20 07 29
Patrick762 commented 3 months ago

Pack polling takes around 5 seconds per pack.

If I have some time tomorrow I'll test the latest version on an AC300 (very similar)

danrichardson commented 3 months ago

I can confirm that the battery pack values are not present on my AC300 0.0.21 as well.

ItsMe00007 commented 3 months ago

AC500+2xB300S: Current status with 0.0.21:

  1. Battery Pack Info is ok in 0.0.16, but unavailable in 0.0.21.

  2. All 3 DC Internal Inputs have wrong scaling factor: Voltage: should be 90,1 instead of 901 Power: should be 303 instead of 30,3 Current: should be 3,3 instead of 0,33

Then it would match the Power/Voltage/Current from PV Input 1 (currently measured externally)

Patrick762 commented 3 months ago

@ItsMe00007 Can you tell me which entity names you mean in 2.?

ItsMe00007 commented 3 months ago

@ItsMe00007 Can you tell me which entity names you mean in 2.?

I mean these entities: Internal DC Input Current Internal DC Input Power Internal DC Input Voltage

Scaling should be: Internal DC Input Current: x100 Internal DC Input Power: x10 Internal DC Input Voltage: x0.1

ItsMe00007 commented 3 months ago

I just tested the new 0.0.22: AC500 Battery Pack info is still "unavailable"

Bildschirmfoto 2024-04-01 um 15 05 38
ItsMe00007 commented 1 month ago

I just tested the new 0.0.23:

AC500 Battery Pack info is still "unavailable", the latest Version where it worked was 0.0.16. I have automations which depend on these values, therefore I had to go o 0.0.16.

Any chance to get that fixed ?

Bildschirmfoto 2024-05-06 um 07 54 52
Patrick762 commented 1 month ago

Should be fixed in Release 0.1