Project

General

Profile

Actions

Bug #10180

closed

Errors are not raised to UI

Added by Ran Katz almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
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 #1

Updated by Dan Smith almost 2 years 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 #2

Updated by Ran Katz almost 2 years ago

To reproduce:

  1. Load the test image for this radio : Quansheng_TG-UV2+.img
  2. go to the settings tab
  3. change the Dual Watch / Crossband monitor value from "normal" to "Dual Watch" (or "Crossband")
  4. 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 #3

Updated by Dan Smith almost 2 years 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 #4

Updated by Dan Smith almost 2 years ago

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

Also available in: Atom PDF