-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Switch to direct hamlib #12
Comments
I'm not opposed to it. I wouldn't do it as a replacement. More like a 3rd option. I personally prefer rigctld. I have it (rigctld) start up as a service in the background and never have to think about it. |
Ok, great. I think I can start working on this issue. Eventually we can discuss how I should solve hamlib configuration, should it be new window, textfile, or in current configuration window |
Okay, I made a branch for it. |
All the hamlib specific code should live in cat_interface.py |
Generally that's all. In case of any custom settings maybe in the future interface can be extended, but for now it's enough. One additional question. Why do you query PTT status? Is it needed for anything? Python bindings for hamlib has very poor documentation and it simply won't work in some cases (some rigs doesn't provide information about ptt so hamlib can't get that info) |
Setting PTT state is needed for voice keying. You can just have it return False if it's not available. |
Ok, finally I manage to set and get status of PTT, only setting RF power left. Maybe today I'll push changes as draft for first review (without tests and not aligned to your branch) |
Strange, I'm applying your changes from your branch in configuration.ui file, rebuilding with rebuild.sh script, and still I can't see those new controls in configuration window. Am I missing something? |
Hmm that is strange. I haven't hook up the controls to any code yet. No
laptop on my vacation. But you should still see them.
…On Fri, Jul 21, 2023, 11:47 PM darvark ***@***.***> wrote:
Strange, I'm applying your changes in configuration.ui file, rebuilding
with rebuild.sh script, and still I can't see those new controls in
configuration window. Am I missing something?
—
Reply to this email directly, view it on GitHub
<#12 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALWVLROFRUD2SRAK4K6OETTXRNZPJANCNFSM6AAAAAA2FVT62E>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Ah, I think I know why your not seeing it. I think you haven't checked out and applied your changes to the branch created for the issue. The branch should be called something like 12-switch-to-direct-hamlb |
I've made changes on my own branch and then applied your changes to configuration.ui (I've found only those changes in your branch), so still it should be visible since changes were applied to UI file |
Hi,
can we switch to use directly hamlib instead of fetching data from third party software like flrig? It should simplify cat communication and can be easily extended to add rotor support.
Only cons of such solution is that we have to configure hamlib from not1mm level, but still this would be much closer to n1mm than current solution.
The text was updated successfully, but these errors were encountered: