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

add(solar tariff): api.akkudoktor.net #19434

Merged
merged 7 commits into from
Apr 8, 2025
Merged

add(solar tariff): api.akkudoktor.net #19434

merged 7 commits into from
Apr 8, 2025

Conversation

Glopix
Copy link
Contributor

@Glopix Glopix commented Mar 5, 2025

@andig
Copy link
Member

andig commented Mar 5, 2025

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 andig marked this pull request as draft March 5, 2025 10:59
@andig andig added the tariffs Specific tariff support label Mar 5, 2025
@Glopix
Copy link
Contributor Author

Glopix commented Mar 6, 2025

@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

@Glopix
Copy link
Contributor Author

Glopix commented Mar 6, 2025

Das forecast-base preset wird nun für die gemeinsamen Parameter verwendet.

@Glopix Glopix marked this pull request as ready for review March 6, 2025 19:50
@naltatis
Copy link
Member

naltatis commented Mar 7, 2025

@Glopix ist mit #19474 schon im Master.

@Glopix
Copy link
Contributor Author

Glopix commented Mar 7, 2025

@naltatis das forecast-base preset? Stimmt, das ist mir gestern auch aufgefallen. Deshalb habe ich es gestern in diesen PR eingebunden.

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
Copy link
Member

Choose a reason for hiding this comment

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

was tut der Parameter?

Copy link
Contributor Author

@Glopix Glopix Mar 11, 2025

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,

@Glopix
Copy link
Contributor Author

Glopix commented Mar 11, 2025

Aus allen Parameterbeschreibungen wurde der default entfernt und dafür bei fast allen Parametern ein default: hinterlegt.
Da hierdurch immer Werte vorhanden sind, wurden die dazugehörigen Inline-Bedingungsausdrücke entfernt ( {{ if .parameter}}&parameter={{ .parameter}}{{ end }} )

Keine defaults wurden hinterlegt für:

  • az / Azimuth (erforderlich),
  • horizon (optional),

da für beide mMn kein sinnvoller default möglich ist.

@github-actions github-actions bot added the stale Outdated and ready to close label Mar 18, 2025
@Glopix Glopix requested a review from andig March 20, 2025 10:05
@github-actions github-actions bot removed the stale Outdated and ready to close label Mar 20, 2025
@github-actions github-actions bot added the stale Outdated and ready to close label Mar 27, 2025
@github-actions github-actions bot closed this Apr 1, 2025
@Glopix
Copy link
Contributor Author

Glopix commented Apr 1, 2025

@andig der PR wurde automatisch geschlossen. Könnt ihr ihn wieder öffnen?

@andig andig reopened this Apr 1, 2025
@andig
Copy link
Member

andig commented Apr 1, 2025

Depends on #19182

@github-actions github-actions bot removed the stale Outdated and ready to close label Apr 1, 2025
@Glopix
Copy link
Contributor Author

Glopix commented Apr 4, 2025

In preparation for the upcoming PR #19182, price has been renamed value.

@andig andig merged commit d631e74 into evcc-io:master Apr 8, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tariffs Specific tariff support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants