MorseKOB / PyKOB

Python implementation of a library for Morse and MorseKOB functionality
https://sites.google.com/site/morsekob/morsekob40
MIT License
4 stars 2 forks source link

In 'Separate key and sounder' mode, the sounder does not follow the key in real time. #288

Closed AESilky closed 8 months ago

AESilky commented 8 months ago

Instead of following the key openings and closings the instant they happen, MKOB waits for a complete character to be sent before replicating it on the sounder. In fact, if I send a string of ten or fifteen dots, the sounder doesn't do anything until I stop sending.

pwdirks commented 8 months ago

 Hi Ed,

I wonder if that’s actually unhelpful for local senders. Maybe an item on the “Issues List” could be created to make the local sounder (or its simulation) more directly track the key input?

Cheers, -Patrick.

On Dec 23, 2023, at 10:17 AM, Ed Silky @.***> wrote:

Instead of following the key openings and closings the instant they happen, MKOB waits for a complete character to be sent before replicating it on the sounder. In fact, if I send a string of ten or fifteen dots, the sounder doesn't do anything until I stop sending.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.

jchausler commented 8 months ago

Hi Ed,

I agree, the sounder, whether real or simulated, needs to follow the key operations click by click, now the screen decoder likely needs to wait for a pause to be able to determine the last character sent and that includes a long string of just dots. I just checked and 2.5 appears to work this way.

73, Chris -----Original Message----- From: MorseKOB/PyKOB @.> Sent: Dec 24, 2023 7:54 PM To: MorseKOB/PyKOB @.> Cc: Subscribed @.***> Subject: Re: [MorseKOB/PyKOB] In 'Separate key and sounder' mode, the sounder does not follow the key in real time. (Issue #288)

Hi Ed,

I wonder if that’s actually unhelpful for local senders. Maybe an item on the “Issues List” could be created to make the local sounder (or its simulation) more directly track the key input?

Cheers, -Patrick.

On Dec 23, 2023, at 10:17 AM, Ed Silky @.***> wrote:

Instead of following the key openings and closings the instant they happen, MKOB waits for a complete character to be sent before replicating it on the sounder. In fact, if I send a string of ten or fifteen dots, the sounder doesn't do anything until I stop sending.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.

— Reply to this email directly, view it on GitHub (https://github.com/MorseKOB/PyKOB/issues/288#issuecomment-1868632359), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ABEY5E3E4LJ5VGUMTS2QHOLYLDFFZAVCNFSM6AAAAABBA6MP3SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYGYZTEMZVHE). You are receiving this because you are subscribed to this thread.Message ID: @.***>

AESilky commented 8 months ago

Patrick and Chris,

I'm not sure about your comments.

This issue is for the problem that the sounder doesn't directly follow the key. In fact, it waits until a full character has been decoded before sounding anything.

My fix is to move the sounder activation directly into the key handling code. With that change, I can't detect any delay by listening to it. I am currently doing some more testing and plan to hook up my scope so I can measure it (like I did for issue #158).

This issue (#288) is for a physical key, and #158 (Speaker lags key) is for the virtual sounder (it's been open for quite some time).

AESilky commented 8 months ago

I was busy today and will be tomorrow and Tuesday as well. I should be able to wrap up my testing and merge in my fix Wednesday or Thursday.

jchausler commented 8 months ago

Hi Ed,

I think what you are saying is what I was suggesting. That the sounder actuations, simulated or real, should directly follow the key rather than wait for the decode algorithm to figure out what was sent. In fact as I said I don't think the decode algorithm could even start its task until a complete character is received and so that should be an entirely separate operation. If you actually just send a connected string of dots, that to the decode routine is just one character. The original Mill program sort of did the same thing in that it would sound the character and only then show it on the screen. When I was learning, I found this very valuable as I couldn't verify what the character was until I had first heard it.

Have a happy holiday season!!!

73, Chris -----Original Message----- From: MorseKOB/PyKOB @.> Sent: Dec 25, 2023 1:18 AM To: MorseKOB/PyKOB @.> Cc: J. Chris Hausler @.>, Comment @.> Subject: Re: [MorseKOB/PyKOB] In 'Separate key and sounder' mode, the sounder does not follow the key in real time. (Issue #288)

Patrick and Chris, I'm not sure about your comments. This issue is for the problem that the sounder doesn't directly follow the key. In fact, it waits until a full character has been decoded before sounding anything. My fix is to move the sounder activation directly into the key handling code. With that change, I can't detect any delay by listening to it. I am currently doing some more testing and plan to hook up my scope so I can measure it (like I did for issue #158 (https://github.com/MorseKOB/PyKOB/issues/158)). This issue (#288 (https://github.com/MorseKOB/PyKOB/issues/288)) is for a physical key, and #158 (https://github.com/MorseKOB/PyKOB/issues/158) (Speaker lags key) is for the virtual sounder (it's been open for quite some time). — Reply to this email directly, view it on GitHub (https://github.com/MorseKOB/PyKOB/issues/288#issuecomment-1868772589), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ABEY5EZONVBVX6NPQL7U4TLYLELE3AVCNFSM6AAAAABBA6MP3SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYG43TENJYHE). You are receiving this because you commented.Message ID: @.***>

AESilky commented 8 months ago

I tested this with a key and the virtual sounder, and Les tested with a key and physical sounder. Everything is working well now.