Skip to content
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

AC500+2B300S difference between 0.0.16 and 0.0.20 #76

Closed
ItsMe00007 opened this issue Mar 16, 2024 · 14 comments
Closed

AC500+2B300S difference between 0.0.16 and 0.0.20 #76

ItsMe00007 opened this issue Mar 16, 2024 · 14 comments

Comments

@ItsMe00007
Copy link

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
Copy link
Owner

Patrick762 commented Mar 16, 2024

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
Copy link
Owner

Patrick762 commented Mar 16, 2024

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
Copy link
Owner

Changes are now in Pull Request #77

@Patrick762
Copy link
Owner

Looks like Enums now work as expected

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

@Patrick762
Copy link
Owner

@ItsMe00007 New version is now available: 0.0.21

@Patrick762 Patrick762 self-assigned this Mar 16, 2024
@Patrick762 Patrick762 added bug Something isn't working verified bug Bug verified by Repository owner labels Mar 16, 2024
@ItsMe00007
Copy link
Author

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
Copy link
Owner

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
Copy link

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

@ItsMe00007
Copy link
Author

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
Copy link
Owner

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

@ItsMe00007
Copy link
Author

ItsMe00007 commented Apr 1, 2024

@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
Copy link
Author

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

Bildschirmfoto 2024-04-01 um 15 05 38

@ItsMe00007
Copy link
Author

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
Copy link
Owner

Should be fixed in Release 0.1

@Patrick762 Patrick762 removed bug Something isn't working verified bug Bug verified by Repository owner labels May 31, 2024
@Patrick762 Patrick762 removed their assignment May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants