-
Notifications
You must be signed in to change notification settings - Fork 59
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
DTMF sketch, codes transmitted seperately, sounds jumbled. #41
Comments
I adapted the DTMF example to send all DTMF tones in the buffer before turning off TX. Take a look and let me know if it works for you. |
Can you add 20ms before the cutoff? last number gets partially cut off. Other than that, works great! |
also invalid codes break the transmission and don't error in serial |
Ok, I've updated for both of those issues. LMK what you think. |
Great. Sorry to be so picky, but 20ms at the beginning would be good too. You have to time it in the loop so that the first one doesn't get cut off. Also, there's a weird noise at the end of each code. Sound file attached; slightly hard to hear but its apparent on the radio. I'll try inspecting with it to see if it affects HT decoding. |
HT won't decode because the codes aren't sent fast enough 😞 |
What timing does your device need?
Timing on the hamshield is configurable. I'll work on an example of that.
…On Tue, Jul 3, 2018, 5:47 PM Chris ***@***.***> wrote:
HT won't decode because the codes aren't sent fast enough 😞
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#41 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AApIHi70YSGHxJAO6L3h8kx-3LW8rjxkks5uDBChgaJpZM4U-hNk>
.
|
I've updated the library with some functions that support manual timing for DTMF. Please take a look at the DTMF example in master and let me know how it works for you. |
I've been trying to make it detect multiple codes in a single serial command and transmit them in one go, but I've been struggling. If I figure it out, I'll PR.
The text was updated successfully, but these errors were encountered: