Skip to content

Conversation

@bharvey88
Copy link
Contributor

@bharvey88 bharvey88 commented Jun 24, 2025

Currently the yaml has a bug where the timeout for presence is ignored because we have lines in the yaml missing.

We will likely need to add this to the r-pro-1 yaml as well since it also uses the LD2450.

More info from keith at ESPHome here: esphome/issues#7006 (comment)

Version:

What does this implement/fix?

Types of changes

  • Bugfix (fixed change that fixes an issue)
  • New feature (thanks!)
  • Breaking change (repair/feature that breaks existing functionality)
  • Dependency Update - Does not publish
  • Other - Does not publish
  • Website of github readme file update - Does not publish
  • Github workflows - Does not publish

Checklist / Checklijst:

  • The code change has been tested and works locally
  • The code change has not yet been tested

If user-visible functionality or configuration variables are added/modified:

  • Added/updated documentation for the web page

TrevorSchirmer and others added 2 commits April 11, 2025 10:22
New ld2450 Library, New Updates
Currently the yaml has a bug where the timeout for presence is ignored because we have lines in the yaml missing.

We will likely need to add this to the r-pro-1 yaml as well since it also uses the LD2450.

More info from keith at ESPHome here: esphome/issues#7006 (comment)
@github-actions github-actions bot requested a review from TrevorSchirmer June 24, 2025 17:15
@bharvey88 bharvey88 changed the base branch from main to beta June 24, 2025 17:17
@bharvey88 bharvey88 added the bugfix Something isn't working label Jun 24, 2025
@TrevorSchirmer
Copy link
Member

@bharvey88 Can you update the version number? Otherwise looks good.

Also need to fill out one of the checkboxes in the future. It is what auto applies the correct label.

image

@TrevorSchirmer TrevorSchirmer merged commit 552103a into beta Jun 26, 2025
10 checks passed
@TrevorSchirmer TrevorSchirmer deleted the ld2450-set-presence-timeout-fix branch June 26, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants