Bug #11372
closedRetevis RA87 doesn't retain scan mode changes
100%
Description
For the Retevis RA87:
Set scan mode in CHIRP for menu option 12 (Scan mode) from TO (the default, timeout scan resume) to CO (Carrier based scan resume). Upload from CHIRP to the RA87. The unit restarts, as usual. But option 12 has remained at the default TO setting.
I verified this with another person also running CHIRP on their Windows laptop, and owning the same radio. We both see the same issue.
What should happen, is that the radio should be set to CO mode for menu 12, scan mode.
Files
Updated by David Oswald 6 months ago
This segment of code could be the issue:
# menu 12 - SCAN
options = ["Carrier Operated (CO)", "Time Operated (TO)",
"SEarch (SE)"]
rs = RadioSettingValueList(options, options[_settings.scan])
rset = RadioSetting("scan", "Scan Resume Method", rs)
rset.set_doc("Menu 12")
basic.append(rset)
I think the "options" array should look like this:
options = ["Time Operated (TO)", "Carrier Operated (CO)", "Search (SE)"]
Updated by David Oswald 6 months ago
I see now that I miscategorized this as CHIRP Legacy, when it should have been NEXT. This is a bug in NEXT.
Updated by Jim Unroe 6 months ago
- File retevis_ra87.py retevis_ra87.py added
- Status changed from New to In Progress
- Assignee set to Jim Unroe
- Target version set to chirp-py3
- Chirp Version changed from legacy to next
Yep. Looks like I had a dyslexic moment when building the options list. Try this test driver module and provide feedback. Here's how: LoadingTestModules
Updated by David Oswald 6 months ago
I tested out the fix. It works correctly; if I set CO as the scan method in CHIRP, it gets uploaded correctly to the radio now. Looks like it was the order of elements in that options array, like I suspected. Thanks for getting to this.
What happens of _ham = True is set? Just curious but there's probably a better place to further discuss the module.
Updated by David Oswald 5 months ago
- File bug_11372.patch bug_11372.patch added
Here's a git patch that contains the proposed change.
Updated by Jim Unroe 5 months ago
Jim Unroe wrote in #note-6:
David Oswald wrote in #note-5:
Here's a git patch that contains the proposed change.
Yeah. I meant to get this submitted but it fell through the cracks. I will try to get to get it done.
Actually I had already submitted this to be included. We are just waiting on it to be approved and merged into CHIRP.
Updated by Anonymous 5 months ago
- Status changed from In Progress to Closed
- % Done changed from 0 to 100
Applied in changeset github|d89bb3cb383943e5082b02e8be22106dac0a6f63.