Optimization of LD2410 radar sensor reading and frame processing

[maxabba]

Optimization of LD2410 radar sensor reading and frame processing

This pull request addresses a critical issue in the LD2410 library where the read() function was always returning false, even when new data was available. The changes significantly improve the reliability and efficiency of reading data from the radar sensor.

Key modifications:

1. Improved read() function:

   • Now returns true if new data has been read or if a complete frame has been processed.
   • This change allows for more accurate detection of new data availability.

2. Optimized read_frame_() function:

   • Now processes data from the circular buffer instead of reading directly from UART.
   • Improves handling of partial data and reduces the possibility of data loss.

3. Enhanced integration with the circular buffer:

   • More efficient use of buffer memory.
   • Reduced risk of buffer overflow.

Expected impact:

• Better detection of new data presence from the sensor.
• More reliable processing of complete frames.
• Overall improved performance in sensor data handling.

Tests performed:

• Verified that read() now returns true when new data is available.
• Checked that frame processing still works correctly with these changes.
• Examined potential side effects in other parts of the library.

Recommended next steps:

1. Thorough code review.
2. Testing in various scenarios to ensure the robustness of the changes.
3. Consider adding additional unit tests to cover new use cases.

These changes should resolve the main reported issue and significantly improve the usability and reliability of the LD2410 library. I'm open to feedback and suggestions for further improvements.

[Humancell]

@ncmreynolds will you be merging these updates and creating a new release?

I was just about to update a bunch of sensors I built with this library and saw this pull that appears to improve the original code.

[maxabba]

@Humancell i'm using in my personal project and it is working very well. you can use mine until @ncmreynolds marge to main

https://github.com/maxabba/ld2410.git

for implement via platformio you can add to ini file like this installing git desktop

lib_deps = 
    https://github.com/maxabba/ld2410.git

[ncmreynolds]

Again, seems entirely reasonable to me.

[Humancell]

Thank you! I'll be testing soon!