kit-cel / gr-radar

GNU Radio Radar Toolbox
GNU General Public License v3.0
238 stars 84 forks source link

Mait 3.10 #53

Closed maitbot closed 1 year ago

maitbot commented 2 years ago

Updated to build with gnuradio 3.10 and pas all tests.

SpontaneousDuck commented 2 years ago

Thanks for the work @maitbot! I was just about to start doing this myself. In the process of building and installing, I did find that. in the GRC files for each block, I needed to update:

templates:
  imports: import radar

to:

templates:
  imports: from gnuradio import radar

for each GRC block definition.

I also found that the following was needed for my build:

Other than that, almost tests are passing for me with python=3.10.2 and gnuradio=3.10.1.1. The only failing test is for ts_fft_cc which seems to be a slight numerical difference between fftw3 and numpy.fft in calculating the FFT (FFTW3 vs PocketFFT). Relaxing https://github.com/maitbot/gr-radar/blob/4e80df4b281d8305e79113f360347edf3f9bb13a/python/radar/qa_ts_fft_cc.py#L53 to 3 decimal places solved this for me. I think we can ignore that problem.

Other than that, I will go through the examples and make sure everything is working for me! 🐱‍👤

marcusmueller commented 2 years ago

Can't build GR 3.10 on my laptop right now. @MLsmd, would it be possible that you try building this branch?

alphafox02 commented 1 year ago

Build completed on 3.10.4 w/ 22.04 python version. However, the one flow graph I opened under examples (examples/simulation/simulator_fmcw_rcs.grc) tossed up just a couple complaints before opening and then I got a module import of 'radar' not found.

I also had to git checkout the mait/gr-3.10 branch as the default mait-3.10 branch linked above would not built, stopping at GrSwig not found.

mbr0wn commented 1 year ago

Merged via #57.