Skip to content

Conversation

@andig
Copy link
Member

@andig andig commented Oct 22, 2025

Fix #24560

@andig andig added enhancement New feature or request experimental Experimental feature labels Oct 22, 2025
@andig andig requested a review from naltatis October 22, 2025 08:12
})
}

site.publish("evopt-batteries", batteries)
Copy link
Member

@naltatis naltatis Oct 22, 2025

Choose a reason for hiding this comment

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

I'd prefer this to be folded into the existing structure:

  "battery": [
    {
      "title": "Sungrow",
      "power": -1977,
      "energy": 5755.5,
      "capacity": 22.4,
      "soc": 51.7,
+     "estimatedEmpty": ___
+     "estimatedFull": ___
      "controllable": true
    }
  ],

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, macht sinn (auch wenn ich nicht weiss, wie). Wo packen wir das für die Fahrzeuge hin?

Copy link
Member

@naltatis naltatis Oct 22, 2025

Choose a reason for hiding this comment

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

Wo packen wir das für die Fahrzeuge hin?

Die publishen wir ja auch. Würd ich auch da direkt im Objekt sehen.

Copy link
Member Author

Choose a reason for hiding this comment

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

Da sind ja aktuell nur die Stammdaten drin. Das hier wären Bewegungsdaten. Wir müssten die dann also einzeln am Loadpoint publizieren. Größeres Topic wäre, mal zu überlegen, ob wir unsere ganze Publishingstruktur mal umkrempeln wollen. Das wird allerdings nicht ohne Fallout gehen und würde ich aktuell gerne vermeiden.

Copy link
Member

Choose a reason for hiding this comment

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

Da sind ja aktuell nur die Stammdaten drin. Das hier wären Bewegungsdaten.

Eine Trennung zwischen Stammdaten und Bewegungsdaten haben wir ja auch nicht wirklich an anderen Stellen battery.0.title/battery.0.power.

Wir müssten die dann also einzeln am Loadpoint publizieren.

Können wir auch machen. Wäre dann konsistent mit den bestehenden Fahrzeugwerten wie range.

Größeres Topic wäre, mal zu überlegen, ob wir unsere ganze Publishingstruktur mal umkrempeln wollen. Das wird allerdings nicht ohne Fallout gehen und würde ich aktuell gerne vermeiden.

🙀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request experimental Experimental feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Optimizer: next steps

3 participants