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

Fronius Solar API V1: add pv lifetime energy #19192

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

Conversation

AloisKlingler
Copy link
Contributor

like #19095 the same for Solar API

sample json output of an inverter:

{
   "Body" : {
      "Data" : {
         "Inverters" : {
            "1" : {
               "DT" : 233,
               "E_Day" : 0,
               "E_Total" : 25624600,
               "E_Year" : 781735.5,
               "P" : 0
            }
         },
         "Site" : {
            "E_Day" : 0,
            "E_Total" : 25624600,
            "E_Year" : 781735.5,
            "Meter_Location" : "unknown",
            "Mode" : "produce-only",
            "P_Akku" : null,
            "P_Grid" : null,
            "P_Load" : null,
            "P_PV" : null,
            "rel_Autonomy" : null,
            "rel_SelfConsumption" : null
         },
         "Version" : "12"
      }
   },
   "Head" : {
      "RequestArguments" : {},
      "Status" : {
         "Code" : 0,
         "Reason" : "",
         "UserMessage" : ""
      },
      "Timestamp" : "2025-02-26T06:50:19+01:00"
   }
}

@andig andig added the devices Specific device support label Feb 26, 2025
@andig andig marked this pull request as draft February 26, 2025 08:28
remove wrong copied password
@andig
Copy link
Member

andig commented Feb 27, 2025

add pv lifetime energy

Du hast Battery hinzu gefügt, nicht pv?

@AloisKlingler
Copy link
Contributor Author

add pv lifetime energy

Du hast Battery hinzu gefügt, nicht pv?

so, jetzt ist es hoffentlich nur für pv demnach aktiv.

@andig
Copy link
Member

andig commented Feb 27, 2025

Nur zur Sicherheit- testest du das auch?

@AloisKlingler
Copy link
Contributor Author

Nur zur Sicherheit- testest du das auch?

🙈🙈🙈
Leider aktuell keine Ressourcen dass ich mir ansehe und installiere was für's bauen notwendig wäre. 😞
Statt ein Ticket zu machen, das dann mit PR welcome liegenbleibt, hab ich mich gleich mit dem PR versucht.

Wenn du die Möglichkeit hast das in einem docker auf :nightly bauen zu lassen schau ich es mir morgen an ob es das tut was es soll und sonst reverten wir den PR wieder?

@andig andig marked this pull request as ready for review February 27, 2025 19:37
@andig andig marked this pull request as draft February 28, 2025 09:30
@github-actions github-actions bot added the stale Outdated and ready to close label Mar 7, 2025
@premultiply premultiply added needs decision Unsure if we should really do this waiting for feedback Suspended progress and removed stale Outdated and ready to close labels Mar 10, 2025
@premultiply
Copy link
Member

Should be tested by someone before merging.

@premultiply premultiply marked this pull request as ready for review March 10, 2025 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support needs decision Unsure if we should really do this waiting for feedback Suspended progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants