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

Fixes and raw data interface for BGT24LTR11-Pulsed-Doppler-Radar lib #171

Merged
merged 5 commits into from
Jan 26, 2021
Merged

Fixes and raw data interface for BGT24LTR11-Pulsed-Doppler-Radar lib #171

merged 5 commits into from
Jan 26, 2021

Conversation

mkiessling-ifx
Copy link
Contributor

  • rename board to "Radar BaseBoard XMC4700"
  • fixes build and linker issues with usage of this library
  • added raw data interface
  • updated examples
  • added example to plot I/Q signal with build in Serial plotter

* do some basic fixes to compile and find the lib.
Linker issues are resolved with next commit
* Remove "DEMO" from board name
* Update latest RadarLib version (Jan2021)
** fixes and corrections
** added raw data interface
** remove linker issues
* added plotter example
* small fix, after testing in IDE
* merged locally into 1.5 release and tested the examples on HW
@techpaul
Copy link
Contributor

techpaul commented Jan 25, 2021

A couple of suggestions

pins_arduino.h missing definition

#define LED_BUILTIN xx  // Standard Arduino LED 

When running other code or examples some use this definition, use the same pin as one of your other LEDs.

In your examples and other files you use digital.write to set a specific level for LED on/off definitions in pins_arduino.h for LED being HIGH or LOW for a specific board mean you can use the LED library to make your examples more portable to other boards using same chipset. Only having to say

Led.Red( ON );
or
Led.Red( OFF );

Is more portable across any Radar board, it takes care of does it need to be HIGH or LOW to turn on or off, depending on what board has been specified.

See LED library

enhanced examples with Led-Object to switch LEDs
@mkiessling-ifx
Copy link
Contributor Author

@techpaul: Thank you for the hint with Led-Object. That makes the examples really more expressive.

@mhollfelder mhollfelder changed the base branch from master to 24Ghz-radar January 26, 2021 07:58
@mhollfelder
Copy link
Contributor

mhollfelder commented Jan 26, 2021

Created new branch 24GHz-radar for 24 Ghz Radar related changes and fixes before merging into develop.
Changed base for the pull request to this new branch.

@mhollfelder mhollfelder merged commit f93dac3 into Infineon:24Ghz-radar Jan 26, 2021
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

Successfully merging this pull request may close these issues.

None yet

3 participants