search

AllStarLink / app_rpt

Refactoring and upgrade of AllStarLink's app_rpt, etc.
9 stars 9 forks source link

Allow faster morse speed. #368

Closed JimZAH closed 4 months ago

JimZAH commented 4 months ago

Fixes #367 Increase the max limit to 35wpm.

JimZAH commented 4 months ago

Hi Tim,

I agree.

35 seems a sane number. I've never heard a repeater or link with anything faster than 30wpm.

The CW idents in the UK are usually around 20 to 25 wpm. We don't have an imposed speed limit.

On 21 July 2024 23:36:47 BST, Tim WD6AWP @.***> wrote:

@tsawyer commented on this pull request.

AllStar is built in the US where part 97 FCC rules limit CW ID to 20wpm. Somebody way back when thought "enforcing" 20wpm was a good idea. It wasn't. ASL has no responsibility to insure radio systems are operated in accordance with local regulations and should not imply so by setting a 20wpm CW speed limit.

I like to know why is 35 the requested limit?

-- Reply to this email directly or view it on GitHub: https://github.com/AllStarLink/app_rpt/pull/368#pullrequestreview-2190410125 You are receiving this because you authored the thread.

Message ID: @.***>

Allan-N commented 4 months ago

Let's merge this.

BEFORE merging we NEED to ensure that this change won't impact existing configs.

KB4MDD commented 4 months ago

If someone has a speed faster than 20wpm defined, after the merge it will go up to the amount set or capped at 35.

JimZAH commented 4 months ago

It should stay the same (whatever it's currently set at). If they've set it at 30 it will go up to 30. The limit is 35.

Allan-N commented 4 months ago

My apologies. I was thinking that this was [another] rpt.conf change. Yes, this change was fine to merge.

Allan-N commented 3 months ago

The fix for this issue has been incorporated into ASL3-Asterisk 20.9.1+asl3-3.0.4-1