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.

Sign up for GitHub

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

Jump to bottom

Tariff: better rate matching error #17255

Merged
merged 4 commits into from
Nov 14, 2024
Merged

Tariff: better rate matching error #17255

merged 4 commits into from
Nov 14, 2024

Conversation

naltatis
Copy link
Member

Improve logging for cases like #17253

[site  ] WARN 2024/11/14 08:38:18 planner: no matching rate: now 2024-11-13T19:38:18+01:00, 24 rates (2024-11-13T23:00:00Z to 2024-11-14T23:00:00Z)

@naltatis naltatis added the enhancement New feature or request label Nov 14, 2024
@naltatis naltatis requested a review from andig November 14, 2024 07:40
@naltatis naltatis mentioned this pull request Nov 14, 2024
Closed
1 task
andig
andig reviewed Nov 14, 2024
View reviewed changes
api/rates.go Outdated Show resolved Hide resolved
api/rates.go Outdated Show resolved Hide resolved
andig
andig reviewed Nov 14, 2024
View reviewed changes
Copy link
Member

@andig andig left a comment

Choose a reason for hiding this comment

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

Wir sollten im Log immer lokale Zeitzone angeben

api/rates.go Outdated Show resolved Hide resolved
api/rates.go Outdated Show resolved Hide resolved
naltatis and others added 2 commits November 14, 2024 11:32
@naltatis @andig
Update api/rates.go
ea34536 
Co-authored-by: andig <andi@evcc.io>
@naltatis @andig
Update api/rates.go
97eaa60 
Co-authored-by: andig <andi@evcc.io>
@andig andig merged commit 1a8b412 into master Nov 14, 2024
6 checks passed
@andig andig deleted the fix/rate_debugging branch November 14, 2024 12:33
jonilala796 pushed a commit to jonilala796/evcc that referenced this pull request Jan 3, 2025
@naltatis @jonilala796
Tariff: better rate matching error (evcc-io#17255)
24cc159
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andig andig andig left review comments

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@naltatis @andig