search

miek / inspectrum

Radio signal analyser
GNU General Public License v3.0
2.04k stars 263 forks source link

Reading SDRangel SigMF files #209

Closed PeteK6TJ closed 1 year ago

PeteK6TJ commented 2 years ago

SDRangel seems to write ci32 SigMF files, which it seems neither inspectrum nor gnuradio can read as SigMF. But in the inputsource.cpp source file of inspectrum, I added a ci32_le type entry and created a S32 sample converter (and a few others) by straightforward plagiarizing of the sample converters already in there. Now I can snoop around for short packets with SDRangel (which has a fast display), save a giant file, and use inspectrum to find what I want to analyze in gnuradio. Open and closed feature request??

catkira commented 1 year ago

@PeteK6TJ I would be interested. Can You make it public on Your github account?

PeteK6TJ commented 1 year ago

Does this email work??

Sent from my iPhone

Pete Mahowald | @.*** | 650 492-3989

On Oct 17, 2022, at 12:09 PM, Benjamin Menküc @.***> wrote:

 I would be interested

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

catkira commented 1 year ago

You can reach me at benjamin@menkuec.de

catkira commented 1 year ago

I quickly did a PR that adds support for ci32, since there is no answer here. https://github.com/miek/inspectrum/pull/214

PeteK6TJ commented 1 year ago

Here's my code.

On Tue, Oct 18, 2022 at 6:30 AM Benjamin Menküc @.***> wrote:

I quickly did a PR that adds support for ci32, since there is no answer here.

213 https://github.com/miek/inspectrum/pull/213

— Reply to this email directly, view it on GitHub https://github.com/miek/inspectrum/issues/209#issuecomment-1282390767, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXAJXZFCAKJZB5HVUTRC4MLWD2Q77ANCNFSM532KMERA . You are receiving this because you were mentioned.Message ID: @.***>

catkira commented 1 year ago

I dont see any code, but I already re-created the PR see here: https://github.com/miek/inspectrum/pull/214

schneider42 commented 1 year ago

This has been merged. Once you build/install inspectrum from source you will be able to make use of the feature. I don't know about timelines for a release so distributions can pick up the change.

catkira commented 1 year ago

Yes I know, I made the post before it was merged ^^