-
-
Notifications
You must be signed in to change notification settings - Fork 829
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
add(solar tariff): api.akkudoktor.net #19434
Conversation
Das unquote sollte es per #19443 nicht mehr brauchen. Schau Dir bitte auch die anderen Typendefinitionen an. Es wäre schön die Parameter konsistent zu halten, z.B. tilt -> dec. Bevor wir weitere Forecast hinzu führen sollen möglichst alle Beschreibungen in defaults.yml überführt werden ein ggf. auch ein base-forecast Tempalte gebaut mit lat/lon/az/dec/kwp. Die scheinen recht universell. |
@andig Die Bezeichner wurden überarbeitet, um Typdefinitionen erweitert und an das Schema der vorhandenen forecast templates angepasst. Das common-forecast schaue ich mir noch an |
Das |
@naltatis das |
help: | ||
en: "returns max- and min- generation from different weathermodels. default: 0" | ||
de: "liefert Max- und Min- Erzeugung aus verschiedenen Wettermodellen. standard: 0" | ||
type: int |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was tut der Parameter?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das geht aus der API Doku leider nicht klar hervor: https://api.akkudoktor.net/#/pv%20generation%20calculation/getForecast
Die Beschreibung habe ich von dort übernommen. Der Parameter scheint auch nichts an der Rückgabe zu ändern. Laut meta
key im Response body ist das wohl ein boolean Parameter, der sich aber durch die Eingabe nicht verändert. Also egal ob 0,1 oder beliebig andere int Werte übergeben werden, bleibt "range": true
{
"meta": {
"lat": 55.45,
"lon": 8.14,
...
"range": true,
Aus allen Parameterbeschreibungen wurde der default entfernt und dafür bei fast allen Parametern ein Keine defaults wurden hinterlegt für:
da für beide mMn kein sinnvoller default möglich ist. |
@andig der PR wurde automatisch geschlossen. Könnt ihr ihn wieder öffnen? |
Depends on #19182 |
In preparation for the upcoming PR #19182, |
Depends on #19474✔Add a solar tariff template for https://api.akkudoktor.net/#/pv%20generation%20calculation/getForecast