Skip to content

feat(generators): bass low-fifth reach — rfoPattern 'low5' (v0.1.15) - #9

Closed
ChrisBeWithYou wants to merge 1 commit into
feat/drop-barrefrom
feat/bass-low-fifth
Closed

ChrisBeWithYou wants to merge 1 commit into
feat/drop-barrefrom
feat/bass-low-fifth

Conversation

@ChrisBeWithYou

Copy link
Copy Markdown
Collaborator

Stacked on #8 (→ #7 → #6 → #5). Review only this diff.

What

bassRootGrip only reached up (fifth on s+1, octave on s+2). The fifth below the root — the reach the low string exists for, and the whole point of a 5-string's B — had no primitive (the item deferred from the string-count audit).

  • bassRootGrip gains lowFifth: P5 below the root on the string below, computed by pitch with an exact-midi check.
  • root_fifth_octave gains the opt-in rfoPattern='low5' variant: R–low5–5–8; unreachable low fifth degrades to the upper 5th.
  • The Root–5th–Octave Ladder rung gains a final vary step teaching the reach.

Pedagogy spec (bass-pedagogy, 2026-07-12 — implemented verbatim)

  • Guards: root.s>=1 and root.f>=2 (at f0/f1 the pc-math octave-push wraps the note up a 4th — the opposite of intent); null → degrade.
  • The default R-5-8-5 box is untouched — the canonical pre-scales lesson; smoke guards it byte-identical with the field absent.
  • Scope: root_fifth_octave only (octave_groove stays pure R-8 per the soul-motown ruling; right_hand_technique modes are pitch-invisible by design).
  • No root-selection bias — nearest-to-prev stands.
  • Anticipation-pickup variant deliberately deferred to the walking/lines rungs.

Verification

  • smoke-strings row (15): low5 geometry (C2·F1·G2·C3), default byte-parity, 4-string null-degrade to the upper 5th.
  • Full smoke 16/19 — same 3 pre-existing base reds.

🤖 Generated with Claude Code

https://claude.ai/code/session_01MS2YFb6UUSwJVV6CmEa25i

bassRootGrip only reached UP (fifth s+1, octave s+2); the fifth BELOW the
root - the reach the low string exists for, and the whole point of a
5-string's B - had no primitive (deferred from the string-count audit).

bassRootGrip gains lowFifth: the P5 below the root on the string below,
computed by pitch with an exact-midi check. Guards (bass-pedagogy spec
2026-07-12): root.s>=1 (a string exists below) AND root.f>=2 (at f0/f1 the
pc-math octave-push wraps the note UP a 4th - the opposite of intent); null
when unavailable.

root_fifth_octave gains the opt-in rfoPattern='low5' variant: R-low5-5-8
(root on the downbeat, traverses both fifths + the octave); an unreachable
low fifth degrades to the upper 5th so the pattern stays playable everywhere.
The default R-5-8-5 box is deliberately untouched (the canonical pre-scales
lesson) - guarded byte-identical by smoke. Scope per spec: root_fifth_octave
ONLY (octave_groove stays pure R-8 per the soul-motown ruling; the
right_hand_technique modes are pitch-invisible by design). No root-selection
bias - nearest-to-prev stands. The Root-5th-Octave rung gains a final vary
step teaching the reach (rfoPattern coded in base so it never leaks across
the rung's own vary steps).

Smoke: smoke-strings row (15) - low5 geometry (R, root-7 on the string
below, 5th, octave), default-parity (field absent === 'r5o', byte-identical),
4-string null-degrade to the upper 5th. Full run 16/19 (the 3 pre-existing
base reds).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MS2YFb6UUSwJVV6CmEa25i
@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e333c211-8fc8-4183-bafa-39e4e613b29a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/bass-low-fifth

Comment @coderabbitai help to get the list of available commands.

@ChrisBeWithYou

Copy link
Copy Markdown
Collaborator Author

Landed on virtuoso-dev via the 2026-07-14 batch integration (v0.2.2), not a GitHub merge event — these branches predate the main→dev reconcile (Byron's Gold-rung #11 took the line to 0.2.x) and were rebased, so their head SHAs don't match the commits that reached dev. All content is verified present in dev (incl. #5's 62fc72e tuningMidis, cherry-picked as f0c78c6). Closing to avoid a duplicate re-merge; the work ships to main via the dev→main PR #4 line. — automated

@ChrisBeWithYou
ChrisBeWithYou deleted the feat/bass-low-fifth branch July 16, 2026 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant