Skip to content

Comments

cat related updates#147

Open
VU2DLE wants to merge 1 commit intog0orx:masterfrom
VU2DLE:master
Open

cat related updates#147
VU2DLE wants to merge 1 commit intog0orx:masterfrom
VU2DLE:master

Conversation

@VU2DLE
Copy link

@VU2DLE VU2DLE commented Aug 3, 2021

Dear OM John,

Greetings!
Would like to inform regarding, few updates in rigctl.c, while experimenting CAT commands to control piHPSDR from RPI for RadioBerry project.

  1. API vfo_filter_changed() has been added to update the new filter value, for the ‘ZZFI’ command
  2. Added 2 missing ‘break’ statement, to make the CAT command NB and RIT sends expected reply.
  3. Vfo rit is incremented and decremented with ‘rit_increment’, instead of constant value (+/- 50), for the commands ‘ZZRU’ and ‘ZZRD’
  4. Updated parsing of command string and calculating the key lock status, for the command ‘LK’
  5. VFO functionalities A>B and A<B and A<>B are implemented as external CAT commands
    ‘ZZCB’ : VFO A to B
    ‘ZZCD’ : VFO B to A
    ‘ZZCF’ : Swap VFO A and B

Many 73s DE
VU2DLE Dileep

@dl1ycf
Copy link
Contributor

dl1ycf commented Oct 4, 2021

Looks reasonable. I have applied this to my master and "OldGPIO" branch. So you are OK with having the RIT step
hard-wired to 10 in the CW modes (that is the only inconsistency I noted about your contribution).

@VU2DLE
Copy link
Author

VU2DLE commented Oct 10, 2021

Looks reasonable. I have applied this to my master and "OldGPIO" branch. So you are OK with having the RIT step hard-wired to 10 in the CW modes (that is the only inconsistency I noted about your contribution).

OM Christoph,

I'm glad that you have made "rigctl" corrections. During my development time, I was curious and discussed with a few CW experts to know the purpose of hard-wired RIT step 10 in CW modes. Thank you for indicating this inconsistency and would like to get-rid of it in my next version.

Many 73s DE VU2DLE

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.

2 participants