Conversation
Adds a consolidated binary sensor that aggregates zone 1, 2, and 3 occupancy into a single sensor for simpler automations.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdded a new template binary_sensor called "Radar Zone Occupancy" to the ESPHome configuration that aggregates occupancy data from three radar zone sensors using logical OR. The sensor includes device classification, icon assignment, and lambda logic combining existing sensors, with minor formatting adjustments to pin configuration. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
confirmed working with discord user hobbyboy merging to beta to test further |
Version: 26.3.2.1
What does this implement/fix?
Adds a consolidated "Radar Zone Occupancy" binary sensor that aggregates the three individual zone occupancy sensors (Zone 1, Zone 2, Zone 3) into a single unified sensor. This makes it easier to write automations that simply check "is there a person in this room?" without referencing individual zone sensors.
Mirrors the same addition made in MSR-2 (ApolloAutomation/MSR-2#47).
Types of changes
Checklist / Checklijst:
If user-visible functionality or configuration variables are added/modified:
Summary by CodeRabbit