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

Added auto-tune functionality to recorder #271

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kcwebby
Copy link

@kcwebby kcwebby commented Oct 1, 2019

Luke -- I don't know how this works, but here's the pull request from the auto-ppm fork. It's 44 commits behind and only one ahead.

@robotastic
Copy link
Owner

Cool - I am working on getting the code cleaned up and I will try to work this in

@bctrainers
Copy link

It may be worth tagging in a note that PPM doesn't work too cleanly on RTL-SDR's - and the usage of 'error' would be better.

If possible, detection via device string if sdr is in-use to have the code throw a warning back to the terminal.

@kcwebby
Copy link
Author

kcwebby commented Dec 4, 2019 via email

@KyleKotowick
Copy link

Hi @robotastic, just wanted to check in on the status of this. Any luck implementing some sort of auto-correct feature?

@robotastic
Copy link
Owner

OK - I really need to get around to adding some form of Auto Tuning!

@robotastic robotastic self-assigned this Dec 15, 2024
@taclane
Copy link
Contributor

taclane commented Dec 15, 2024

I've got a branch I've been using for almost a year now to feed the FLL Band Edge outputs into a rolling average per-device. It works great for p25 voice and control channels, but I haven't tried other systems. Now that everything is going through the channelizer in 5.0, the method should be applicable to most systems.

Leveraging the FLL block (which already is doing some auto-tune itself) to maintain a fine-tune correction value is likely more effective than adjusting the PPM based on message count. The feedback from the FLL is pretty immediate, and only takes a handful of calls to bring the tuner offset under 100 Hz.

I'll try and clean it up, and maybe we can finally close out this and #217.

@robotastic
Copy link
Owner

That would be amazing!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants