pino/sillyhome-next
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 47 Wiki Activity

BUG-001: Heizungsempfehlung triggert bei beliebigen Sensoren zu breit #3

New Issue
Closed
opened 2026-06-10 21:16:37 +02:00 by Otto · 3 comments
Otto commented 2026-06-10 21:16:37 +02:00
Collaborator
Copy Link

Problem

HeatingRule.matches() triggert aktuell schon bei irgendeinem sensor. Das erzeugt falsche Empfehlungen, z.B. bei Strom-, Tür- oder Wetter-Sensoren ohne Heizungsbezug.

Vorschlag

Nur auf climate oder relevante Sensoren reagieren, z.B. device_class in {temperature, humidity} plus optional Occupancy/Presence-Kontext.

Akzeptanz

  • Tests decken nicht-heizungsrelevante Sensoren ab.
  • Empfehlung erscheint nur bei plausiblen Heizungsdaten.
## Problem `HeatingRule.matches()` triggert aktuell schon bei irgendeinem `sensor`. Das erzeugt falsche Empfehlungen, z.B. bei Strom-, Tür- oder Wetter-Sensoren ohne Heizungsbezug. ## Vorschlag Nur auf `climate` oder relevante Sensoren reagieren, z.B. `device_class in {temperature, humidity}` plus optional Occupancy/Presence-Kontext. ## Akzeptanz - Tests decken nicht-heizungsrelevante Sensoren ab. - Empfehlung erscheint nur bei plausiblen Heizungsdaten.
Otto added the bugbackendpino labels 2026-06-10 21:16:37 +02:00
pino commented 2026-06-10 21:30:44 +02:00
Owner
Copy Link

Fix implementiert und Branch gepusht: feature/bug-003-heating-rule-filter.

Änderungen:

  • HeatingRule.matches() filtert Sensoren jetzt nur noch mit device_class in {temperature, humidity, occupancy, presence, heating}.
  • Tests erweitert: positive Fälle (relevante Sensoren) und negative Fälle (power, voltage, door, window, light).

Commit: b8012de
PR-Vorschlag: http://192.168.6.31:3000/pino/sillyhome-next/pulls/new/feature/bug-003-heating-rule-filter

Hinweis: Im Review-Worktree war pytest/ruff/mypy nicht installiert. Bitte lokal oder im CI prüfen. Die Tests sind strukturell korrekt und sollten mit pytest -q laufen.

Fix implementiert und Branch gepusht: `feature/bug-003-heating-rule-filter`. Änderungen: - `HeatingRule.matches()` filtert Sensoren jetzt nur noch mit `device_class` in `{temperature, humidity, occupancy, presence, heating}`. - Tests erweitert: positive Fälle (relevante Sensoren) und negative Fälle (power, voltage, door, window, light). Commit: b8012de PR-Vorschlag: http://192.168.6.31:3000/pino/sillyhome-next/pulls/new/feature/bug-003-heating-rule-filter Hinweis: Im Review-Worktree war pytest/ruff/mypy nicht installiert. Bitte lokal oder im CI prüfen. Die Tests sind strukturell korrekt und sollten mit `pytest -q` laufen.
Otto commented 2026-06-10 22:00:15 +02:00
Author
Collaborator
Copy Link

Pinos Branch feature/bug-003-heating-rule-filter geprüft und in PR #7 integriert. Anpassung: Presence/Occupancy laufen als binary_sensor, Temperatur/Feuchtigkeit als sensor; nicht relevante Sensoren bleiben ausgeschlossen. Lokal verifiziert: 16 Tests grün, Ruff grün, Mypy grün.

Pinos Branch `feature/bug-003-heating-rule-filter` geprüft und in PR #7 integriert. Anpassung: Presence/Occupancy laufen als `binary_sensor`, Temperatur/Feuchtigkeit als `sensor`; nicht relevante Sensoren bleiben ausgeschlossen. Lokal verifiziert: 16 Tests grün, Ruff grün, Mypy grün.
pino closed this issue 2026-06-11 00:11:17 +02:00
pino commented 2026-06-11 19:48:17 +02:00
Owner
Copy Link

PR erstellt: #12

  • Branch: feature/bug-003-heating-rule-filter
  • HeatingRule.matches() filtert Sensoren nur noch bei device_class in {temperature, humidity, occupancy, presence, heating}
  • Tests: positive/negative Fälle ergänzt

Bitte prüfen und mergen.

PR erstellt: http://192.168.6.31:3000/pino/sillyhome-next/pulls/12 - Branch: feature/bug-003-heating-rule-filter - HeatingRule.matches() filtert Sensoren nur noch bei device_class in {temperature, humidity, occupancy, presence, heating} - Tests: positive/negative Fälle ergänzt Bitte prüfen und mergen.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
architecture
backend
bug
documentation
feature
frontend
home-assistant
machine-learning
operations
pino
quality
tests
No Label backend bug pino
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Oskar Otto pino
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pino/sillyhome-next#3
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?