Bug #10180: Errors are not raised to UI - CHIRP

Actions

Copy link

Bug #10180

closed

Errors are not raised to UI

Added by Ran Katz over 1 year ago. Updated over 1 year ago.

Status:

Closed

Priority:

Normal

Assignee:

Dan Smith

Category:

Target version:

chirp-py3

Start date:

12/13/2022

Due date:

% Done:

100%

Estimated time:

Chirp Version:

next (py3)

Model affected:

tg-uv2+

Platform:

MacOS

Debug Log:

I read the instructions above:

Description

I have some "functional" errors implemented for some not-allowed setting combinations,
In the current daily version they pop-up an error window, in the py3 there is no pop up and I can only see the errors the terminal window.

Actions

Copy link

Updated by Dan Smith over 1 year ago

Assignee set to Dan Smith
Target version set to chirp-py3

Can you give me an example so I can repro? In general the new UI is pretty good at catching and raising (because if the simpler design without the radio thread). But, if you can tell me what to do to make it happen I'll fix it.

If I need a specific image, then please attach, and then tell me what values to put in that will not get reported.

Actions

Copy link

Updated by Ran Katz over 1 year ago

To reproduce:

Load the test image for this radio : Quansheng_TG-UV2+.img
go to the settings tab
change the Dual Watch / Crossband monitor value from "normal" to "Dual Watch" (or "Crossband")
check the keypad lock checkbox

at this point there should have been (and is on the py2 version) a pop up error, I can see the error in the terminal but no pop up...

(reverse order i.e. keypad lock first and then dual watch is also an error and exhibits the same behavior)

Running on: MacOS 12.6
python 3.10.8 venv
command line, recent (yesterday's) chirp py3 version

BTW, I am getting the following warning when I start:
" Cannot set locale to language "English (Israel)". " I don't know if it is relevant.

Actions

Copy link

Updated by Dan Smith over 1 year ago

Aha, perfect, I see thanks. I had the memory editor in mind, but I'll track this down.

The translation thing is only because you're running from the tree and not from a build where the locale stuff is there. I need to quiet that down, so I'll do that as well. Thanks!

Actions

Copy link