MIDI Pick-up Mode: the rapid-movement branch is unreachable
メンテナーはふだん 5 日以内に返信
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 76/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- cpp
調査の方向性
src/audiomixerboard.cpp の midiPickupTryApply、crossing branch、prevMidi、およびそれらの呼び出し箇所から始め、意図された動作について src/clientsettingsdlg.cpp の Pick-up Mode のヘルプテキストを読んでください。履歴の更新と前のサンプルの選択を追跡し、そのうえで、厳密に挟み込む急速な動きが処理されることを検証し、既存の許容範囲も引き続き正しいことを確認してください。
索引モデルが issue の本文から書いたものです。
説明
🤖 AI: Pick-up Mode's rapid-movement handling never executes. Its help text promises a control waits until the physical controller matches the software value, and an inner branch exists to also catch a controller swept fast enough that its values "skip over" that value. That branch is unreachable, for two independent reasons.
midiPickupTryApply returns before appending to the history buffer, so while a control is armed the buffer stays empty and the size() >= 2 guard in front of the crossing test never holds. Independently, prevMidi reads back(), while the sole call site has already pushed the incoming sample onto that same buffer — so prevMidi is the incoming sample, and the bracket test reduces to midiValue == currentValue, which the tolerance check above it has already accepted.
Driving those helpers verbatim across every strictly-bracketing two-sample crossing of the fader range, both samples outside the tolerance: on ee40cfe2, 0 of 147,440 pick up. Repairing either defect alone leaves 0. Repairing both gives 147,440. What remains on stock is the tolerance window by itself — armed at fader 50, 5 of 101 controller positions are accepted, 48 through 52 — so a fader swept briskly steps over the window and stops responding, which is the case the unreachable branch was written for.
Present since the MIDI GUI PR.
🤖 This message was written by AI and reviewed by @mcfnord.
- 主要言語
- C
- スター
- 1.1k
- フォーク
- 248
- 平均マージ
- 7日 14時間
- マージ済み PR（30日）
- 6
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
jamulussoftware/jamulus のほかの issue
-
Move translation checker (and potentially other runners) to ARM runner対応中かも @ann0see が 24 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
jamulussoftware/jamulus#3953 · コメント 2 件 ·
メンテナーはふだん 5 日以内に返信
-
Reused channel slot records the previous occupant's audio対応中かも @mcfnord が 3 日前に担当しました。 オープンAI bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
jamulussoftware/jamulus#3901 · コメント 4 件 · リアクション 1 件 ·
メンテナーはふだん 5 日以内に返信
-
AI
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
jamulussoftware/jamulus#3846 ·
メンテナーはふだん 5 日以内に返信
-
feature request
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
jamulussoftware/jamulus#3964 · コメント 3 件 ·
メンテナーはふだん 5 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
jamulussoftware/jamulus#3961 ·
メンテナーはふだん 5 日以内に返信
jamulussoftware/jamulus の issue をすべて見る
似ている issue
-
rc_runtime_activate_richpresence leaves a half-initialised entry when the buffer allocation failsオープン
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
RetroAchievements/rcheevos#558 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
libsdl-org/SDL#16464 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
chore(gateway): emit INFO budget reserved/settled logs for proactivity v2 (chip task_2855f4ec)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbackend
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
BasedHardware/omi#20940 ·
メンテナーはふだん 1 日以内に返信