fix(cat): make the Kenwood/Elecraft link an explicit USB-or-network choice

The network option IS implemented — the same ASCII CAT over TCP instead
of a cable, for a ser2net bridge, an Ethernet-serial adapter, or a radio
exposing its raw CAT port. What was wrong is how it was offered: the COM
port and the network address sat side by side with nothing to say that
the address wins whenever it is not empty.

Worse, the example address read 192.168.1.50:4532. That is Hamlib
rigctld's port — a different protocol, and the one OpsLog SERVES under
'Share CAT'. Anyone copying the example was pointing the radio link at
OpsLog's own server.

Now one choice, then the fields that belong to it. And when the far end
answers something that is not the radio's CAT, the error says that rather
than 'check the baud rate', which over TCP is advice about a setting that
cannot be the cause.
This commit is contained in:
2026-08-25 17:27:44 +02:00
parent 97ae05d688
commit 2f2fced15c
4 changed files with 41 additions and 6 deletions
+4 -2
View File
@@ -9,7 +9,8 @@
"Multi-monitor: the saved window position is now checked against the monitors themselves, not the rectangle that spans them. Monitors rarely fill that rectangle, and a window in one of the leftover gaps passed the old test while being invisible. A position that is genuinely lost is moved onto the nearest screen — keeping the window size — instead of being handed back to Windows, and the screen layout is written to the log at every start.", "Multi-monitor: the saved window position is now checked against the monitors themselves, not the rectangle that spans them. Monitors rarely fill that rectangle, and a window in one of the leftover gaps passed the old test while being invisible. A position that is genuinely lost is moved onto the nearest screen — keeping the window size — instead of being handed back to Windows, and the screen layout is written to the log at every start.",
"PowerGenius XL: the amplifier's real state is read at startup. Its status frame carries no 'operate' field — the state is in 'state' — so on the direct GSCP link the flag was never read at all and OpsLog opened claiming STANDBY on an amp that was in line, with the first press of the button then commanding the state it was already in. IDLE means in line, not keyed.", "PowerGenius XL: the amplifier's real state is read at startup. Its status frame carries no 'operate' field — the state is in 'state' — so on the direct GSCP link the flag was never read at all and OpsLog opened claiming STANDBY on an amp that was in line, with the first press of the button then commanding the state it was already in. IDLE means in line, not keyed.",
"Antenna Genius: an option to write the SELECTED antenna into MY_ANTENNA, under the name it carries on the switch, ahead of the band default from Operating conditions. Which of the two ports counts is decided by the antenna jack the radio is transmitting on (ANT1/ANT2 on a Flex), with the jack-to-port wiring set once in Preferences — neither device can report it. When the port cannot be told, the log keeps the band default rather than naming an antenna at random.", "Antenna Genius: an option to write the SELECTED antenna into MY_ANTENNA, under the name it carries on the switch, ahead of the band default from Operating conditions. Which of the two ports counts is decided by the antenna jack the radio is transmitting on (ANT1/ANT2 on a Flex), with the jack-to-port wiring set once in Preferences — neither device can report it. When the port cannot be told, the log keeps the band default rather than naming an antenna at random.",
"The awards tab beside the entry form (F3) now offers only the awards this station follows, and drops the ones switched off — the same list the Awards tab reads. It offered every award that existed, so a station chasing three of them picked references out of a list of twenty." "The awards tab beside the entry form (F3) now offers only the awards this station follows, and drops the ones switched off — the same list the Awards tab reads. It offered every award that existed, so a station chasing three of them picked references out of a list of twenty.",
"Kenwood / Elecraft CAT: USB and network are now an explicit choice, and only the fields of the chosen one are shown. Both were offered at once with nothing to say that the network address wins whenever it is filled in — and the example address named port 4532, which is Hamlib rigctld, a different protocol and the one OpsLog itself serves under Share CAT. Pointed at the wrong kind of port, the link now says so instead of blaming the baud rate."
], ],
"fr": [ "fr": [
"Console Elecraft : les mesures d'émission sont lues dès que la RADIO se déclare en émission, et plus seulement quand OpsLog l'a mise en émission. Passer en émission par le PTT de façade, une pédale ou le bouton du micro laissait le panneau croire à une réception — et comme les barres de puissance et de ROS ne sont lues qu'en émission, elles ne l'étaient jamais pour qui manipule à la main.", "Console Elecraft : les mesures d'émission sont lues dès que la RADIO se déclare en émission, et plus seulement quand OpsLog l'a mise en émission. Passer en émission par le PTT de façade, une pédale ou le bouton du micro laissait le panneau croire à une réception — et comme les barres de puissance et de ROS ne sont lues qu'en émission, elles ne l'étaient jamais pour qui manipule à la main.",
@@ -18,7 +19,8 @@
"Multi-écrans : la position enregistrée est désormais vérifiée contre les écrans eux-mêmes, et non contre le rectangle qui les englobe. Les écrans remplissent rarement ce rectangle, et une fenêtre tombée dans un des trous passait l'ancien test tout en étant invisible. Une position réellement perdue est déplacée sur l'écran le plus proche — en conservant la taille de la fenêtre — au lieu d'être rendue à Windows, et la disposition des écrans est écrite dans le journal à chaque démarrage.", "Multi-écrans : la position enregistrée est désormais vérifiée contre les écrans eux-mêmes, et non contre le rectangle qui les englobe. Les écrans remplissent rarement ce rectangle, et une fenêtre tombée dans un des trous passait l'ancien test tout en étant invisible. Une position réellement perdue est déplacée sur l'écran le plus proche — en conservant la taille de la fenêtre — au lieu d'être rendue à Windows, et la disposition des écrans est écrite dans le journal à chaque démarrage.",
"Power Genius XL : l'état réel de l'amplificateur est lu au démarrage. Sa trame d'état ne contient pas de champ « operate » — l'état est dans « state » — si bien que sur la liaison GSCP directe l'indicateur n'était jamais lu : OpsLog s'ouvrait en annonçant STANDBY sur un ampli en ligne, et le premier appui commandait l'état dans lequel il se trouvait déjà. IDLE veut dire en ligne, pas en émission.", "Power Genius XL : l'état réel de l'amplificateur est lu au démarrage. Sa trame d'état ne contient pas de champ « operate » — l'état est dans « state » — si bien que sur la liaison GSCP directe l'indicateur n'était jamais lu : OpsLog s'ouvrait en annonçant STANDBY sur un ampli en ligne, et le premier appui commandait l'état dans lequel il se trouvait déjà. IDLE veut dire en ligne, pas en émission.",
"Antenna Genius : une option pour inscrire l'antenne SÉLECTIONNÉE dans MY_ANTENNA, sous le nom qu'elle porte sur le switch, avant l'antenne par défaut des conditions de trafic. C'est la prise d'antenne sur laquelle la radio émet (ANT1/ANT2 sur un Flex) qui décide du port retenu, le câblage prise→port se règlant une fois dans les préférences — aucun des deux appareils ne peut le dire. Quand le port ne peut pas être déterminé, le journal conserve l'antenne par défaut plutôt que d'en nommer une au hasard.", "Antenna Genius : une option pour inscrire l'antenne SÉLECTIONNÉE dans MY_ANTENNA, sous le nom qu'elle porte sur le switch, avant l'antenne par défaut des conditions de trafic. C'est la prise d'antenne sur laquelle la radio émet (ANT1/ANT2 sur un Flex) qui décide du port retenu, le câblage prise→port se règlant une fois dans les préférences — aucun des deux appareils ne peut le dire. Quand le port ne peut pas être déterminé, le journal conserve l'antenne par défaut plutôt que d'en nommer une au hasard.",
"L'onglet des diplômes à côté de la saisie (F3) ne propose plus que les diplômes suivis par la station, et écarte ceux qui sont désactivés — la même liste que l'onglet Diplômes. Il proposait tous les diplômes existants : une station qui en chasse trois choisissait ses références dans une liste de vingt." "L'onglet des diplômes à côté de la saisie (F3) ne propose plus que les diplômes suivis par la station, et écarte ceux qui sont désactivés — la même liste que l'onglet Diplômes. Il proposait tous les diplômes existants : une station qui en chasse trois choisissait ses références dans une liste de vingt.",
"CAT Kenwood / Elecraft : USB et réseau sont désormais un choix explicite, et seuls les champs du mode retenu s'affichent. Les deux étaient proposés en même temps sans rien indiquer que l'adresse réseau l'emporte dès qu'elle est renseignée — et l'exemple d'adresse citait le port 4532, celui de Hamlib rigctld, un autre protocole, et justement celui qu'OpsLog propose lui-même sous « Partager le CAT ». Pointé sur un port de mauvaise nature, le lien le dit maintenant au lieu d'accuser la vitesse série."
] ]
}, },
{ {
+26 -2
View File
@@ -3181,6 +3181,25 @@ export function SettingsModal({ onClose, onSaved, initialSection, onMainPaneChan
)} )}
{(catCfg.backend === 'kenwood' || catCfg.backend === 'elecraft') && ( {(catCfg.backend === 'kenwood' || catCfg.backend === 'elecraft') && (
<> <>
{/* USB or network, as a choice.
The two were offered side by side with nothing to say which one
the backend would use it prefers the network address whenever
the field is not empty, which is invisible from here. */}
<div className="space-y-1">
<Label>{t('cat.kwLink')}</Label>
<div className="inline-flex rounded-md border border-border overflow-hidden text-xs">
{[['usb', t('cat.kwLinkUsb')], ['net', t('cat.kwLinkNet')]].map(([v, label]) => (
<button key={v} type="button"
onClick={() => setCatCfg((s) => ({ ...s, kenwood_host: v === 'usb' ? '' : (s.kenwood_host || '') }))}
className={cn('px-3 py-1.5 font-medium',
((catCfg.kenwood_host || '').trim() !== '') === (v === 'net')
? 'bg-primary text-primary-foreground' : 'text-muted-foreground hover:bg-muted')}>
{label}
</button>
))}
</div>
</div>
{(catCfg.kenwood_host || '').trim() === '' && (<>
<div className="space-y-1"> <div className="space-y-1">
<Label>{t('cat.kenwoodPort')}</Label> <Label>{t('cat.kenwoodPort')}</Label>
<div className="flex gap-2"> <div className="flex gap-2">
@@ -3202,13 +3221,18 @@ export function SettingsModal({ onClose, onSaved, initialSection, onMainPaneChan
{[4800, 9600, 19200, 38400, 57600, 115200].map((r) => <SelectItem key={r} value={String(r)}>{r}</SelectItem>)} {[4800, 9600, 19200, 38400, 57600, 115200].map((r) => <SelectItem key={r} value={String(r)}>{r}</SelectItem>)}
</SelectContent> </SelectContent>
</Select> </div> </Select> </div>
</>)}
{(catCfg.kenwood_host || '').trim() !== '' && (
<div className="space-y-1"> <div className="space-y-1">
<Label>{t('cat.kenwoodHost')}</Label> <Label>{t('cat.kenwoodHost')}</Label>
<Input <Input
value={catCfg.kenwood_host || ''} value={catCfg.kenwood_host || ''}
placeholder="192.168.1.50:4532" placeholder="192.168.1.50:4999"
onChange={(e) => setCatCfg((s) => ({ ...s, kenwood_host: e.target.value }))} onChange={(e) => setCatCfg((s) => ({ ...s, kenwood_host: e.target.value }))}
/> </div> />
<p className="text-xs text-muted-foreground">{t('cat.kenwoodHostHint')}</p>
</div>
)}
<div className="space-y-1"> <div className="space-y-1">
<label className="flex items-center gap-2 text-xs cursor-pointer"> <label className="flex items-center gap-2 text-xs cursor-pointer">
<Checkbox checked={!!catCfg.kenwood_low_lines} onCheckedChange={(c) => setCatCfg((s) => ({ ...s, kenwood_low_lines: !!c }))} /> <Checkbox checked={!!catCfg.kenwood_low_lines} onCheckedChange={(c) => setCatCfg((s) => ({ ...s, kenwood_low_lines: !!c }))} />
File diff suppressed because one or more lines are too long
+9
View File
@@ -273,6 +273,15 @@ func (k *Kenwood) Connect() error {
// commands back produces the same). Say which of the two it is, and quote // commands back produces the same). Say which of the two it is, and quote
// what came back, because that is the fact that decides where to look. // what came back, because that is the fact that decides where to look.
if seen := k.heard; seen != "" { if seen := k.heard; seen != "" {
// Over a NETWORK link, baud means nothing and saying "check the baud
// rate" sends the operator to a setting that cannot be the cause.
// The usual mistake there is a different protocol on the other end:
// Hamlib's rigctld (4532) answers text, so it looks alive and never
// replies to ID;. OpsLog itself serves that port under "Share CAT",
// which makes it an easy number to reach for.
if k.host != "" {
return fmt.Errorf("kenwood: %s is sending data but no reply to ID; or IF; — got %q. That end is not speaking the radio's CAT: point this at a serial-over-network bridge or the radio's own raw CAT port, not at Hamlib rigctld (4532)", k.host, seen)
}
return fmt.Errorf("kenwood: %s is sending data but no reply to ID; or IF; — got %q. Check the baud rate (set to %d here) and that nothing else is echoing the port", k.where(), seen, k.baud) return fmt.Errorf("kenwood: %s is sending data but no reply to ID; or IF; — got %q. Check the baud rate (set to %d here) and that nothing else is echoing the port", k.where(), seen, k.baud)
} }
if k.host != "" { if k.host != "" {