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

188 add Farnsworth timing to k1usn SST Contest #196

Merged

Conversation

w7sst
Copy link
Owner

@w7sst w7sst commented Mar 7, 2023

Introduces Farnsworth timing for the K1USN Slow Speed Test (SST) only. #188.

The Farnsworth character speed is specified using the following MorseRunner.ini setting:

[Settings]
FarnsworthCharacterRate=25

To increase the spacing between characters and words, slow the sending speed down using the usual UI control. For example, a UI speed of 18 with the above .INI file settings, the resulting Farnsworth speed would be 18/25 (18 wpm using characters sent at 25 wpm).

Question: Can you think of an alternate name for MinFarnsworthWpmC? I struggled to find an appropriate name for the Farnsworth character speed. Perhaps FarnsworthCharacterRate=25?
Update: Scott suggested to go with FarnsworthCharacterRate. Thank you Scott!

@w7sst w7sst linked an issue Mar 7, 2023 that may be closed by this pull request
11 tasks
@w7sst
Copy link
Owner Author

w7sst commented Mar 7, 2023

This represents my second version of the Farnsworth timing support. There are lots of comments in the code in VCL/FarnsKeyer.pas. There are also two papers mentioned describing the math used by this algorithm:

Please see additional comments in the original issue #188.

@scotthibbs
Copy link
Collaborator

I like FarnsworthCharacterRate=25 better than MinFarnsworthWpmC. I think it's clearer than adding the "minimum" wording as people might confuse this with the minimum that the cw can be sent all together.

Once someone does code review, maybe another prerelease to test, while I work on getting the release wording and help files ready for the new version.

(BTW, I find it cool that we caught an original bug with the lack of spacing in the cw. What a cool catch that for years no one noticed.)

@w7sst
Copy link
Owner Author

w7sst commented Mar 7, 2023

I like FarnsworthCharacterRate=25 better than MinFarnsworthWpmC. I think it's clearer than adding the "minimum" wording as people might confuse this with the minimum that the cw can be sent all together.

I'll change it over to FarnsworthCharacterRate. Thank you for the input. Cheers.

@w7sst w7sst merged commit d4a0c4a into main Mar 7, 2023
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.

Add Farnsworth Timing to K1USN Slow Speed Test (SST)
3 participants