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

Update plugin_sofar_old #728

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Conversation

kkubicki
Copy link
Contributor

@kkubicki kkubicki commented Feb 8, 2024

  • Remove special characters from serial number
  • Update entities parameters

These changes I had to make to have full support from HA energy dashboard with my Sofar 2200TL-G3.
I've noticed that the inverter reports serial number with leading two special characters (same for SB1 and SM1E series I suppose):
<00><0C><53><41><33><45><53><32><32><32><4E><33>
I mitigate this issue by stripping seriesnumber from special characters.

@wills106 If you feel this is too extensive change, I can extract it to separate plugin.

- Remove special characters from serial number
- Update entities parameters
@wills106
Copy link
Owner

wills106 commented Feb 8, 2024

seriesnumber.count was only intended as a quick fix for #522 (reply in thread)

I didn't realise the energy dashboard wasn't fully working on sofar_old as no one has reported it.

@kkubicki
Copy link
Contributor Author

kkubicki commented Feb 8, 2024

Yeah, it is probably Product code that is always sent prior serial number, I tried to get it from other address (0x2003, 0x2004) but the result is the same.

In my case, Today and Total Production was missing from the Solar energy sources list.

@wills106 wills106 merged commit 737fee1 into wills106:main Feb 8, 2024
2 checks passed
@kkubicki kkubicki deleted the feature_update_sofar_old branch February 8, 2024 14:58
@kkubicki
Copy link
Contributor Author

kkubicki commented Feb 8, 2024

@wills106 I have seconds thoughts regarding ActivePower change from KILO_WATT_HOUR to KILO_WATT (this change applies only to X3 inverters, for X1 ActivPower was already KILO_WATT). I do not know what is the correct unit for this parameter, in my case (X1 inverter) it seems to be actual power sent out so kW makes more sense.
There seem to be some confusion regarding this, some plugins use kWh, other ones kW...

@wills106
Copy link
Owner

wills106 commented Feb 8, 2024

kW tends to be used for Grid connected like import / export. Edit: I meant to say realtime import / export. Historic like daily or yearly tends to be in kWh. Atleast on the SolaX

kWh used for transfer to and from battery. Edit: and historic like above.

I haven't pushed it out as a release yet, so you can relook at it and see what fits best.

@wills106
Copy link
Owner

wills106 commented Feb 8, 2024

On my SolaX some realtime:

image

Daily totals / historic:
image

image

Does the Sofar LCD give some pointers?

@kkubicki
Copy link
Contributor Author

kkubicki commented Feb 8, 2024

It is a very simple inverter, LCD has only a couple of lines. Here's excerpt from manual:

Standard display
This shows a rolling display of the DC voltage/current (PV1), energy
yields (today/total), grid voltage and current, as well as the status.

I can push another commit that restores previous settings for X3 inverters, and leaves X1 changes only. That would be safest thing to do, I do not want you to have extra work because of my changes. On the other hand, unit of Power is kW, not kWh after all.

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

Successfully merging this pull request may close these issues.

2 participants