- Notifications
You must be signed in to change notification settings - Fork810
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
add(solar tariff): api.akkudoktor.net#19434
base:master
Are you sure you want to change the base?
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. Dascommon-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. Lautmeta 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. |
Depends on#19474✔Add a solar tariff template forhttps://api.akkudoktor.net/#/pv%20generation%20calculation/getForecast