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.

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

Release 0.1.4 breaks radar.begin() return value #32

Open
jenschr opened this issue Oct 16, 2024 · 3 comments
Open

Release 0.1.4 breaks radar.begin() return value #32

jenschr opened this issue Oct 16, 2024 · 3 comments

Comments

@jenschr
Copy link

jenschr commented Oct 16, 2024

I just installed the 0.1.4 version in Arduino and radar.begin(RADAR_SERIAL) is always returning false. Setting the "waitForRadar" parameter to false will bypass the issue since it does not check the UART connection. If otherwise configured correctly, everything will work, but if code relies on the radar.begin functionality, it will fail.

Steps to reproduce:

  • Open the basicSensor.ino example
  • Upload to your device
  • Watch the serial output to see: "LD2410 radar sensor initialising: not connected" on startup

Steps to fix:

  • Use Library Manager to reinstall version 0.1.3 and all will work

Tested with ESP32C3 DevkitM 1.

@Humancell
Copy link

@maxabba have you seen this? Do you have a fix where you can provide a pull request to fix this?

@maxabba
Copy link
Collaborator

maxabba commented Dec 12, 2024

The issue originates from the logic of the circular buffer with the ESP32 task. I will try to add a support fix for both the 'normal' routine and the task routine in the coming days.

@maxabba
Copy link
Collaborator

maxabba commented Dec 12, 2024

I made a pull request to fix this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants