TK481 features, extra tests #1251
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
kk7ds
commented
Feb 4, 2025
kk7ds
commented
- Flatten browser UI and add addresses to elements
- Add string hex editing in browser
- Add dynamic subdevices feature flag
- Reload dynamic-subdev editors when volatile settings change
- tk481: Add call flag to mem.extra
- Add accelerator key for RepeaterBook query
- Add "include cached" toggle for RepeaterBook query
- Add test to validate_memory() all channels in test images
- vx7: Fix sample image with invalid frequencies
- Clean up yaesu_clone transmit echo munching
- icx90: Fix missing tuning steps for TV memories
- ft450: Mark as nostep_tuning because 1Hz tuning
- tk690: Expose proper PLL steps for 690,790,890
- ga510: Remove obviously-invalid channel from sample image
- uv5r: Remove invalid 1kHz step channels from sample image
- Enable validate_all_steps test for all, except some failing drivers
|
@N7XSD I wrote a test to poke your tuning step problem in this stack and immediately caused myself a bunch of extra pain when I found plenty of other drivers with the same problem from a decade ago. This has an xfail entry for your driver, so if this makes it in first you might need to remove that once you fix. |
The goal here is to mostly only create new subelements for structs, not arrays of ints, which makes it much easier to read things.
This will be used for indicating that we have a variable number of subdevices that can change (like for zoned/grouped radios). Adds this flag to several models that currently require a manual reload to update subdevices.
Related to #11803 This allows us to do a proximity search across all cached data instead of just the single area we're querying right now.
This fixes a bunch of issues found by this driver (mostly charset things) and introduces a dynamic list of XFAILs so we can be more prescriptive on these.
Two of the channels in the sample image have 2500Hz step sizes, which the radio cannot resolve. They display on the radio between the two steps (but not completely) but the radio is locking to the lower step in each case. The new UI properly rejects these during entry, but the old one did not, which means they're no longer even enter-able.
This is fundamentally a 5kHz step system, so denote it as such.
This radio can truly resolve 1Hz frequencies, so flag it as such.
These radios can't just resolve any frequency, so expose the proper values from the spec sheet for each model.
These were imported from a standard torture example frequency plan back in the old UI which did not assert proper alignment.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.