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

rigctl "m" command not supported, causes fldigi grumpiness #8

Open
g0hww opened this issue Jun 13, 2018 · 2 comments
Open

rigctl "m" command not supported, causes fldigi grumpiness #8

g0hww opened this issue Jun 13, 2018 · 2 comments

Comments

@g0hww
Copy link

g0hww commented Jun 13, 2018

I've had wsjtx running nicely with linhpsdr using the model 240 PiHPSDR backend for a few days.
I've just built hamlib 3.3 from git repo and fldigi 4.0.17 from tarball in order to get fldigi working with linhpsdr.

It seems that fldigi has issues with linhpsdr because it can't get or set the mode of the linhpsdr rx.
I've manually tried rigctl with the "m" command but it times out getting a response, with the message "get_mode: error = Communication timed out". This seems to be happening within fldigi too.

Having a nose around in the linhpsdr code, it seems that the model 240 backend maps this to the "MD" command, but there seems to be no implementation in parse_cmd() of rigctl.c that handles the "MD" command that isn't preprocessed out with "#ifdef PIHPSDR". Of course I tried commenting out those preprocessor directives to see if that got me anywhere, but then suffered from enough compilation errors that I concluded that there was no merit in that optimistically attempted bodge.

I wonder if this is an oversight or otherwise is hopefully something high up on the TODO list?

Thanks for providing us with this nice software. I'm afraid I have a couple of other issues to report and don't want to sound ungrateful :)

Cheers,

Darren, G0HWW

@g0orx
Copy link
Owner

g0orx commented Jun 16, 2018

The current CAT commands need some more work which will be completed in the next update.
Don't worry about reporting problems, that is what the Beta release is for. I did have to rush the current version to be able to release it for Friedrichshafen.

@m5evt
Copy link
Contributor

m5evt commented Apr 9, 2020

MD now supported in this pull request #50

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

No branches or pull requests

3 participants