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

Send_morse command not following standard for Extended Response Protocol #1365

Closed
ny4i opened this issue Aug 18, 2023 · 2 comments
Closed

Send_morse command not following standard for Extended Response Protocol #1365

ny4i opened this issue Aug 18, 2023 · 2 comments
Labels
bug fixed Issue has been fixed
Milestone

Comments

@ny4i
Copy link

ny4i commented Aug 18, 2023

I send a ,b NY4I command and receive back the RPRT 0 at the front of the message:

C:\Windows\system32>echo ,f VFOA | ncat -w 1 127.0.0.1 4532
get_freq: VFOA,Frequency: 7029110,RPRT 0

C:\Windows\system32>echo ,b NY4I | ncat -w 1 127.0.0.1 4532
,RPRT 0rse: currVFO,NY4I

This is when using an Elecraft K4.

@mdblack98 mdblack98 added the bug label Aug 18, 2023
@mdblack98 mdblack98 added this to the 4.6 milestone Aug 18, 2023
mdblack98 added a commit that referenced this issue Aug 18, 2023
@mdblack98 mdblack98 added the needs test Patches have been submitted but need testing to close issue label Aug 18, 2023
mdblack98 added a commit that referenced this issue Aug 19, 2023
@ny4i
Copy link
Author

ny4i commented Aug 21, 2023

The format of the reply is correct.

Note it appears inconsistent that send_morse does not require a VFO but stop_morse does require a VFO in VFO mode.

C:\Windows\system32>echo ,\stop_morse VFOA| ncat -w 1 127.0.0.1 4532
stop_morse: VFOA,RPRT 0

mdblack98 added a commit that referenced this issue Aug 21, 2023
@ny4i
Copy link
Author

ny4i commented Aug 22, 2023

stop_morse, send_morse, send_voice_mem and stop_voice_mem all working. Tested on K4.

@mdblack98 mdblack98 added fixed Issue has been fixed and removed needs test Patches have been submitted but need testing to close issue labels Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fixed Issue has been fixed
Projects
None yet
Development

No branches or pull requests

2 participants