Skip to content

Add SunSpec meter #19624

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add SunSpec meter #19624

wants to merge 5 commits into from

Conversation

benesolar
Copy link
Contributor

@benesolar benesolar commented Mar 9, 2025

Fix #19905

this way it can be used with UI config
For older Symo, Primo, Eco, IG, ... Inverters (all with Display)
and newer GEN24 Symo, Primo, Tauro, Verto.

Than this can be reverted: #19341

@andig andig added the devices Specific device support label Mar 10, 2025
@andig
Copy link
Member

andig commented Mar 10, 2025

It's not clear what the purpose of this PR is. Why maintain an entire config rather than a simple option?

@premultiply
Copy link
Member

Was ist hier der Vorteil eines zusätzlichen Templates statt des vorhandenen Templates?

@benesolar
Copy link
Contributor Author

Anbindung von zweiten Smartmeter am Fronius Gen24 der die Erzeugung von einem alten Wechselrichter misst.

YAML Konfig:

  - name: pv3
    type: custom
    power:
      source: sunspec
      id: 201
      uri: 192.168.1.249:502
      value:
        - 203:W
    energy:
      source: sunspec
      id: 201
      uri: 192.168.1.249:502
      value:
        - 203:TotWhExp
      scale: 0.001

Über das UI kann ich nur eine zusätzliche PV Anlage einbinden.
Daher müsste im aktuellen Template unter usage: pv die ID 201 einstellbar sein, z.B. die Modbus Werte Power: 203:W und Energy: 203:TotWhExp abgefragt werden. Würde das auch funktionieren oder hab ich einen Denkfehler?

@andig
Copy link
Member

andig commented Mar 11, 2025

Warum sollte das nicht funktionieren? Ich sehe dennoch keinen Grund für ein separates Template.

products:
- brand: Fronius
description:
de: Smartmeter (über Wechselrichter)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eigentlich nicht "über WR" sondern statt WR- der Teil kann raus?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wir kommunizieren ja nicht direkt mit dem Smartmeter. Sondern fragen am Wechselrichter die Smartmeterdaten ab.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ist das nicht Semantik? Wir nutzen ein meter model und welches Gerät auch immer hinter genau der IP steckt liefert das meter. Dass das bei Fronius der WR ist wäre ja eher Zufall oder Implementierungsdetail.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Habe ich als Hinweis gedacht, dass klar ist das die Wechselrichter IP einzugeben ist.
Aber wie du das für sinnvoll erachtest.

@premultiply premultiply marked this pull request as ready for review April 8, 2025 19:30
@premultiply premultiply changed the title Add fronius-smartmeter-inverter Add SunSpec meter Apr 8, 2025
@benesolar
Copy link
Contributor Author

benesolar commented Apr 9, 2025

Ohne Hilfe und Default IDs wirds schwierig für den Benutzer.
Lässt sich das alternativ bei den einzelnen unterstützten Geräten hinterlegen? Sicher auch ein Thema bei anderen Templates.

Hier ein Beispiel für die Probleme:
#20509

@github-actions github-actions bot added the stale Outdated and ready to close label Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support stale Outdated and ready to close
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Änderung des sunspec-inverter.yaml, oder Einführung eines sunspec-smartmeter.yaml
3 participants