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

Remove output audio thread #46

Merged
merged 9 commits into from
Dec 16, 2023
Merged

Remove output audio thread #46

merged 9 commits into from
Dec 16, 2023

Conversation

jj1bdx
Copy link
Owner

@jj1bdx jj1bdx commented Dec 15, 2023

Output audio thread is no longer required because:

  • PortAudio takes care of synchronization for real-time output;
  • libsndfile also takes care of file output;
  • And no other output drivers exist in airspy-fmradion.

Also, other minor fixes are included.

I will make this a pull request since I need to test this for at least a few days before making this a release.

The version number is 20231215-direct-out, an irregular one.

@jj1bdx jj1bdx self-assigned this Dec 15, 2023
@jj1bdx
Copy link
Owner Author

jj1bdx commented Dec 16, 2023

So far this code does not do harm on playing radio in real-time nor on recording the audio to a WAV file.

jj1bdx and others added 2 commits December 16, 2023 13:53
* To avoid output clipping in AM/CW/USB/LSB/WSPR reception
* To lower generating clicking sound on CW and more smooth SSB reception
AM Decoder AGC fix

* To avoid output clipping in AM/CW/USB/LSB/WSPR reception
* To lower generating clicking sound on CW and smoother SSB reception
@jj1bdx jj1bdx merged commit 599dc43 into main Dec 16, 2023
@jj1bdx
Copy link
Owner Author

jj1bdx commented Dec 16, 2023

I've tested this pull request on two real-world recordings, and worked fine.
LGTM, so this patch will be in the next release, with the documentation fix.

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.

1 participant