Revive RIG_LEVEL_BWC (or equivalent) — set/get filter width independently of set_mode
メンテナーはふだん 4 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- c
調査の方向性
Start by tracing the level-flag definitions in include/hamlib/rig.h and src/idx_builtin.h, then read the disabled bandwidth handlers in rigs/jrc/jrc.c and the existing newcat bandwidth functions and set_level/get_level dispatch. The key decision is how to represent a new level when all 64 flag bits are allocated. Done means a viable independent bandwidth level is exposed for the Yaesu family without resending the mode.
索引モデルが issue の本文から書いたものです。
説明
Motivation
I'm building a hamlib-driven remote control panel intended to work
across rig models with minimal per-rig-specific config — a physical pot
mapped to filter width, for example. Currently the only entry point for
bandwidth is set_mode(rig, vfo, mode, width) / get_mode(...) — width
is bundled with mode, not independently settable via set_level. That
means every bandwidth-only change has to resend the mode too, and the
caller has to already know and pass through the current mode correctly
just to touch bandwidth — awkward for a UI element that's conceptually
unrelated to mode switching.
Prior art already in the tree
include/hamlib/rig.h:1103—RIG_LEVEL_BWC("Bandwidth Control, arg
int (Hz)") is fully documented but the#defineis commented out.src/idx_builtin.h:144—LVL_BWCmirrors this, also commented out.rigs/jrc/jrc.c(NRD-535/545) has a complete get/set
implementation wired toRIG_LEVEL_BWC— real CAT commands
("W%0*d"to set,"W"to query) — but both blocks are wrapped in
#if 0and unreachable.rigs/jrc/nrd535.c/nrd545.c's level masks
even still carry a/*RIG_LEVEL_BWC*/comment marking where it used
to be included.
So this was built once, apparently disabled (likely during the 32-bit →
64-bit level-flag migration, since bit 31 was the old scheme's awkward
last bit) and never revived.
For the Yaesu/newcat family specifically, the hard part is already done:
newcat_set_rx_bandwidth/newcat_get_rx_bandwidth already implement
per-mode width get/set correctly — they're just only ever called from
inside set_mode/get_mode, never exposed as an independent level.
One real obstacle, not glossing over it
All 64 bits of the setting_t flag space (CONSTANT_64BIT_FLAG) are
already allocated, up through bit 63. So this isn't just "uncomment the
define" — either an existing, effectively-unused level would need to be
retired to free a bit, or the level-flag representation needs to grow.
That's a maintainer-level design call, not something I can resolve from
outside.
Proposal
Revive RIG_LEVEL_BWC (or a renamed equivalent) on a viable bit, wire
it into newcat's set_level/get_level dispatch for the Yaesu family
by reusing the existing newcat_set_rx_bandwidth/get_rx_bandwidth,
using JRC's disabled implementation as a design precedent for the
get/set shape. I can test against real FT-2000 hardware, and have
bench-measured per-mode width tables for it (#2147) as a correctness
reference.
Analysis of the hamlib source (rig.h, idx_builtin.h, jrc.c) and issue drafted with assistance from Claude (Anthropic).
- 主要言語
- C
- スター
- 851
- フォーク
- 308
- 平均マージ
- 5日 5時間
- マージ済み PR(30日)
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Hamlib/Hamlib のほかの issue
-
Enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
メンテナーはふだん 4 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 4 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
Hamlib/Hamlib#2149 · コメント 5 件 ·
メンテナーはふだん 4 日以内に返信
-
grbltrk: Control uses WPos (work position), but reads use MPos (machine position)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 50/100
メンテナーはふだん 4 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
メンテナーはふだん 4 日以内に返信
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
FStarLang/karamel#784 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
DaveGamble/cJSON#1100 ·
-
UB: (int) cast of a user double outside int range in dispatch / vm_index_is_int / flat_index_atオープンarea:runtime-vm
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
InauguralSystems/EigenScript#1748 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
leanprover/lean4#15677 ·
メンテナーはふだん 1 日以内に返信