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

sma integration doesn't support inverters with 3 inputs #52850

Closed
scotte opened this issue Jul 10, 2021 · 2 comments
Closed

sma integration doesn't support inverters with 3 inputs #52850

scotte opened this issue Jul 10, 2021 · 2 comments
Assignees

Comments

@scotte
Copy link

scotte commented Jul 10, 2021

The problem

The SMA solar inverter integration doesn't currently support inverters with 3 inputs.

It's a fairly easy fix and I'll look into submitting a pull request. The following hack to lib/python3.8/site-packages/pysma/__init__.py resolves it:

@@ -143,10 +143,13 @@
                     #   Old 6100_0046C200 is the power generated by the inverter (always equal to grid_power)
                     Sensor("6380_40251E00_0", "pv_power_a", "W"),
                     Sensor("6380_40251E00_1", "pv_power_b", "W"),
+                    Sensor("6380_40251E00_2", "pv_power_c", "W"),
                     Sensor("6380_40451F00_0", "pv_voltage_a", "V", 100),
                     Sensor("6380_40451F00_1", "pv_voltage_b", "V", 100),
+                    Sensor("6380_40451F00_2", "pv_voltage_c", "V", 100),
                     Sensor("6380_40452100_0", "pv_current_a", "A", 1000),
                     Sensor("6380_40452100_1", "pv_current_b", "A", 1000),
+                    Sensor("6380_40452100_2", "pv_current_c", "A", 1000),
                     Sensor("6400_0046C300", "pv_gen_meter", "kWh", 1000),
                     Sensor("6400_00260100", "total_yield", "kWh", 1000),
                     Sensor("6400_00262200", "daily_yield", "Wh"),

What is version of Home Assistant Core has the issue?

2021.6.6

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Core

Integration causing the issue

sma

Link to integration documentation on our website

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@probot-home-assistant
Copy link

Hey there @kellerza, @rklomp, mind taking a look at this issue as its been labeled with an integration (sma) you are listed as a codeowner for? Thanks!
(message by CodeOwnersMention)

@kellerza
Copy link
Member

kellerza commented Jul 12, 2021

Thanks for the issue&fix, @scotte !

Please open a PR on the pysma repo

@github-actions github-actions bot locked and limited conversation to collaborators Aug 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants