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

[Bug]: "Add support for user-defined extra sensors" seems to be out of date #769

Closed
5 tasks done
BradleyFord opened this issue Jul 15, 2024 · 2 comments
Closed
5 tasks done
Labels
bug Something isn't working

Comments

@BradleyFord
Copy link

Describe the issue

I am struggling to replicate due to the large number of underlying code changes, if is possible to update this commit to be a better reflection of the current changes require?

f2b98c9

Describe your Huawei Solar Setup

Inverter Type:
Inverter Firmware version:
sDongle present: Yes / No
sDongle Type: sDongleA-05 (WiFi / Ethernet) / SDongleA-03 (4G)
sDongle Connectivitiy: WiFi / Ethernet / 4G
sDongle Firmware:
Power meter present: three phase / single phase / no
Optimizers Present: Yes / No
Battery: LUNA2000-SO xxkWh / LUNA2000-S1 xxkWh (Released 2024) / LG RESU xxkWh / None
Battery Firmware version:
Huawei Solar integration version:

How do you connect to the inverter?

Via the SDongle, wired connection

Upload your Diagnostics File

Drag & Drop your Diagnostics File here.

Upload your relevant debug logs

none

Please confirm the following:

  • I'm running the latest release of Home Assistant.
  • I'm running the latest release of this integration.
  • I did not find an existing issue describing this problem.
  • I did upload the diagnostics-file that I could retrieve from the 'Devices & Services Page'
  • I did upload the relevant debug logs (via 'Enable Debug Logging'-feature or by manually configuring HA logging)
@BradleyFord BradleyFord added the bug Something isn't working label Jul 15, 2024
@BradleyFord BradleyFord changed the title [Bug]: "Add support for user-defined extra sensors" seems to be way out of date [Bug]: "Add support for user-defined extra sensors" seems to be out of date Jul 15, 2024
@Roving-Ronin
Copy link
Contributor

@BradleyFord if its of assistance, have a look at this current pull request: #768
This was adding 5 sensors (and removing 2 that give incorrect readings). With the Library ( https://github.com/wlcrs/huawei-solar-lib ) already defining these modbus registers, it is a matter of adding them into the sensor.py, strings.json and language files. You do need to keep the sensors in the same order as the corresponding register for them is listed in the library.

@wlcrs
Copy link
Owner

wlcrs commented Jul 21, 2024

I've removed the outdated branch and did a small update to the wiki. But mainly, as Roving-Ronin said: look at #768 to see how you can add extra sensors.

@wlcrs wlcrs closed this as completed Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants