Project

General

Profile

Actions

Bug #11471

closed

TIDRADIO TD-H3 power levels for A / B VFO are have incorrect options

Added by Adam Smith 9 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
08/08/2024
Due date:
% Done:

100%

Estimated time:
Chirp Version:
next
Model affected:
TIDRADIO TD-H3
Platform:
Windows
I read the instructions above:
Yes

Description

What is the behavior you are seeing?
In the Settings tab, "A/B Channel" section, the "A TX Power" and "B TX Power" settings (which set the power levels associated with the A and B channels when radio is in VFO mode) have three options, "Low", "Mid", and "High" when the radio supports only two power level settings. In my testing, the "High" power level setting in CHIRP unexpectedly results in the radio showing Low power (Menu item 2. Tx Power == Low) which "Mid" setting in CHIRP results in the radio showing Tx Power == High. (Low power setting in CHIRP correctly results in Low on radio.)

What is the behavior you were expecting?
Expected behavior is that the "A TX Power" and "B TX Power" fields in the Settings tab, "A/B Channel" section for a TIDRADIO TD-H3 should have only the "High" and "Low" power options, and that those options should result in the corresponding power level being programmed in the VFO A / B settings of the radio.
Note: This is similar to, but not the same issue as, previous Issues #11163 and #11263. Power levels for stored channels in the Memories tab appear to be set correctly with just two options "Low" and "High" as supported by radio.

Can you reproduce the problem all the time?
The problem is reproduceable with the following software versions:
TIDRADIO TD-H3 purchased as "HAM" version and running software version 240702 in NORMAL/unlocked mode.
CHIRP next-20240706 on Python 3.10.8 wxPython 4.2.0 msw (phoenix) wxWidgets 3.2.0

What are the steps required to reproduce the problem?
Step 1:
Download configuration from radio using Vendor "TIDRADIO" and Model "TD-H3". VFO A power level should be set to "Low" and VFO B power level should be set to "High" in the radio.

Observe that the Settings tab, "A/B Channel" section field "A TX Power" is correctly "Low", but "B TX Power" is unexpectedly "Mid".
Step 2:
Change the "A Tx Power" field to "High"; Save the configuration; upload to the radio.
Observe that the VFO A power level in menu item "2. Tx Power" is now "Low" despite being set to "High" in CHIRP.

Is this specific to a certain radio model (driver) or something that you can reproduce with another radio?
This appears to be specific to TD-H3 but I don't have other relevant models of radio to test.

Debug log and .img file attached, as well as screen grab of field with 3 power level options.
clip of CHIRP Settings tab showing more power level options than the TD-H3 supports


Files

clipboard-202408082122-xlunq.png (18.6 KB) clipboard-202408082122-xlunq.png clip of CHIRP Settings tab showing more power level options than the TD-H3 supports Adam Smith, 08/08/2024 06:22 PM
TIDRADIO_TD-H3_20240808 test for bug report.img (8.18 KB) TIDRADIO_TD-H3_20240808 test for bug report.img Adam Smith, 08/08/2024 06:51 PM
tdh8.py (82.6 KB) tdh8.py Dan Smith, 08/12/2024 05:49 PM
Actions #1

Updated by Dan Smith 9 months ago

I really think that setting the VFO state in radios is a totally useless feature, so I'd rather just remove this than fix it.

Are you reporting this (thanks by the way) because it's wrong, or because you really have some compelling reason to control this from software and need it fixed?

Actions #2

Updated by Adam Smith 9 months ago

The existing behavior is pretty misleading/unexpected (setting "High" power in CHIRP produces the "Low" power behavior in the radio, assuming that the menu display and the actual power output are in sync). I would call that "wrong", and would generally subscribe to a principal of "if it's there, it should work".

As to what constitutes a "compelling reason" to set this, I'm not sure I have a killer argument. If the radio supports the setting, it would be nice to allow it / make it work in a predictable way. Clearly you could make the same setting from the front panel, but by that line of logic I don't need software to program the radio at all? I suppose one might automate controlling a radio entirely from the software/programming interface, and want to set this parameter. Maybe in a case where you wanted to set something temporarily without changing or adding memory channels. But I'm not actually suggesting that that's a common or sensible idea. Perhaps more realistic, one might want to set radios to well-known state before giving them to new users, or using for training/demonstration, etc.

So I guess the answer is "I reported it because it's wrong", and with the perhaps hopeful view that it seems likely to be straightforward to fix, given that the similar setting/options are correctly supported in the channel memories on this radio. That said I know nothing about CHIRP's code or the complexities of interfacing with these radios, in particular.

Then again, I'm a software product manager by trade, so presuming that bugs are easy to fix may be an occupational hazard ;-).

Actions #3

Updated by Dan Smith 9 months ago

Yep, definitely agree that if it's there it should work, no question about that :)

The H3's power handling on memories is odd and inconvenient. It took quite a bit of round-and-round to get it working properly (not for any good reason other than obscurity and I didn't write the initial driver), so I'm a bit jaded. So for something less important like the active state (which I'm biased to think is not a very useful feature in the first place), I'm probably not interested in spending much effort on it.

Attached is a super simplistic change that assumes the VFO power levels work the same way as the memory ones. Can you test it with LoadingTestModules ? Please confirm it reports the same as the radio and commands the radio to the right thing. If I open your image with this applied, I see A=Low, B=High, which I think is what you describe as correct.

Thanks!

Actions #4

Updated by Adam Smith 9 months ago

Updated to latest CHIRP-next (-20240807). Downloaded module from this issue.
Set radio to VFO A "High", VFO B "Low", downloaded config to CHIRP, looks correct in the corresponding TX Power fields.
Changed config on computer side to A "Low", B "High". The "Mid" power option no longer appears in the CHIRP UI. Uploaded config to radio, and observed updated values on the radio.

Brief test, but it appears to me that it works on both download and upload, on my radio.

For clarity, I don't have the equipment at hand to test the actual power output, but assuming that the radio's on-screen menu reflects the actual setting then this looks good!

Thanks Dan!

Actions #5

Updated by Dan Smith 9 months ago

Yep, as long as the radio thinks it's in Low or High mode, that's all we can do. CHIRP tells the radio only symbolically which power level to use, not anything absolute. So, thanks I'll include this for the next build.

Just to expound on my "not useful" comment.. The VFO state is basically a "scratch pad" by which you dial to a new/temporary frequency arrangement and either operate there briefly, or use that as input to store a memory channel. So it's very useful on the radio itself, but setting up the scratch pad from the computer is sort of a non-feature, IMHO. It's ephemeral state, so kinda meh.

Actions #6

Updated by Adam Smith 9 months ago

Yeah I understand the context of your comment, no worries. Thanks for looking at the issue quickly and getting to a fix! Not sure it's a big deal - but in my case the radio I received is acting a bit strange in some other power-level-related ways, and any apparent issue in how CHIRP was working made me suspicious that the programming process could be at fault rather than the specific unit.

To be clear - I believe your fix works, and my radio may have a separate issue about generating full power.

Thanks again!

Actions #7

Updated by Dan Smith 9 months ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF