-
Notifications
You must be signed in to change notification settings - Fork 274
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
WIP: Seeking input - I'm looking to get a consistent CW ID, every {interval} minutes, only when in use #549
base: master
Are you sure you want to change the base?
Conversation
This seems directly related to #671 I would think that @g4klx might be more inclined to look at this if there was a configuration switch that enabled this behavior. It's apparently a very well-worn subject about how the CW ID works (or doesn't) and apparently there isn't much appetite for changing the default behavior. Having this exist as an optional setting that enables this kind of behavior would perhaps be more likely to get reviewed. My two cents. |
Agreed. That will require a bit more digging into the code to learn how the configuration and implementation are tied together. There is a setting for ID or no ID, but I'd guess you're suggesting something a little more granular. |
I think because this is an international project, there are constituencies that have different needs/preferences/requirements. @g4klx said that in the old OpenDV forum on Yahoo, they argued about CWID and what's in the code right now was the compromise solution. The problem is that the compromise solution is way out of compliance with what repeater operators in the U.S. expect to have in place. So, I think the deltas should leave in the existing CWID regime, but based on a configuration value, swap that functionality for what you've created instead. Configuration values are trucked around as global variables, if memory serves. I would have to go back and look again, and I don't have time right now. Funny enough, hams in the US are starting to realize that busy MMDVM repeaters are not transmitting CWID enough, and I had to fill them in on the fact that CWID is working as the software intends it to work -- that is, not often enough for U.S. repeater operators. |
I'll leave this here as possible implementation notes:
|
This set of mods seems to work, most of the time. I have noticed two problems and I'd like some guidance.
I've removed many instances where the CW ID Timer was stopped because that resets the timer and, in the US at least, resetting the timer when restarting isn't acceptable. I toyed with the idea of changing the behaviour of the Timer class to allow stop/start without reset, but haven't convinced myself that's needed yet.