Skip to content

Support Icom IC-705 CAT - #229

Open
ruilvo wants to merge 8 commits into
rt-bishop:mainfrom
ruilvo:feature-icom-705-support-try3
Open

Support Icom IC-705 CAT#229
ruilvo wants to merge 8 commits into
rt-bishop:mainfrom
ruilvo:feature-icom-705-support-try3

Conversation

@ruilvo

@ruilvo ruilvo commented Jul 24, 2026

Copy link
Copy Markdown

So, I basically vibe-coded in support for the Icom IC-705.

It supports two-transceiver mode, same as for the Yaesu transceivers, but I also added "split mode". This mode uses the split functionality in the transceiver to keep both Rx and Tx frequencies in tune, on a single radio.

Please use a merge-commit to merge this in, do not use a fast-forward. This is a horrible commit history...

Closes #152.

ruilvo added 7 commits July 24, 2026 20:07
- Added support for Icom IC-705 radio model with split mode functionality.
- Updated RadioTrackingService to handle single-radio split mode and dual-radio configurations.
- Modified MainContainer to provide appropriate radio controllers based on selected model.
- Enhanced SettingsRepo to include split mode preference in radio control settings.
- Updated UI in SettingsDialog to allow toggling of split mode for IC-705 and adjust device selection accordingly.
- Improved handling of baud rates based on selected radio model.
- Added extended operations in IRadioController for IC-705 specific commands.
Uses 0x25 01 command to set the Tx frequency,
which is the correct command when in split mode.
I was setting the Tx frequency only when we were
in Tx mode, but the Icom 705 will accept the
command to set the Tx frequency even when in Rx mode, so
there is no need to check the PTT state before sending the command.
@ruilvo
ruilvo force-pushed the feature-icom-705-support-try3 branch from cae9b30 to 81525b6 Compare July 24, 2026 19:10
@ruilvo

ruilvo commented Jul 24, 2026

Copy link
Copy Markdown
Author

Also, I don't really get why two buttons, so I have this other branch where I merged the two buttons and even made the button "block" while it's connecting.

I think it looks pretty cool (picture links to Youtube)

Single connect and track button

If you want that, see it in ruilvo/Look4Sat@6f71a6c

@rt-bishop

Copy link
Copy Markdown
Owner

Hey @ruilvo! Thanks a lot for this contribution, that's an interesting piece of work. I don't mind people using AI in any way, so it's great to see more functionality that covers someone's usecase. It may take me some time to review the code, as I don't have the gear to test it on. Hopefully the Yaesu functionality still works as well =)

What do you mean by the horrible commit history? I prefer to use squash as so many devs are horrible with their commit messages. I only use merge/rebase for cases when I actually care about intermediate commits. Your suggested feature seems to be complete and self-contained, so I don't see the reason to use merge here. Sounds like a holy war topic, ha-ha)

@ruilvo

ruilvo commented Jul 28, 2026

Copy link
Copy Markdown
Author

Hi,
Thanks!
Yeah, in the meanwhile I did an interactive rebase and re-wrote the commit history so it looks half-decent now.
I don't think I broke the Yaesu functionality, at lease I hope I didn't.
The Icom IC-705 functionality I of-course tested with my own device.

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.

Sending Up/Downlink-Frequencies to Transceiver via Bluetooth

2 participants