Skip to content

Conversation

@swiffer
Copy link
Collaborator

@swiffer swiffer commented Jun 8, 2025

aligns usage of filters used in consumption gross queries with other panels to avoid some edge cases.

as all other queries are using date filters on start_date of a drive let's ensure we are doing the same for queries relying on positions / advanced logic.

before this change - selecting a period with a start date at the time of driving (actual drive start out of selected period) consumption gross was off.

grafik

after this change - numbers to align correctly

grafik

@netlify
Copy link

netlify bot commented Jun 8, 2025

Deploy Preview for teslamate ready!

Name Link
🔨 Latest commit 3dc21f4
🔍 Latest deploy log https://app.netlify.com/projects/teslamate/deploys/6846aab5bfc6f10008645247
😎 Deploy Preview https://deploy-preview-4787--teslamate.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@swiffer swiffer added the area:dashboard Related to a Grafana dashboard label Jun 8, 2025
@swiffer swiffer self-assigned this Jun 8, 2025
@swiffer swiffer requested a review from JakobLichterfeld June 8, 2025 12:31
@JakobLichterfeld JakobLichterfeld changed the title fix: align effects of timerange filter fix: align usage of time-range filters to fix consumption gross issues in some edge cases Jun 9, 2025
Copy link
Member

@JakobLichterfeld JakobLichterfeld left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks so much!

@JakobLichterfeld JakobLichterfeld merged commit bbbb20f into main Jun 9, 2025
18 of 19 checks passed
@JakobLichterfeld JakobLichterfeld deleted the swiffer/align-effects-of-timerange-filter branch June 9, 2025 09:35
@JakobLichterfeld JakobLichterfeld added this to the v2.1.0 milestone Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dashboard Related to a Grafana dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants