mirror of
https://github.com/9x/sheetmusictrainer.git
synced 2026-11-03 22:03:54 +01:00
Compare commits
13 Commits
feature/ph
...
6253194632
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
6253194632 | ||
|
|
ffb94cc93f | ||
|
|
9dc95e3787 | ||
|
|
263b6f6b69 | ||
|
|
d76d24ab01 | ||
|
|
a965dde585 | ||
|
|
fc8f9bfb44 | ||
|
|
2ed1a88184 | ||
|
|
57a6a654a0 | ||
|
|
85fe20bf51 | ||
|
|
b214c209f9 | ||
|
|
15b8705907 | ||
| da060e5bb9 |
5
TOOLS_LOCAL.md
Normal file
5
TOOLS_LOCAL.md
Normal file
@@ -0,0 +1,5 @@
|
||||
# Local deployment (nyx server) — beta channel
|
||||
- Managed by Nyx; deploy script: `~/deploy/smt-beta.sh [branch]` (fetch, build, docker rebuild, restart)
|
||||
- Container: `smt-beta` (nginx:alpine, 127.0.0.1:8081), image `smt-beta-image`
|
||||
- Tailnet URL: http://nyx-rg3jpx6w.internal.0jm.de/ (tailscale serve → 8081)
|
||||
- Note: `tailscale serve --https` not available on this tailnet (feature not enabled); plain HTTP inside tailnet only
|
||||
@@ -23,6 +23,30 @@ Implementation of [PHRASE_MODE_PLAN.md](../PHRASE_MODE_PLAN.md) and
|
||||
| R13 Safe lifecycle | ✅ | Pause on tab-hide, mic error, mic-off, dialogs; audio-group cancellation; run tokens; StrictMode-safe; unmount cleanup |
|
||||
| R14 Tested delivery | ✅ | 149 unit/integration tests + real-browser synthetic-mic suite (see below) |
|
||||
|
||||
## Late feedback round (post first delivery)
|
||||
|
||||
- **Guitar octave bug**: the phrase renderer drew sounding pitch; fixed by
|
||||
deriving the written pitch through the shared `spelledPitch` helper, and
|
||||
clef/transpose resolution extracted to `resolveClefTranspose` (used by both
|
||||
modes — requested reuse of proven single-note logic).
|
||||
- **Count-in**: an extra 5th click at the score's downbeat made players start
|
||||
a beat early (perceived as "play the note before"); removed (the count-in's
|
||||
last click IS the downbeat).
|
||||
- **Tempo timing proven**: browser test plays every Ode note exactly in its
|
||||
audio window → 8/8 matched (also caught + fixed false scoring during the
|
||||
count-in itself).
|
||||
- **Repeated notes**: release-based re-arm + 2.5 s at-your-pace timeout + a
|
||||
clear status hint ("release, then strike again").
|
||||
- **Auto-continue** (tempo, default on): exercises roll to the next section
|
||||
(wrapping), melodies roll a new seed, scale drills restart.
|
||||
- **BPM widget**: −/+ buttons + slider + free-typing text field (commit on
|
||||
blur/Enter) — the single-note widget pattern.
|
||||
- **Live-note staff** next to the mic readout: written-pitch mini staff with
|
||||
8va/8vb normalization; box matches the mic button; VexFlow's
|
||||
`space_above_staff_ln` (~40.5px) offset compensated (see TECHNICAL_REALIZATION).
|
||||
- **Library**: 19 items (5 verified PD melodies + 14 labeled originals).
|
||||
- **Zen centering** + live staff sizing/overlap per user feedback.
|
||||
|
||||
## Verification actually performed
|
||||
|
||||
- `npm run lint`, `npm run build`, `npm test -- --run`: all green (149 tests,
|
||||
|
||||
@@ -332,6 +332,7 @@ function App() {
|
||||
midi={displayedPitch ? displayedPitch.midi : null}
|
||||
transpose={resolveClefTranspose(currentInstrumentDef, settings.difficulty).transpose}
|
||||
keySignature={settings.keySignature}
|
||||
clef={currentInstrumentDef.clefMode === 'treble' ? 'treble' : currentInstrumentDef.clefMode === 'bass' ? 'bass' : undefined}
|
||||
theme={settings.theme}
|
||||
/>
|
||||
<div className={`pitch-readout ${displayedPitch ? 'active' : ''}`}>
|
||||
|
||||
@@ -1,9 +1,12 @@
|
||||
import { Settings, Guitar, Music, Gauge } from 'lucide-react';
|
||||
import React, { useEffect, useState } from 'react';
|
||||
import { Guitar, Music, Gauge } from 'lucide-react';
|
||||
import { TUNINGS, INSTRUMENT_TUNINGS } from '../music/Tunings';
|
||||
import { INSTRUMENT_DEFINITIONS } from '../music/InstrumentConfigs';
|
||||
import { getTempoMarking } from '../music/TempoMarkings';
|
||||
import { TuningMeter } from './TuningMeter';
|
||||
|
||||
import { TuningMeter } from './TuningMeter';
|
||||
import { TargetNoteControls } from './TargetNoteControls';
|
||||
import { MetronomeWidget } from './MetronomeWidget';
|
||||
import { phraseRunBus } from '../hooks/phraseRunBus';
|
||||
import { useSettings } from '../context/useSettings';
|
||||
import { type Difficulty, type RhythmSettings } from '../types/SettingsTypes';
|
||||
|
||||
@@ -17,10 +20,10 @@ export const Controls: React.FC<ControlsProps> = ({ currentPitch }) => {
|
||||
const { settings, updateSettings } = useSettings();
|
||||
// Alias to minimize refactor, or just use updateSettings.
|
||||
const onUpdateSettings = updateSettings;
|
||||
const handleDifficultyChange = (e: React.ChangeEvent<HTMLSelectElement>) => {
|
||||
onUpdateSettings({ ...settings, difficulty: e.target.value as Difficulty });
|
||||
};
|
||||
|
||||
// Phrase run gate: the metronome only ticks while the phrase trainer is
|
||||
// counting in / playing (armed by the user toggle, fired by the run).
|
||||
const [phraseRunActive, setPhraseRunActive] = useState(false);
|
||||
useEffect(() => phraseRunBus.subscribe(setPhraseRunActive), []);
|
||||
const handleTuningChange = (e: React.ChangeEvent<HTMLSelectElement>) => {
|
||||
onUpdateSettings({ ...settings, tuningId: e.target.value });
|
||||
};
|
||||
@@ -110,23 +113,6 @@ export const Controls: React.FC<ControlsProps> = ({ currentPitch }) => {
|
||||
If Tuning is hidden, Note Set becomes Item 2.
|
||||
I'll forgo complexity and just render what's available. */}
|
||||
|
||||
{/* Row 2, Col 1 */}
|
||||
<div className="control-group">
|
||||
<label className="control-label">
|
||||
<Settings size={18} />
|
||||
<span>Note Set</span>
|
||||
</label>
|
||||
<select
|
||||
value={settings.difficulty}
|
||||
onChange={handleDifficultyChange}
|
||||
className="control-select"
|
||||
>
|
||||
{currentInstrumentDef.ranges.map(r => (
|
||||
<option key={r.id} value={r.id}>{r.label}</option>
|
||||
))}
|
||||
</select>
|
||||
</div>
|
||||
|
||||
{/* Row 2, Col 2 */}
|
||||
{!inPhraseMode && (
|
||||
<div className="control-group">
|
||||
@@ -157,42 +143,11 @@ export const Controls: React.FC<ControlsProps> = ({ currentPitch }) => {
|
||||
</div>
|
||||
)}
|
||||
|
||||
{/* Row 3: Fret Range (Full Width) */}
|
||||
{currentInstrumentDef.ranges.find(r => r.id === settings.difficulty)?.type === 'custom_fret' && (
|
||||
<div className="control-group" style={{ gridColumn: '1 / -1' }}>
|
||||
<label className="control-label">
|
||||
<span>Fret Range</span>
|
||||
</label>
|
||||
<div style={{ display: 'flex', gap: '8px', width: '100%' }}>
|
||||
<div style={{ flex: 1, display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<span style={{ fontSize: '12px', opacity: 0.7 }}>Min</span>
|
||||
<input
|
||||
type="number"
|
||||
min="0"
|
||||
max="24"
|
||||
value={settings.customMinFret ?? 0}
|
||||
onChange={(e) => onUpdateSettings({ ...settings, customMinFret: parseInt(e.target.value) || 0 })}
|
||||
className="control-input"
|
||||
style={{ width: '100%', padding: '4px' }}
|
||||
/>
|
||||
</div>
|
||||
<div style={{ flex: 1, display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<span style={{ fontSize: '12px', opacity: 0.7 }}>Max</span>
|
||||
<input
|
||||
type="number"
|
||||
min="0"
|
||||
max="24"
|
||||
value={settings.customMaxFret ?? 12}
|
||||
onChange={(e) => onUpdateSettings({ ...settings, customMaxFret: parseInt(e.target.value) || 0 })}
|
||||
className="control-input"
|
||||
style={{ width: '100%', padding: '4px' }}
|
||||
/>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
|
||||
{/* Target-note filter: unified controls, all modes */}
|
||||
<TargetNoteControls />
|
||||
|
||||
{/* Bottom Section: Tools Grid (2 Columns now) */}
|
||||
<div className="tools-grid" style={inPhraseMode ? { gridTemplateColumns: '1fr' } : undefined}>
|
||||
{/* Tool 1: Tuner */}
|
||||
@@ -220,121 +175,17 @@ export const Controls: React.FC<ControlsProps> = ({ currentPitch }) => {
|
||||
)}
|
||||
</div>
|
||||
|
||||
{/* Tool 2: Metronome (single-note modes only) */}
|
||||
{!inPhraseMode && (
|
||||
{/* Tool 2: Metronome (all modes) */}
|
||||
<div style={{ display: 'flex', flexDirection: 'column', gap: '8px', border: '1px solid rgba(128,128,128,0.2)', padding: '12px', borderRadius: '8px' }}>
|
||||
<div style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between' }}>
|
||||
<label className="control-label" style={{ marginBottom: 0 }}>
|
||||
<span>Metronome</span>
|
||||
</label>
|
||||
<button
|
||||
className={`switch-button ${settings.rhythm.active ? 'active' : ''}`}
|
||||
onClick={() => updateRhythm({ active: !settings.rhythm.active })}
|
||||
title={settings.rhythm.active ? "Turn Off" : "Turn On"}
|
||||
>
|
||||
<div className="switch-thumb" />
|
||||
</button>
|
||||
</div>
|
||||
|
||||
{settings.rhythm.active && (
|
||||
<div className="rhythm-details" style={{ display: 'flex', flexDirection: 'column', gap: '6px', paddingTop: '8px', borderTop: '1px solid rgba(255,255,255,0.1)' }}>
|
||||
<div style={{ display: 'flex', gap: '4px' }}>
|
||||
<button
|
||||
className={`control-button small ${settings.rhythm.mode === 'bpm' ? 'active' : ''}`}
|
||||
onClick={() => updateRhythm({ mode: 'bpm' })}
|
||||
style={{ flex: 1, fontSize: '10px' }}
|
||||
>BPM</button>
|
||||
<button
|
||||
className={`control-button small ${settings.rhythm.mode === 'seconds' ? 'active' : ''}`}
|
||||
onClick={() => updateRhythm({ mode: 'seconds' })}
|
||||
style={{ flex: 1, fontSize: '10px' }}
|
||||
>Timer</button>
|
||||
</div>
|
||||
|
||||
{settings.rhythm.mode === 'bpm' ? (
|
||||
<div style={{ display: 'flex', flexDirection: 'column', gap: '4px' }}>
|
||||
<div style={{ display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<button
|
||||
className="control-button small"
|
||||
onClick={() => updateRhythm({ bpm: Math.max(30, settings.rhythm.bpm - 1) })}
|
||||
style={{ width: '24px', padding: '0', display: 'flex', alignItems: 'center', justifyContent: 'center' }}
|
||||
>
|
||||
-
|
||||
</button>
|
||||
<input
|
||||
type="number"
|
||||
min="30"
|
||||
max="300"
|
||||
value={settings.rhythm.bpm}
|
||||
onChange={(e) => updateRhythm({ bpm: Math.max(1, parseInt(e.target.value) || 60) })}
|
||||
className="control-input"
|
||||
style={{
|
||||
width: '48px',
|
||||
textAlign: 'center',
|
||||
padding: '2px'
|
||||
}}
|
||||
/>
|
||||
<button
|
||||
className="control-button small"
|
||||
onClick={() => updateRhythm({ bpm: Math.min(300, settings.rhythm.bpm + 1) })}
|
||||
style={{ width: '24px', padding: '0', display: 'flex', alignItems: 'center', justifyContent: 'center' }}
|
||||
>
|
||||
+
|
||||
</button>
|
||||
<input
|
||||
type="range"
|
||||
min="30"
|
||||
max="240"
|
||||
step="1"
|
||||
value={settings.rhythm.bpm}
|
||||
onChange={(e) => updateRhythm({ bpm: Number(e.target.value) })}
|
||||
style={{ flex: 1 }}
|
||||
/>
|
||||
</div>
|
||||
<div style={{ fontSize: '11px', opacity: 0.7, textAlign: 'center' }}>
|
||||
{getTempoMarking(settings.rhythm.bpm)}
|
||||
</div>
|
||||
</div>
|
||||
) : (
|
||||
<div style={{ display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<span style={{ fontSize: '12px', minWidth: '32px' }}>{settings.rhythm.seconds}s</span>
|
||||
<input
|
||||
type="range"
|
||||
min="1"
|
||||
max="60"
|
||||
step="1"
|
||||
value={settings.rhythm.seconds}
|
||||
onChange={(e) => updateRhythm({ seconds: Number(e.target.value) })}
|
||||
style={{ flex: 1 }}
|
||||
/>
|
||||
</div>
|
||||
)}
|
||||
|
||||
<div style={{ display: 'flex', justifyContent: 'space-between', alignItems: 'center' }}>
|
||||
<label style={{ fontSize: '10px', display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={settings.rhythm.autoAdvance}
|
||||
onChange={(e) => updateRhythm({ autoAdvance: e.target.checked })}
|
||||
/>
|
||||
Auto
|
||||
</label>
|
||||
|
||||
<label style={{ fontSize: '10px', display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={settings.rhythm.sound}
|
||||
onChange={(e) => updateRhythm({ sound: e.target.checked })}
|
||||
/>
|
||||
Sound
|
||||
</label>
|
||||
</div>
|
||||
</div>
|
||||
)}
|
||||
<MetronomeWidget
|
||||
rhythm={settings.rhythm}
|
||||
onUpdate={updateRhythm}
|
||||
showAutoAdvance={!inPhraseMode}
|
||||
syncMode={inPhraseMode ? 'option' : null}
|
||||
gate={inPhraseMode ? phraseRunActive : null}
|
||||
/>
|
||||
</div>
|
||||
|
||||
)}
|
||||
|
||||
</div>
|
||||
</div>
|
||||
);
|
||||
|
||||
@@ -14,10 +14,12 @@ import { Renderer, Stave, StaveNote, Accidental, Voice, Formatter, Annotation }
|
||||
import { getNoteInKey } from '../music/NoteUtils';
|
||||
|
||||
interface LiveNoteStaffProps {
|
||||
/** Sounding midi of the live detected pitch, or null when silent. */
|
||||
midi: number | null;
|
||||
transpose: number;
|
||||
keySignature: string;
|
||||
/** Force a clef (e.g. the instrument's usual clef). When omitted the
|
||||
* clef auto-switches treble/bass around middle C (piano-style). */
|
||||
clef?: 'treble' | 'bass';
|
||||
width?: number;
|
||||
theme?: 'light' | 'dark' | 'auto';
|
||||
}
|
||||
@@ -32,6 +34,7 @@ export const LiveNoteStaff: React.FC<LiveNoteStaffProps> = ({
|
||||
midi,
|
||||
transpose = 0,
|
||||
keySignature = 'C',
|
||||
clef: clefOverride,
|
||||
width = 130,
|
||||
theme = 'auto',
|
||||
}) => {
|
||||
@@ -63,8 +66,9 @@ export const LiveNoteStaff: React.FC<LiveNoteStaffProps> = ({
|
||||
const stave = new Stave(0, STAVE_Y, width - 2);
|
||||
stave.setDefaultLedgerLineStyle({ strokeStyle: resolvedColor, lineWidth: 2 });
|
||||
const written0 = midi !== null ? midi + transpose : null;
|
||||
// Clef like the grand-staff split: below middle C reads bass clef.
|
||||
const clef: 'treble' | 'bass' = written0 !== null && written0 < 60 ? 'bass' : 'treble';
|
||||
// Instrument-fixed clef when provided (guitar always treble, bass
|
||||
// always bass); otherwise piano-style split at middle C.
|
||||
const clef: 'treble' | 'bass' = clefOverride ?? (written0 !== null && written0 < 60 ? 'bass' : 'treble');
|
||||
stave.addClef(clef);
|
||||
if (keySignature) stave.addKeySignature(keySignature);
|
||||
stave.setContext(context).draw();
|
||||
@@ -109,7 +113,7 @@ export const LiveNoteStaff: React.FC<LiveNoteStaffProps> = ({
|
||||
new Formatter().joinVoices([voice]).format([voice], width - 40);
|
||||
voice.draw(context, stave);
|
||||
}
|
||||
}, [midi, transpose, keySignature, width, theme, schemeVersion]);
|
||||
}, [midi, transpose, keySignature, clefOverride, width, theme, schemeVersion]);
|
||||
|
||||
return (
|
||||
<div
|
||||
|
||||
232
src/components/MetronomeWidget.tsx
Normal file
232
src/components/MetronomeWidget.tsx
Normal file
@@ -0,0 +1,232 @@
|
||||
/**
|
||||
* Metronome widget — always-visible tempo controls + pendulum.
|
||||
*
|
||||
* Modes:
|
||||
* - Plain (single-note modes): one on/off toggle arms sound + animation.
|
||||
* - Sync option (Phrase Mode): the toggle switches between
|
||||
* * FREE — the metronome runs independently (own clock, starts as soon
|
||||
* as toggled, with or without clicks per the sound checkbox)
|
||||
* * SYNC — the metronome starts with the exercise (count-in included),
|
||||
* is silent while idle, and its pendulum mirrors the run's beats.
|
||||
*
|
||||
* Exactly ONE sound checkbox exists (here). In sync mode it governs the
|
||||
* run's click scheduler too — clicks can never double.
|
||||
*/
|
||||
import { useEffect, useRef, useState, useCallback } from 'react';
|
||||
import { useMetronome } from '../hooks/useMetronome';
|
||||
import { getTempoMarking } from '../music/TempoMarkings';
|
||||
import { phraseBeatBus } from '../hooks/phraseBeatBus';
|
||||
import { audioEngine } from '../audio/AudioEngine';
|
||||
import type { RhythmSettings } from '../types/SettingsTypes';
|
||||
|
||||
interface MetronomeWidgetProps {
|
||||
rhythm: RhythmSettings;
|
||||
/** Persist changes. */
|
||||
onUpdate: (updates: Partial<RhythmSettings>) => void;
|
||||
/** Single-note mode extra: auto-advance targets on each tick. */
|
||||
showAutoAdvance?: boolean;
|
||||
/** Compact layout (phrase transport row). */
|
||||
compact?: boolean;
|
||||
/** 'option': show the sync/free toggle (Phrase Mode). null: plain on/off. */
|
||||
syncMode?: 'option' | null;
|
||||
/** External gate (sync mode): true while the exercise runs. */
|
||||
gate?: boolean | null;
|
||||
}
|
||||
|
||||
export const MetronomeWidget: React.FC<MetronomeWidgetProps> = ({
|
||||
rhythm,
|
||||
onUpdate,
|
||||
showAutoAdvance = false,
|
||||
compact = false,
|
||||
syncMode = null,
|
||||
gate = null,
|
||||
}) => {
|
||||
const [beatParity, setBeatParity] = useState(0);
|
||||
const lastTickRef = useRef(0);
|
||||
const handleTick = useCallback((beat: number) => {
|
||||
const now = performance.now();
|
||||
if (now - lastTickRef.current < 60) return; // glitch guard
|
||||
lastTickRef.current = now;
|
||||
setBeatParity(beat % 2);
|
||||
}, []);
|
||||
|
||||
const isSync = syncMode === 'option' && !!rhythm.syncToExercise;
|
||||
const runActive = gate === true;
|
||||
// Sync: pendulum/click only while the run is active (the run's scheduler
|
||||
// is the click source; the widget's own scheduler stays off).
|
||||
// Free/plain: the widget's own scheduler runs whenever armed.
|
||||
const running = isSync ? runActive : rhythm.active;
|
||||
|
||||
const { restart } = useMetronome({
|
||||
bpm: rhythm.bpm,
|
||||
volume: rhythm.sound ? rhythm.volume : 0,
|
||||
playing: running && !isSync,
|
||||
onTick: handleTick,
|
||||
});
|
||||
void restart;
|
||||
|
||||
// Sync: pendulum mirrors the phrase run's beats (audio-time scheduled).
|
||||
useEffect(() => {
|
||||
if (!isSync || !runActive) return;
|
||||
return phraseBeatBus.subscribe((_beat, at) => {
|
||||
const delay = Math.max(0, (at - audioEngine.now()) * 1000);
|
||||
window.setTimeout(() => setBeatParity(p => (p + 1) % 2), delay);
|
||||
});
|
||||
}, [isSync, runActive]);
|
||||
|
||||
const beatMs = 60000 / Math.max(1, rhythm.bpm);
|
||||
const armed = rhythm.active;
|
||||
|
||||
return (
|
||||
<div style={{ display: 'flex', flexDirection: 'column', gap: '8px' }}>
|
||||
{/* Header: label + mode toggle */}
|
||||
<div style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between' }}>
|
||||
<label className="control-label" style={{ marginBottom: 0 }}>
|
||||
<span>Metronome</span>
|
||||
</label>
|
||||
{syncMode === 'option' ? (
|
||||
<button
|
||||
className={`switch-button ${rhythm.syncToExercise ? 'active' : ''}`}
|
||||
onClick={() => onUpdate({
|
||||
syncToExercise: !rhythm.syncToExercise,
|
||||
active: true, // armed in either mode; the mode decides behavior
|
||||
})}
|
||||
title={rhythm.syncToExercise
|
||||
? 'Sync: starts with the exercise (count-in). Click while idle: off.'
|
||||
: 'Free: runs independently — start/stop it yourself.'}
|
||||
>
|
||||
<div className="switch-thumb" />
|
||||
</button>
|
||||
) : (
|
||||
<button
|
||||
className={`switch-button ${rhythm.active ? 'active' : ''}`}
|
||||
onClick={() => onUpdate({ active: !rhythm.active })}
|
||||
title={rhythm.active ? 'Turn Off' : 'Turn On (sound + pendulum)'}
|
||||
>
|
||||
<div className="switch-thumb" />
|
||||
</button>
|
||||
)}
|
||||
</div>
|
||||
|
||||
{/* Always visible body; dimmed when disarmed */}
|
||||
<div style={{ display: 'flex', flexDirection: 'column', gap: '6px', paddingTop: '8px', borderTop: '1px solid rgba(255,255,255,0.1)', opacity: armed ? 1 : 0.5 }}>
|
||||
{/* Pendulum */}
|
||||
<div
|
||||
style={{
|
||||
position: 'relative',
|
||||
height: compact ? 14 : 18,
|
||||
borderRadius: 9,
|
||||
background: 'color-mix(in srgb, var(--color-text-main) 8%, transparent)',
|
||||
overflow: 'hidden',
|
||||
}}
|
||||
aria-hidden
|
||||
>
|
||||
<div
|
||||
style={{
|
||||
position: 'absolute',
|
||||
top: '50%',
|
||||
left: beatParity === 0 ? '6%' : '88%',
|
||||
width: compact ? 10 : 12,
|
||||
height: compact ? 10 : 12,
|
||||
borderRadius: '50%',
|
||||
background: 'var(--color-text-main)',
|
||||
transform: 'translateY(-50%)',
|
||||
transition: `left ${beatMs}ms linear`,
|
||||
display: armed && running ? 'block' : 'none',
|
||||
}}
|
||||
/>
|
||||
</div>
|
||||
|
||||
{/* BPM / Timer controls */}
|
||||
<div style={{ display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
{!compact && (
|
||||
<div style={{ display: 'flex', gap: '4px', flexShrink: 0 }}>
|
||||
<button
|
||||
className={`control-button small ${rhythm.mode === 'bpm' ? 'active' : ''}`}
|
||||
onClick={() => onUpdate({ mode: 'bpm' })}
|
||||
style={{ fontSize: '10px' }}
|
||||
>BPM</button>
|
||||
<button
|
||||
className={`control-button small ${rhythm.mode === 'seconds' ? 'active' : ''}`}
|
||||
onClick={() => onUpdate({ mode: 'seconds' })}
|
||||
style={{ fontSize: '10px' }}
|
||||
>Timer</button>
|
||||
</div>
|
||||
)}
|
||||
{rhythm.mode === 'bpm' ? (
|
||||
<>
|
||||
<button
|
||||
className="control-button small"
|
||||
onClick={() => onUpdate({ bpm: Math.max(30, rhythm.bpm - 1) })}
|
||||
style={{ width: '24px', padding: '0', display: 'flex', alignItems: 'center', justifyContent: 'center' }}
|
||||
>−</button>
|
||||
<input
|
||||
type="number"
|
||||
min="30"
|
||||
max="300"
|
||||
value={rhythm.bpm}
|
||||
onChange={(e) => onUpdate({ bpm: Math.max(1, parseInt(e.target.value) || 60) })}
|
||||
className="control-input"
|
||||
style={{ width: '48px', textAlign: 'center', padding: '2px' }}
|
||||
/>
|
||||
<button
|
||||
className="control-button small"
|
||||
onClick={() => onUpdate({ bpm: Math.min(300, rhythm.bpm + 1) })}
|
||||
style={{ width: '24px', padding: '0', display: 'flex', alignItems: 'center', justifyContent: 'center' }}
|
||||
>+</button>
|
||||
<input
|
||||
type="range"
|
||||
min="30"
|
||||
max="240"
|
||||
step="1"
|
||||
value={rhythm.bpm}
|
||||
onChange={(e) => onUpdate({ bpm: Number(e.target.value) })}
|
||||
style={{ flex: 1 }}
|
||||
/>
|
||||
</>
|
||||
) : (
|
||||
<>
|
||||
<span style={{ fontSize: '12px', minWidth: '32px' }}>{rhythm.seconds}s</span>
|
||||
<input
|
||||
type="range"
|
||||
min="1"
|
||||
max="60"
|
||||
step="1"
|
||||
value={rhythm.seconds}
|
||||
onChange={(e) => onUpdate({ seconds: Number(e.target.value) })}
|
||||
style={{ flex: 1 }}
|
||||
/>
|
||||
</>
|
||||
)}
|
||||
</div>
|
||||
{rhythm.mode === 'bpm' && (
|
||||
<div style={{ fontSize: '11px', opacity: 0.7, textAlign: 'center' }}>
|
||||
{getTempoMarking(rhythm.bpm)}
|
||||
</div>
|
||||
)}
|
||||
|
||||
{/* Options row */}
|
||||
<div style={{ display: 'flex', justifyContent: 'space-between', alignItems: 'center' }}>
|
||||
{showAutoAdvance ? (
|
||||
<label style={{ fontSize: '10px', display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={rhythm.autoAdvance}
|
||||
onChange={(e) => onUpdate({ autoAdvance: e.target.checked })}
|
||||
/>
|
||||
Auto
|
||||
</label>
|
||||
) : <span />}
|
||||
<label style={{ fontSize: '10px', display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={rhythm.sound}
|
||||
onChange={(e) => onUpdate({ sound: e.target.checked })}
|
||||
/>
|
||||
Click sound
|
||||
</label>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
);
|
||||
};
|
||||
@@ -13,7 +13,7 @@
|
||||
* microphone frame rate.
|
||||
*/
|
||||
import React, { useEffect, useRef, useState } from 'react';
|
||||
import { Renderer, Stave, StaveNote, GhostNote, Accidental, Voice, Formatter, StaveConnector, Beam, StaveTie, Annotation } from 'vexflow';
|
||||
import { Renderer, Stave, StaveNote, GhostNote, Accidental, Voice, Formatter, StaveConnector, Beam, StaveTie, Annotation, Dot } from 'vexflow';
|
||||
import { scoreEvents, type Score } from '../score/model';
|
||||
import { STEP_LETTERS, type StepLetter } from '../score/model';
|
||||
import type { NoteStatus } from '../game/PhraseMatcher';
|
||||
@@ -40,6 +40,7 @@ interface Fragment {
|
||||
ticks: number;
|
||||
isAttack: boolean;
|
||||
isEighth: boolean;
|
||||
isSixteenth: boolean;
|
||||
staveKey: 'treble' | 'bass';
|
||||
rest: boolean;
|
||||
}
|
||||
@@ -47,7 +48,9 @@ interface Fragment {
|
||||
// Duration ticks → VexFlow duration code (subset the model guarantees).
|
||||
function durationCode(ticks: number): string | null {
|
||||
switch (ticks) {
|
||||
case 120: return '16';
|
||||
case 240: return '8';
|
||||
case 360: return '8d';
|
||||
case 480: return '4';
|
||||
case 720: return '4d';
|
||||
case 960: return '2';
|
||||
@@ -162,6 +165,11 @@ export const PhraseSheetMusic: React.FC<PhraseSheetMusicProps> = ({
|
||||
const ticks = fragEnd - cursor;
|
||||
const code = durationCode(ticks);
|
||||
if (!code) { cursor = fragEnd; continue; }
|
||||
// 'd' suffix encodes a dotted duration; the visible dot is
|
||||
// a modifier that must be added explicitly (VexFlow only
|
||||
// reads the base duration for spacing otherwise).
|
||||
const isDotted = code.endsWith('d');
|
||||
const baseCode = isDotted ? code.slice(0, -1) : code;
|
||||
|
||||
const rest = ev.pitch === null;
|
||||
const staveKey: 'treble' | 'bass' = rest
|
||||
@@ -171,7 +179,7 @@ export const PhraseSheetMusic: React.FC<PhraseSheetMusicProps> = ({
|
||||
|
||||
let note: StaveNote;
|
||||
if (rest) {
|
||||
note = new StaveNote({ keys: ['b/4'], duration: `${code}r`, clef: staveKey });
|
||||
note = new StaveNote({ keys: ['b/4'], duration: `${baseCode}r`, clef: staveKey });
|
||||
} else {
|
||||
const p = ev.pitch!;
|
||||
// WRITTEN pitch for the staff: sounding midi + display
|
||||
@@ -192,9 +200,10 @@ export const PhraseSheetMusic: React.FC<PhraseSheetMusicProps> = ({
|
||||
measureState[stateKey] = written.alter;
|
||||
note = new StaveNote({
|
||||
keys: [keyString(written.step, written.alter, written.octave)],
|
||||
duration: code,
|
||||
duration: baseCode,
|
||||
clef: staveKey,
|
||||
});
|
||||
if (isDotted) Dot.buildAndAttach([note]);
|
||||
if (glyph) note.addModifier(new Accidental(glyph));
|
||||
}
|
||||
|
||||
@@ -226,6 +235,7 @@ export const PhraseSheetMusic: React.FC<PhraseSheetMusicProps> = ({
|
||||
ticks,
|
||||
isAttack,
|
||||
isEighth: code === '8' && !rest,
|
||||
isSixteenth: code === '16' && !rest,
|
||||
staveKey,
|
||||
rest,
|
||||
});
|
||||
@@ -262,6 +272,14 @@ export const PhraseSheetMusic: React.FC<PhraseSheetMusicProps> = ({
|
||||
if (r === 0) treble.addTimeSignature(`${score.meter.numerator}/${score.meter.denominator}`);
|
||||
treble.setMeasure(measure.number);
|
||||
}
|
||||
// Chord symbol above the staff (bar-indexed), drawn via the
|
||||
// stave's annotation area so it shifts with the clef offset.
|
||||
const chordSym = score.chordSymbols?.[m];
|
||||
if (chordSym) {
|
||||
try {
|
||||
treble.setSection(chordSym, 0);
|
||||
} catch { /* courtesy label — never fatal */ }
|
||||
}
|
||||
let bass: Stave | null = null;
|
||||
if (isGrand) {
|
||||
bass = new Stave(x, y + 130, barWidth);
|
||||
@@ -319,16 +337,24 @@ export const PhraseSheetMusic: React.FC<PhraseSheetMusicProps> = ({
|
||||
line.setContext(context).draw();
|
||||
}
|
||||
|
||||
// --- Beams: consecutive eighth pairs within one beat ---
|
||||
const eighths = frags.filter(f => f.isEighth && f.note instanceof StaveNote);
|
||||
for (let i = 0; i + 1 < eighths.length; i++) {
|
||||
const a = eighths[i], b2 = eighths[i + 1];
|
||||
// --- Beams: consecutive eighth/16th pairs within one beat ---
|
||||
// Note: the Beam must be registered on each note via setBeam,
|
||||
// otherwise VexFlow still draws flags (beam === undefined in
|
||||
// shouldDrawFlag) and you get BOTH flag and beam.
|
||||
const beamed = frags.filter(f => (f.isEighth || f.isSixteenth) && f.note instanceof StaveNote);
|
||||
for (let i = 0; i + 1 < beamed.length; i++) {
|
||||
const a = beamed[i], b2 = beamed[i + 1];
|
||||
if (a.staveKey !== b2.staveKey) continue;
|
||||
if (b2.start - a.start !== 240) continue;
|
||||
// consecutive within the same beat, same subdivision size
|
||||
const step = a.isSixteenth ? 120 : 240;
|
||||
if (b2.start - a.start !== step) continue;
|
||||
if ((a.ticks !== step) || (b2.ticks !== step)) continue;
|
||||
const beatA = Math.floor((a.start - measureStart) / 480);
|
||||
const beatB = Math.floor((b2.start - measureStart) / 480);
|
||||
if (beatA === beatB) {
|
||||
const beam = new Beam([a.note, b2.note], false);
|
||||
a.note.setBeam(beam);
|
||||
b2.note.setBeam(beam);
|
||||
beam.setContext(context).draw();
|
||||
}
|
||||
}
|
||||
|
||||
@@ -13,13 +13,15 @@ import { forwardRef, useCallback, useEffect, useImperativeHandle, useMemo, useRe
|
||||
import { useSettings } from '../context/useSettings';
|
||||
import { useAudioPlayer } from '../hooks/useAudioPlayer';
|
||||
import { usePhraseTrainer, type PhraseTrainerApi, type PhrasePhase } from '../hooks/usePhraseTrainer';
|
||||
import { getPhraseSettings, type PhraseSettings } from '../types/SettingsTypes';
|
||||
import { getPhraseSettings, getPracticeFilter, type PhraseSettings, type PracticeFilter } from '../types/SettingsTypes';
|
||||
import { rawFrameCount } from '../hooks/rawFrameBus';
|
||||
import { audioEngine } from '../audio/AudioEngine';
|
||||
import { computePlayableNotes, isFrettedInstrument, positionsWithinWindow, fretWindowNotes } from '../music/playableRange';
|
||||
import { computePlayableNotes, isFrettedInstrument, positionsWithinWindow } from '../music/playableRange';
|
||||
import { generateMelody } from '../music/melodyGenerator';
|
||||
import { generateScaleDrill } from '../music/scaleDrills';
|
||||
import { isMode, MODE_LABELS, TONICS, type ModeId } from '../music/scales';
|
||||
import { generateArpeggio, type ArpeggioDegree } from '../music/arpeggioGenerator';
|
||||
import { isMode, type ModeId } from '../music/scales';
|
||||
import { ARPEGGIO_DEGREE_LABELS } from '../music/arpeggioGenerator';
|
||||
import { scoreEvents, type Result, type Score, type ScoreEvent } from '../score/model';
|
||||
import { getNoteDetails } from '../music/NoteUtils';
|
||||
import { EXERCISES, loadExercise } from '../exercises/library';
|
||||
@@ -68,18 +70,64 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
const setSettings = updateSettings;
|
||||
const { playNote } = useAudioPlayer();
|
||||
const phrase = getPhraseSettings(settings);
|
||||
// Unified target-note controls: Phrase Mode reads/writes the shared
|
||||
// practice filter so all modes expose identical key/fret-window state.
|
||||
const pf = getPracticeFilter(settings);
|
||||
phrase.keyTonic = pf.keyTonic;
|
||||
phrase.keyMode = pf.keyMode;
|
||||
phrase.fretWindowEnabled = pf.fretWindowEnabled;
|
||||
phrase.fretMin = pf.fretMin;
|
||||
phrase.fretMax = pf.fretMax;
|
||||
// Unified metronome: phrase tempo reads from the shared rhythm BPM
|
||||
// (single source of truth across modes; the phrase-local BPM control
|
||||
// was replaced by the common metronome widget in Controls).
|
||||
phrase.bpm = settings.rhythm.bpm;
|
||||
// Click decision — ONE source: the metronome widget's sound checkbox.
|
||||
// - Tempo + syncToExercise: the run clicks when armed && sound
|
||||
// (silent while idle; starts with the exercise incl. count-in).
|
||||
// - Tempo + free metronome: the widget clicks freely; the run stays
|
||||
// silent (its beats are visual only) to avoid doubling.
|
||||
// - Step: widget clicks freely; phrase clickSound follows the switch.
|
||||
const sync = settings.rhythm.syncToExercise;
|
||||
phrase.clickSound = phrase.pace === 'tempo'
|
||||
? (sync ? settings.rhythm.active && settings.rhythm.sound : false)
|
||||
: settings.rhythm.active && settings.rhythm.sound;
|
||||
const setPhrase = useCallback((updates: Partial<PhraseSettings>) => {
|
||||
updateSettings(s => ({ ...s, phrase: { ...getPhraseSettings(s), ...updates } }));
|
||||
updateSettings(s => {
|
||||
const next = { ...s, phrase: { ...getPhraseSettings(s), ...updates } };
|
||||
// Mirror shared filter fields into the unified practice filter.
|
||||
const shared: Partial<PracticeFilter> = {};
|
||||
if ('keyTonic' in updates) shared.keyTonic = updates.keyTonic;
|
||||
if ('keyMode' in updates) shared.keyMode = updates.keyMode;
|
||||
if ('fretWindowEnabled' in updates) shared.fretWindowEnabled = updates.fretWindowEnabled;
|
||||
if ('fretMin' in updates) shared.fretMin = updates.fretMin;
|
||||
if ('fretMax' in updates) shared.fretMax = updates.fretMax;
|
||||
if (Object.keys(shared).length > 0) next.practice = { ...getPracticeFilter(s), ...shared };
|
||||
return next;
|
||||
});
|
||||
}, [updateSettings]);
|
||||
|
||||
const currentInstrumentDef = INSTRUMENT_DEFINITIONS[settings.instrument];
|
||||
const currentTuning = TUNINGS[settings.tuningId];
|
||||
const fretted = isFrettedInstrument(settings.instrument) && !!currentTuning;
|
||||
|
||||
// ---- Playable pool ----------------------------------------------------
|
||||
const pool = useMemo(() => {
|
||||
if (phrase.fretWindowEnabled && fretted && currentTuning) {
|
||||
return fretWindowNotes(currentTuning, phrase.fretMin, phrase.fretMax);
|
||||
// Unified filter: strings ∩ fret window, computed identically to
|
||||
// the single-note modes (full 0–12 fretboard when no filter is
|
||||
// active — the legacy difficulty ranges no longer apply).
|
||||
if (fretted && currentTuning) {
|
||||
const all = currentTuning.strings;
|
||||
const selected = pf.strings.length > 0
|
||||
? all.filter((_, i) => pf.strings.includes(i))
|
||||
: all;
|
||||
if (selected.length === 0) return [];
|
||||
const lo = pf.fretWindowEnabled ? Math.max(0, Math.min(24, pf.fretMin)) : 0;
|
||||
const hi = pf.fretWindowEnabled ? Math.max(lo, Math.min(24, pf.fretMax)) : 12;
|
||||
return selected.flatMap(open => {
|
||||
const notes: number[] = [];
|
||||
for (let fret = lo; fret <= hi; fret++) notes.push(open + fret);
|
||||
return notes;
|
||||
}).sort((a, b) => a - b);
|
||||
}
|
||||
return computePlayableNotes({
|
||||
instrumentId: settings.instrument,
|
||||
@@ -88,7 +136,8 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
customMinFret: settings.customMinFret,
|
||||
customMaxFret: settings.customMaxFret,
|
||||
});
|
||||
}, [phrase.fretWindowEnabled, phrase.fretMin, phrase.fretMax, fretted, currentTuning,
|
||||
}, [pf.strings, pf.fretWindowEnabled, pf.fretMin, pf.fretMax, fretted, currentTuning,
|
||||
phrase.fretWindowEnabled, phrase.fretMin, phrase.fretMax,
|
||||
settings.instrument, settings.difficulty, settings.tuningId, settings.customMinFret, settings.customMaxFret]);
|
||||
|
||||
// ---- Imported score (session-only) --------------------------------------
|
||||
@@ -98,20 +147,6 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
const [melodySeed, setMelodySeed] = useState(() => Math.floor(Math.random() * 100000));
|
||||
const fileInputRef = useRef<HTMLInputElement>(null);
|
||||
const pendingAutoStart = useRef(false);
|
||||
// BPM text field: editing stays free-form while typing; commits on
|
||||
// blur/Enter (single-note widget behavior).
|
||||
const [bpmText, setBpmText] = useState(String(phrase.bpm));
|
||||
const [lastBpm, setLastBpm] = useState(phrase.bpm);
|
||||
if (phrase.bpm !== lastBpm) {
|
||||
// React's "adjust state when a prop changes" pattern (no effect).
|
||||
setLastBpm(phrase.bpm);
|
||||
setBpmText(String(phrase.bpm));
|
||||
}
|
||||
const commitBpm = useCallback(() => {
|
||||
const v = parseInt(bpmText, 10);
|
||||
if (Number.isNaN(v)) { setBpmText(String(phrase.bpm)); return; }
|
||||
setPhrase({ bpm: Math.max(30, Math.min(180, v)) });
|
||||
}, [bpmText, phrase.bpm, setPhrase]);
|
||||
|
||||
const handleImportFile = useCallback((file: File) => {
|
||||
setImportError(null);
|
||||
@@ -151,6 +186,22 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
direction: phrase.scaleDirection,
|
||||
coverage: phrase.scaleCoverage,
|
||||
meter: { numerator: phrase.meterNumerator, denominator: 4 },
|
||||
rhythm: phrase.scaleRhythm,
|
||||
}, pool);
|
||||
}
|
||||
if (phrase.material === 'arpeggio') {
|
||||
return generateArpeggio({
|
||||
keyTonic: phrase.keyTonic,
|
||||
keyMode: (isMode(phrase.keyMode) ? phrase.keyMode : 'major') as ModeId,
|
||||
degree: (phrase.arpeggioDegree as ArpeggioDegree) || 'I',
|
||||
pattern: phrase.arpeggioPattern,
|
||||
coverage: phrase.arpeggioCoverage,
|
||||
meter: { numerator: phrase.meterNumerator, denominator: 4 },
|
||||
bars: phrase.arpeggioBars,
|
||||
progression: phrase.arpeggioProgression,
|
||||
chordSelection: phrase.arpeggioChordSelection as ArpeggioDegree[],
|
||||
rhythm: phrase.arpeggioRhythm,
|
||||
seed: melodySeed,
|
||||
}, pool);
|
||||
}
|
||||
if (phrase.material === 'library') {
|
||||
@@ -164,7 +215,10 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
}
|
||||
return null;
|
||||
}, [phrase.material, phrase.keyTonic, phrase.keyMode, phrase.bars, phrase.meterNumerator,
|
||||
phrase.rhythmLevel, phrase.scaleDirection, phrase.scaleCoverage, phrase.libraryId,
|
||||
phrase.rhythmLevel, phrase.scaleDirection, phrase.scaleCoverage, phrase.scaleRhythm, phrase.libraryId,
|
||||
phrase.arpeggioDegree, phrase.arpeggioPattern, phrase.arpeggioCoverage,
|
||||
phrase.arpeggioBars, phrase.arpeggioProgression, phrase.arpeggioRhythm,
|
||||
phrase.arpeggioChordSelection, melodySeed,
|
||||
melodySeed, pool, importedScore]);
|
||||
|
||||
const fullScore: Result<Score> | null = useMemo(() => {
|
||||
@@ -192,6 +246,7 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
clickSound: phrase.clickSound,
|
||||
inputMode: phrase.inputMode,
|
||||
previewVolume: settings.autoPlayVolume ?? 0.4,
|
||||
autoStartOnNote: phrase.autoStartOnNote,
|
||||
},
|
||||
listening,
|
||||
micError,
|
||||
@@ -250,7 +305,7 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
}, [scoreOk, fixedMaterial, pool, fullScore]);
|
||||
|
||||
const nextAction = useCallback(() => {
|
||||
if (phrase.material === 'melody') {
|
||||
if (phrase.material === 'melody' || (phrase.material === 'arpeggio' && phrase.arpeggioBars > 1)) {
|
||||
setMelodySeed(s => (s + 1) % 999983);
|
||||
} else if (fixedMaterial && scoreOk) {
|
||||
const total = fullScore!.value.measures.length;
|
||||
@@ -400,6 +455,22 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
{rangeWarning && <div className="phrase-warning">{rangeWarning}</div>}
|
||||
|
||||
<div className="feedback-area">
|
||||
{trainer.phase === 'countIn' && (
|
||||
<span
|
||||
className="phrase-countin"
|
||||
style={{
|
||||
display: 'inline-block',
|
||||
minWidth: '20px',
|
||||
padding: '1px 7px',
|
||||
marginRight: '8px',
|
||||
borderRadius: '10px',
|
||||
fontSize: '11px',
|
||||
opacity: 0.75,
|
||||
background: 'color-mix(in srgb, var(--color-text-main) 10%, transparent)',
|
||||
}}
|
||||
aria-label={`Count-in: ${trainer.countInLeft} beats left`}
|
||||
>{trainer.countInLeft}</span>
|
||||
)}
|
||||
<div className="instruction-text phrase-status">{statusText}</div>
|
||||
</div>
|
||||
|
||||
@@ -423,7 +494,7 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
<RotateCcw size={18} />
|
||||
Retry
|
||||
</button>
|
||||
{(phrase.material === 'melody' || fixedMaterial) && (
|
||||
{(phrase.material === 'melody' || phrase.material === 'arpeggio' || fixedMaterial) && (
|
||||
<button className="hint-button" onClick={nextAction} title="New melody / next bars (N)">
|
||||
<SkipForward size={18} />
|
||||
{phrase.material === 'melody' ? 'New' : 'Next'}
|
||||
@@ -444,7 +515,7 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
<RotateCcw size={18} />
|
||||
Retry
|
||||
</button>
|
||||
{(phrase.material === 'melody' || fixedMaterial) && (
|
||||
{(phrase.material === 'melody' || phrase.material === 'arpeggio' || fixedMaterial) && (
|
||||
<button className="hint-button" onClick={nextAction} title="New / Next (N)">
|
||||
<SkipForward size={18} />
|
||||
{phrase.material === 'melody' ? 'New' : 'Next'}
|
||||
@@ -483,28 +554,16 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
<select value={phrase.material} onChange={e => setPhrase({ material: e.target.value as PhraseSettings['material'] })}>
|
||||
<option value="melody">Melodies (generated)</option>
|
||||
<option value="scale">Scale drills</option>
|
||||
<option value="arpeggio">Arpeggios (generated)</option>
|
||||
<option value="library">Exercises</option>
|
||||
<option value="import">Imported file</option>
|
||||
</select>
|
||||
</label>
|
||||
|
||||
{(phrase.material === 'melody' || phrase.material === 'scale') && (
|
||||
<>
|
||||
<label>Key
|
||||
<select value={phrase.keyTonic} onChange={e => setPhrase({ keyTonic: e.target.value })}>
|
||||
{Object.keys(TONICS).filter(t => !['C#', 'Gb', 'D#', 'G#', 'A#'].includes(t)).map(t => (
|
||||
<option key={t} value={t}>{t}</option>
|
||||
))}
|
||||
</select>
|
||||
</label>
|
||||
<label>Mode
|
||||
<select value={phrase.keyMode} onChange={e => setPhrase({ keyMode: e.target.value })}>
|
||||
{(Object.keys(MODE_LABELS) as ModeId[]).map(m => (
|
||||
<option key={m} value={m}>{MODE_LABELS[m]}</option>
|
||||
))}
|
||||
</select>
|
||||
</label>
|
||||
</>
|
||||
{(phrase.material === 'melody' || phrase.material === 'scale' || phrase.material === 'arpeggio') && (
|
||||
<span style={{ fontSize: '11px', opacity: 0.7, alignSelf: 'center' }}>
|
||||
Key/mode: use the "Target key" controls below the sheet — shared across modes.
|
||||
</span>
|
||||
)}
|
||||
|
||||
{phrase.material === 'melody' && (
|
||||
@@ -521,9 +580,10 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
</select>
|
||||
</label>
|
||||
<label>Rhythm
|
||||
<select value={phrase.rhythmLevel} onChange={e => setPhrase({ rhythmLevel: Number(e.target.value) === 2 ? 2 : 1 })}>
|
||||
<select value={phrase.rhythmLevel} onChange={e => setPhrase({ rhythmLevel: Math.max(1, Math.min(3, Number(e.target.value))) as 1 | 2 | 3 })}>
|
||||
<option value={1}>Simple</option>
|
||||
<option value={2}>Mixed</option>
|
||||
<option value={3}>Elaborate (16ths)</option>
|
||||
</select>
|
||||
</label>
|
||||
</>
|
||||
@@ -554,6 +614,98 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
</>
|
||||
)}
|
||||
|
||||
{phrase.material === 'arpeggio' && (
|
||||
<>
|
||||
<label>Chord
|
||||
<select value={phrase.arpeggioDegree} onChange={e => setPhrase({ arpeggioDegree: e.target.value, arpeggioBars: 1 })}>
|
||||
{(Object.keys(ARPEGGIO_DEGREE_LABELS) as ArpeggioDegree[]).map(d => (
|
||||
<option key={d} value={d}>{ARPEGGIO_DEGREE_LABELS[d]}</option>
|
||||
))}
|
||||
</select>
|
||||
</label>
|
||||
<label>Pattern
|
||||
<select value={phrase.arpeggioPattern} onChange={e => setPhrase({ arpeggioPattern: e.target.value as PhraseSettings['arpeggioPattern'] })}>
|
||||
<option value="up">Up</option>
|
||||
<option value="down">Down</option>
|
||||
<option value="updown">Up & down</option>
|
||||
<option value="1235">1-2-3-5</option>
|
||||
<optgroup label="Giuliani studies">
|
||||
<option value="giuliani-pim">p-i-m</option>
|
||||
<option value="giuliani-pima">p-i-m-a</option>
|
||||
<option value="giuliani-pami">p-a-m-i</option>
|
||||
<option value="giuliani-mami">m-a-m-i</option>
|
||||
<option value="giuliani-ami">a-m-i</option>
|
||||
<option value="giuliani-pimamim">p-i-m-a-m-i</option>
|
||||
<option value="giuliani-papima">p-a-p-i-m-a</option>
|
||||
<option value="giuliani-amipim">a-m-i-p-i-m</option>
|
||||
<option value="giuliani-miamim">m-i-a-m-i-m</option>
|
||||
</optgroup>
|
||||
</select>
|
||||
</label>
|
||||
<label>Coverage
|
||||
<select value={phrase.arpeggioCoverage} onChange={e => setPhrase({ arpeggioCoverage: e.target.value as PhraseSettings['arpeggioCoverage'] })}>
|
||||
<option value="one-octave">One octave</option>
|
||||
<option value="two-octave">Two octaves</option>
|
||||
</select>
|
||||
</label>
|
||||
<label>Length
|
||||
<select value={phrase.arpeggioBars} onChange={e => setPhrase({ arpeggioBars: Math.max(1, Math.min(8, Number(e.target.value))) })}>
|
||||
{[1, 2, 4, 6, 8].map(b => <option key={b} value={b}>{b === 1 ? 'Single chord' : `${b} bars`}</option>)}
|
||||
</select>
|
||||
</label>
|
||||
{phrase.arpeggioBars > 1 && (
|
||||
<label>Chords
|
||||
<select value={phrase.arpeggioProgression} onChange={e => setPhrase({ arpeggioProgression: e.target.value as PhraseSettings['arpeggioProgression'] })}>
|
||||
<option value="functional">Functional (I-IV-V...)</option>
|
||||
<option value="diatonic-cycle">Diatonic cycle</option>
|
||||
<option value="random">Random</option>
|
||||
</select>
|
||||
</label>
|
||||
)}
|
||||
{phrase.arpeggioBars > 1 && (
|
||||
<div style={{ gridColumn: '1 / -1' }}>
|
||||
<span style={{ fontSize: '11px', opacity: 0.7, display: 'block', marginBottom: '4px' }}>
|
||||
Chord pool ({phrase.arpeggioChordSelection.length === 0 ? 'all 7' : `${phrase.arpeggioChordSelection.length} selected`}):
|
||||
</span>
|
||||
<div style={{ display: 'flex', gap: '6px', flexWrap: 'wrap' }}>
|
||||
{(Object.keys(ARPEGGIO_DEGREE_LABELS) as ArpeggioDegree[]).map(d => {
|
||||
const active = phrase.arpeggioChordSelection.length === 0 || phrase.arpeggioChordSelection.includes(d);
|
||||
return (
|
||||
<button
|
||||
key={d}
|
||||
className={`control-button small ${active ? 'active' : ''}`}
|
||||
style={{ minWidth: '42px', padding: '5px 8px', fontSize: '12px' }}
|
||||
onClick={() => {
|
||||
// Toggle; empty selection = all back on.
|
||||
const cur = phrase.arpeggioChordSelection.length === 0
|
||||
? (Object.keys(ARPEGGIO_DEGREE_LABELS) as ArpeggioDegree[])
|
||||
: [...phrase.arpeggioChordSelection as ArpeggioDegree[]];
|
||||
const next = cur.includes(d) ? cur.filter(x => x !== d) : [...cur, d];
|
||||
setPhrase({ arpeggioChordSelection: next });
|
||||
}}
|
||||
>
|
||||
{ARPEGGIO_DEGREE_LABELS[d]}
|
||||
</button>
|
||||
);
|
||||
})}
|
||||
</div>
|
||||
</div>
|
||||
)}
|
||||
<label>Rhythm
|
||||
<select value={phrase.arpeggioRhythm} onChange={e => setPhrase({ arpeggioRhythm: e.target.value as PhraseSettings['arpeggioRhythm'] })}>
|
||||
<option value="quarters">Quarter notes</option>
|
||||
<option value="eighths">Eighth notes</option>
|
||||
</select>
|
||||
</label>
|
||||
<label>Meter
|
||||
<select value={phrase.meterNumerator} onChange={e => setPhrase({ meterNumerator: Number(e.target.value) === 3 ? 3 : 4 })}>
|
||||
<option value={4}>4/4</option>
|
||||
<option value={3}>3/4</option>
|
||||
</select>
|
||||
</label>
|
||||
</>
|
||||
)}
|
||||
|
||||
{phrase.material === 'library' && (
|
||||
<label>Exercise
|
||||
<select value={phrase.libraryId || EXERCISES[0].id} onChange={e => setPhrase({ libraryId: e.target.value, startBar: 1 })}>
|
||||
@@ -578,11 +730,13 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
{[1, 2, 4, 6, 8].map(b => <option key={b} value={b}>{b}</option>)}
|
||||
</select>
|
||||
</label>
|
||||
<label>Octave
|
||||
<label>Transpose
|
||||
<select value={octaveShift} onChange={e => setOctaveShift(Number(e.target.value))}>
|
||||
<option value={-1}>-1</option>
|
||||
<option value={-2}>-2 oct</option>
|
||||
<option value={-1}>-1 oct</option>
|
||||
<option value={0}>0</option>
|
||||
<option value={1}>+1</option>
|
||||
<option value={1}>+1 oct</option>
|
||||
<option value={2}>+2 oct</option>
|
||||
</select>
|
||||
</label>
|
||||
</>
|
||||
@@ -604,61 +758,32 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
)}
|
||||
|
||||
{fretted && (
|
||||
<>
|
||||
<label className="phrase-check">
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={phrase.fretWindowEnabled}
|
||||
onChange={e => setPhrase({ fretWindowEnabled: e.target.checked })}
|
||||
/>
|
||||
Fret window
|
||||
</label>
|
||||
{phrase.fretWindowEnabled && (
|
||||
<>
|
||||
<label>Min fret
|
||||
<input type="number" min={0} max={24} value={phrase.fretMin}
|
||||
onChange={e => setPhrase({ fretMin: Math.max(0, Math.min(24, parseInt(e.target.value) || 0)) })} />
|
||||
</label>
|
||||
<label>Max fret
|
||||
<input type="number" min={0} max={24} value={phrase.fretMax}
|
||||
onChange={e => setPhrase({ fretMax: Math.max(0, Math.min(24, parseInt(e.target.value) || 0)) })} />
|
||||
</label>
|
||||
</>
|
||||
)}
|
||||
</>
|
||||
<span style={{ fontSize: '11px', opacity: 0.7, alignSelf: 'center' }}>
|
||||
Fret window: use the "Fret window" controls below the sheet — shared across modes.
|
||||
</span>
|
||||
)}
|
||||
|
||||
<label>Pace
|
||||
<select value={phrase.pace} onChange={e => setPhrase({ pace: e.target.value as PhraseSettings['pace'] })}>
|
||||
<select
|
||||
value={phrase.pace}
|
||||
onChange={e => {
|
||||
const pace = e.target.value as PhraseSettings['pace'];
|
||||
setPhrase({ pace });
|
||||
// "Sync to metronome" only makes sense with the
|
||||
// metronome running — switch it on automatically.
|
||||
if (pace === 'tempo' && !settings.rhythm.active) {
|
||||
setSettings(s => ({ ...s, rhythm: { ...s.rhythm, active: true } }));
|
||||
}
|
||||
}}
|
||||
>
|
||||
<option value="step">At your pace</option>
|
||||
<option value="tempo">With tempo</option>
|
||||
<option value="tempo">Sync to metronome</option>
|
||||
</select>
|
||||
</label>
|
||||
{phrase.pace === 'tempo' && (
|
||||
<label>BPM
|
||||
<div style={{ display: 'flex', gap: 4, alignItems: 'center' }}>
|
||||
<button className="control-button small" onClick={() => setPhrase({ bpm: Math.max(30, phrase.bpm - 1) })}>−</button>
|
||||
<input
|
||||
type="text"
|
||||
inputMode="numeric"
|
||||
value={bpmText}
|
||||
onChange={e => { if (/^\d{0,3}$/.test(e.target.value)) setBpmText(e.target.value); }}
|
||||
onBlur={commitBpm}
|
||||
onKeyDown={e => { if (e.key === 'Enter') (e.target as HTMLInputElement).blur(); }}
|
||||
style={{ width: '46px', textAlign: 'center' }}
|
||||
/>
|
||||
<button className="control-button small" onClick={() => setPhrase({ bpm: Math.min(180, phrase.bpm + 1) })}>+</button>
|
||||
</div>
|
||||
<input
|
||||
type="range"
|
||||
min={30}
|
||||
max={180}
|
||||
step={1}
|
||||
value={phrase.bpm}
|
||||
onChange={e => setPhrase({ bpm: Number(e.target.value) })}
|
||||
aria-label="Phrase tempo"
|
||||
/>
|
||||
</label>
|
||||
<span style={{ fontSize: '11px', opacity: 0.7, alignSelf: 'center' }}>
|
||||
Tempo (BPM): set it on the metronome in the tools below — shared across modes.
|
||||
</span>
|
||||
)}
|
||||
<label className="phrase-check">
|
||||
<input
|
||||
@@ -669,14 +794,9 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
Virtual input only
|
||||
</label>
|
||||
{phrase.pace === 'tempo' && (
|
||||
<label className="phrase-check">
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={phrase.clickSound}
|
||||
onChange={e => setPhrase({ clickSound: e.target.checked })}
|
||||
/>
|
||||
Click sound
|
||||
</label>
|
||||
<span style={{ fontSize: '11px', opacity: 0.7, alignSelf: 'center' }}>
|
||||
Clicks: use the metronome's "Click sound" checkbox (with count-in, synced).
|
||||
</span>
|
||||
)}
|
||||
<label className="phrase-check">
|
||||
<input
|
||||
@@ -686,9 +806,17 @@ export const PhraseTrainer = forwardRef<PhraseHandle, PhraseTrainerProps>(
|
||||
/>
|
||||
Auto-continue
|
||||
</label>
|
||||
<label className="phrase-check">
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={phrase.autoStartOnNote}
|
||||
onChange={e => setPhrase({ autoStartOnNote: e.target.checked })}
|
||||
/>
|
||||
Start on first note (no count-in)
|
||||
</label>
|
||||
</div>
|
||||
{phrase.pace === 'tempo' && phrase.clickSound && (
|
||||
<p className="phrase-note">Clicks are short noise bursts, not pitched tones — but speaker bleed can still disturb the microphone. Headphones recommended.</p>
|
||||
{phrase.pace === 'tempo' && (
|
||||
<p className="phrase-note">The run clicks a full count-in bar, then one click per beat. The metronome pendulum mirrors these beats — set the tempo on the metronome tool.</p>
|
||||
)}
|
||||
<p className="phrase-note">
|
||||
Matching checks pitch, not fingering — several fretboard positions produce the same pitch.
|
||||
|
||||
233
src/components/TargetNoteControls.tsx
Normal file
233
src/components/TargetNoteControls.tsx
Normal file
@@ -0,0 +1,233 @@
|
||||
/**
|
||||
* Unified target-note controls, shared by all game modes (v2 controls).
|
||||
*
|
||||
* - Key filter: restrict target notes to a tonic+mode (independent of the
|
||||
* displayed key signature on the staff — optional sync checkbox).
|
||||
* - Strings: pick which strings contribute target notes (fretted instruments).
|
||||
* - Fret window: restrict target notes to a fret range (fretted instruments).
|
||||
*
|
||||
* In Phrase Mode the same controls bind to the shared practice filter
|
||||
* (mirrored into phrase settings), so state stays consistent across modes.
|
||||
*/
|
||||
import { useMemo } from 'react';
|
||||
import { Key, Guitar, Music } from 'lucide-react';
|
||||
import { useSettings } from '../context/useSettings';
|
||||
import { getPracticeFilter, type PracticeFilter } from '../types/SettingsTypes';
|
||||
import { MODE_LABELS, TONICS, isMode, keyFor, type ModeId } from '../music/scales';
|
||||
import { isFrettedInstrument } from '../music/playableRange';
|
||||
import { TUNINGS } from '../music/Tunings';
|
||||
|
||||
/** Conventional string names, low→high, for guitar/bass tunings. */
|
||||
const GUITAR_STRING_NAMES = ['E6', 'A5', 'D4', 'G3', 'B2', 'E1'];
|
||||
const BASS_STRING_NAMES = ['E4', 'A3', 'D2', 'G1'];
|
||||
|
||||
function stringNames(tuningId: string, count: number): string[] {
|
||||
const names = tuningId.startsWith('bass') ? BASS_STRING_NAMES : GUITAR_STRING_NAMES;
|
||||
// Align from the low end (index 0 = lowest)
|
||||
if (count === names.length) return names;
|
||||
return Array.from({ length: count }, (_, i) => {
|
||||
const midi = TUNINGS[tuningId]?.strings[i] ?? 0;
|
||||
const pc = midi % 12;
|
||||
const LETTERS = ['C', 'C#', 'D', 'Eb', 'E', 'F', 'F#', 'G', 'Ab', 'A', 'Bb', 'B'];
|
||||
return LETTERS[pc] ?? `S${i + 1}`;
|
||||
});
|
||||
}
|
||||
|
||||
/** Display key signature names that correspond to a tonic+mode pair. */
|
||||
const MAJOR_TONIC_TO_SIGNATURE: Record<string, string> = {
|
||||
'C': 'C', 'G': 'G', 'D': 'D', 'A': 'A', 'E': 'E', 'B': 'C#', 'F#': 'C#',
|
||||
'Db': 'Db', 'Ab': 'Ab', 'Eb': 'Eb', 'Bb': 'Bb', 'F': 'F',
|
||||
};
|
||||
const MINOR_TONIC_TO_SIGNATURE: Record<string, string> = {
|
||||
'C': 'Eb', 'G': 'Bb', 'D': 'F', 'A': 'C', 'E': 'G', 'B': 'D', 'F#': 'A',
|
||||
'Db': 'Eb', 'Ab': 'Bb', 'Eb': 'Gb', 'Bb': 'Db', 'F': 'Dm',
|
||||
};
|
||||
|
||||
export const TargetNoteControls: React.FC = () => {
|
||||
const { settings, updateSettings } = useSettings();
|
||||
const pf = getPracticeFilter(settings);
|
||||
|
||||
const setFilter = (updates: Partial<PracticeFilter>) => {
|
||||
updateSettings(s => {
|
||||
const next: typeof s = { ...s, practice: { ...getPracticeFilter(s), ...updates } };
|
||||
// Optional coupling: display key signature follows the target key.
|
||||
if (updates.keyTonic !== undefined || updates.keyMode !== undefined) {
|
||||
const tonic = updates.keyTonic ?? pf.keyTonic;
|
||||
const mode = updates.keyMode ?? pf.keyMode;
|
||||
if (s.keyFollowsTarget) {
|
||||
const table = mode === 'minor' ? MINOR_TONIC_TO_SIGNATURE : MAJOR_TONIC_TO_SIGNATURE;
|
||||
const sig = table[tonic];
|
||||
if (sig) next.keySignature = sig;
|
||||
}
|
||||
}
|
||||
return next;
|
||||
});
|
||||
};
|
||||
|
||||
const fretted = useMemo(
|
||||
() => isFrettedInstrument(settings.instrument) && !!TUNINGS[settings.tuningId],
|
||||
[settings.instrument, settings.tuningId],
|
||||
);
|
||||
|
||||
const stringCount = TUNINGS[settings.tuningId]?.strings.length ?? 0;
|
||||
const names = stringNames(settings.tuningId, stringCount);
|
||||
|
||||
const toggleString = (index: number) => {
|
||||
const cur = pf.strings.length > 0 ? [...pf.strings] : Array.from({ length: stringCount }, (_, i) => i);
|
||||
const next = cur.includes(index) ? cur.filter(i => i !== index) : [...cur, index].sort((a, b) => a - b);
|
||||
// Deselecting all = back to "all strings" (empty array)
|
||||
setFilter({ strings: next.length === stringCount ? [] : next });
|
||||
};
|
||||
|
||||
const allSelected = pf.strings.length === 0;
|
||||
|
||||
return (
|
||||
<div className="practice-filter-grid">
|
||||
{/* Key filter */}
|
||||
<div className="control-group" style={{ gridColumn: '1 / -1' }}>
|
||||
<div style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between' }}>
|
||||
<label className="control-label">
|
||||
<Key size={18} />
|
||||
<span>Target key (limits notes)</span>
|
||||
</label>
|
||||
<button
|
||||
className={`switch-button ${pf.keyEnabled ? 'active' : ''}`}
|
||||
onClick={() => setFilter({ keyEnabled: !pf.keyEnabled })}
|
||||
title={pf.keyEnabled ? 'Limit to key: on' : 'Limit to key: off'}
|
||||
>
|
||||
<div className="switch-thumb" />
|
||||
</button>
|
||||
</div>
|
||||
{pf.keyEnabled && (
|
||||
<>
|
||||
<div style={{ display: 'flex', gap: '8px', marginTop: '8px' }}>
|
||||
<select
|
||||
className="control-select"
|
||||
value={pf.keyTonic}
|
||||
onChange={e => setFilter({ keyTonic: e.target.value })}
|
||||
aria-label="Target key tonic"
|
||||
>
|
||||
{Object.keys(TONICS).filter(t => !['C#', 'Gb', 'D#', 'G#', 'A#'].includes(t)).map(t => (
|
||||
<option key={t} value={t}>{t}</option>
|
||||
))}
|
||||
</select>
|
||||
<select
|
||||
className="control-select"
|
||||
value={pf.keyMode}
|
||||
onChange={e => setFilter({ keyMode: isMode(e.target.value) ? e.target.value : 'major' })}
|
||||
aria-label="Target key mode"
|
||||
>
|
||||
{(Object.keys(MODE_LABELS) as ModeId[]).map(m => (
|
||||
<option key={m} value={m}>{MODE_LABELS[m]}</option>
|
||||
))}
|
||||
</select>
|
||||
</div>
|
||||
<label style={{ fontSize: '11px', display: 'flex', alignItems: 'center', gap: '6px', marginTop: '6px', opacity: 0.85 }}>
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={settings.keyFollowsTarget ?? false}
|
||||
onChange={e => updateSettings(s => {
|
||||
const next = { ...s, keyFollowsTarget: e.target.checked };
|
||||
// When enabling sync, snap the display signature now.
|
||||
if (e.target.checked) {
|
||||
const table = pf.keyMode === 'minor' ? MINOR_TONIC_TO_SIGNATURE : MAJOR_TONIC_TO_SIGNATURE;
|
||||
const sig = table[pf.keyTonic];
|
||||
if (sig) next.keySignature = sig;
|
||||
}
|
||||
return next;
|
||||
})}
|
||||
/>
|
||||
Sync display key signature with target key
|
||||
</label>
|
||||
</>
|
||||
)}
|
||||
</div>
|
||||
|
||||
{/* Strings (fretted instruments) */}
|
||||
{fretted && (
|
||||
<div className="control-group" style={{ gridColumn: '1 / -1' }}>
|
||||
<div style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between' }}>
|
||||
<label className="control-label">
|
||||
<Guitar size={18} />
|
||||
<span>Strings (limits notes)</span>
|
||||
</label>
|
||||
{allSelected && (
|
||||
<span style={{ fontSize: '11px', opacity: 0.7 }}>all</span>
|
||||
)}
|
||||
</div>
|
||||
<div style={{ display: 'flex', gap: '6px', marginTop: '8px', flexWrap: 'wrap' }}>
|
||||
{Array.from({ length: stringCount }, (_, i) => {
|
||||
const active = allSelected || pf.strings.includes(i);
|
||||
return (
|
||||
<button
|
||||
key={i}
|
||||
className={`control-button small ${active ? 'active' : ''}`}
|
||||
onClick={() => toggleString(i)}
|
||||
title={`String ${i + 1} (${names[i]})`}
|
||||
style={{ flex: '1 1 0', minWidth: '40px', padding: '6px 4px', fontSize: '11px' }}
|
||||
>
|
||||
{names[i]}
|
||||
</button>
|
||||
);
|
||||
})}
|
||||
</div>
|
||||
{pf.strings.length > 0 && (
|
||||
<button
|
||||
className="link-button"
|
||||
style={{ fontSize: '11px', marginTop: '6px', padding: 0 }}
|
||||
onClick={() => setFilter({ strings: [] })}
|
||||
>
|
||||
Reset to all strings
|
||||
</button>
|
||||
)}
|
||||
</div>
|
||||
)}
|
||||
|
||||
{/* Fret window (fretted instruments) */}
|
||||
{fretted && (
|
||||
<div className="control-group" style={{ gridColumn: '1 / -1' }}>
|
||||
<div style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between' }}>
|
||||
<label className="control-label">
|
||||
<Music size={18} />
|
||||
<span>Fret window (limits notes)</span>
|
||||
</label>
|
||||
<button
|
||||
className={`switch-button ${pf.fretWindowEnabled ? 'active' : ''}`}
|
||||
onClick={() => setFilter({ fretWindowEnabled: !pf.fretWindowEnabled })}
|
||||
title={pf.fretWindowEnabled ? 'Fret window: on' : 'Fret window: off'}
|
||||
>
|
||||
<div className="switch-thumb" />
|
||||
</button>
|
||||
</div>
|
||||
{pf.fretWindowEnabled && (
|
||||
<div style={{ display: 'flex', gap: '8px', marginTop: '8px', alignItems: 'center' }}>
|
||||
<div style={{ flex: 1, display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<span style={{ fontSize: '12px', opacity: 0.7 }}>Min</span>
|
||||
<input
|
||||
type="number" min={0} max={24}
|
||||
className="control-input"
|
||||
style={{ width: '100%', padding: '4px' }}
|
||||
value={pf.fretMin}
|
||||
onChange={e => setFilter({ fretMin: Math.max(0, Math.min(24, parseInt(e.target.value) || 0)) })}
|
||||
/>
|
||||
</div>
|
||||
<div style={{ flex: 1, display: 'flex', alignItems: 'center', gap: '4px' }}>
|
||||
<span style={{ fontSize: '12px', opacity: 0.7 }}>Max</span>
|
||||
<input
|
||||
type="number" min={0} max={24}
|
||||
className="control-input"
|
||||
style={{ width: '100%', padding: '4px' }}
|
||||
value={pf.fretMax}
|
||||
onChange={e => setFilter({ fretMax: Math.max(pf.fretMin, Math.min(24, parseInt(e.target.value) || 0)) })}
|
||||
/>
|
||||
</div>
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
);
|
||||
};
|
||||
|
||||
// Re-export keyFor for potential external use (avoids unused-import lint)
|
||||
void keyFor;
|
||||
@@ -16,6 +16,7 @@ export interface ScheduleEntry {
|
||||
readonly at: number;
|
||||
/** Beat number within the count-in (count-in-beat only). */
|
||||
readonly beat?: number;
|
||||
readonly accent?: boolean;
|
||||
/** Event index (note-end only). */
|
||||
readonly index?: number;
|
||||
}
|
||||
@@ -41,7 +42,7 @@ export function buildSchedule(score: Score, bpm: number, startAt: number, countI
|
||||
const entries: ScheduleEntry[] = [];
|
||||
|
||||
for (let i = 0; i < countInBeats; i++) {
|
||||
entries.push({ kind: 'count-in-beat', at: startAt + i * spb, beat: i });
|
||||
entries.push({ kind: 'count-in-beat', at: startAt + i * spb, beat: i, accent: i === 0 });
|
||||
}
|
||||
|
||||
const scoreStart = startAt + countInBeats * spb;
|
||||
@@ -53,7 +54,7 @@ export function buildSchedule(score: Score, bpm: number, startAt: number, countI
|
||||
const barTicks = score.meter.numerator * PPQ;
|
||||
for (let i = 0; i < beatCount; i++) {
|
||||
const tick = i * PPQ;
|
||||
entries.push({ kind: 'beat', at: scoreStart + (tick / PPQ) * spb, beat: (tick % barTicks === 0) ? i : undefined, index: i });
|
||||
entries.push({ kind: 'beat', at: scoreStart + (tick / PPQ) * spb, beat: i, accent: tick % barTicks === 0, index: i });
|
||||
}
|
||||
|
||||
scoreEvents(score).forEach((e, i) => {
|
||||
|
||||
21
src/hooks/phraseBeatBus.ts
Normal file
21
src/hooks/phraseBeatBus.ts
Normal file
@@ -0,0 +1,21 @@
|
||||
/**
|
||||
* Beat clock shared between the phrase trainer's scheduler and the global
|
||||
* metronome widget. The phrase run emits every beat (count-in + score beats)
|
||||
* here; the metronome widget mirrors them on its pendulum so the visual
|
||||
* stays in sync with the audible phrase click. Beats carry an absolute
|
||||
* AudioContext timestamp so the widget can schedule the visual swing at the
|
||||
* right moment even when callbacks fire ahead of time.
|
||||
*/
|
||||
type BeatListener = (beat: number, at: number) => void;
|
||||
|
||||
const listeners = new Set<BeatListener>();
|
||||
|
||||
export const phraseBeatBus = {
|
||||
emit(beat: number, at: number): void {
|
||||
listeners.forEach(l => l(beat, at));
|
||||
},
|
||||
subscribe(l: BeatListener): () => void {
|
||||
listeners.add(l);
|
||||
return () => { listeners.delete(l); };
|
||||
},
|
||||
};
|
||||
26
src/hooks/phraseRunBus.ts
Normal file
26
src/hooks/phraseRunBus.ts
Normal file
@@ -0,0 +1,26 @@
|
||||
/**
|
||||
* Tiny pub/sub for the phrase trainer's running state.
|
||||
*
|
||||
* Lets the (global) metronome widget gate its pendulum/click on the phrase
|
||||
* run: in Phrase Mode the metronome should only be active while the trainer
|
||||
* is counting in or playing, not while idle.
|
||||
*/
|
||||
type Listener = (running: boolean) => void;
|
||||
|
||||
const listeners = new Set<Listener>();
|
||||
let running = false;
|
||||
|
||||
export const phraseRunBus = {
|
||||
set(v: boolean): void {
|
||||
if (v === running) return;
|
||||
running = v;
|
||||
listeners.forEach(l => l(v));
|
||||
},
|
||||
get(): boolean {
|
||||
return running;
|
||||
},
|
||||
subscribe(l: Listener): () => void {
|
||||
listeners.add(l);
|
||||
return () => { listeners.delete(l); };
|
||||
},
|
||||
};
|
||||
@@ -8,12 +8,8 @@ import {
|
||||
getRandomNote,
|
||||
getNoteDetails
|
||||
} from '../music/NoteUtils';
|
||||
import {
|
||||
TUNINGS,
|
||||
getOpenStringNotes,
|
||||
getFirstPositionNotes,
|
||||
} from '../music/Tunings';
|
||||
import { INSTRUMENT_DEFINITIONS } from '../music/InstrumentConfigs';
|
||||
import { computeTargetPool } from '../music/targetPool';
|
||||
import { getPracticeFilter } from '../types/SettingsTypes';
|
||||
import {
|
||||
NOTE_MATCH_THRESHOLD_MS,
|
||||
NOTE_MATCH_GRACE_MS,
|
||||
@@ -36,65 +32,24 @@ export const useGameLogic = (
|
||||
const [tracker] = useState(() => new MatchTracker(NOTE_MATCH_THRESHOLD_MS, NOTE_MATCH_GRACE_MS));
|
||||
|
||||
// --- Valid Notes Calculation ---
|
||||
const currentTuning = TUNINGS[settings.tuningId];
|
||||
const currentInstrumentDef = INSTRUMENT_DEFINITIONS[settings.instrument];
|
||||
|
||||
const validNotes = useMemo(() => {
|
||||
const rangeConfig = currentInstrumentDef.ranges.find(r => r.id === settings.difficulty);
|
||||
|
||||
const getNotesFromConfig = (config: typeof rangeConfig) => {
|
||||
if (!config) return [];
|
||||
|
||||
if (config.type === 'open_strings') {
|
||||
if (currentTuning) return getOpenStringNotes(currentTuning);
|
||||
return config.notes || [];
|
||||
}
|
||||
|
||||
if (config.type === 'first_position') {
|
||||
if (currentTuning) return getFirstPositionNotes(currentTuning);
|
||||
return config.notes || [];
|
||||
}
|
||||
|
||||
if (config.type === 'custom_fret') {
|
||||
if (currentTuning) {
|
||||
const minFret = settings.customMinFret ?? config.defaultMinFret ?? 0;
|
||||
const maxFret = settings.customMaxFret ?? config.defaultMaxFret ?? 12;
|
||||
const notes = new Set<number>();
|
||||
currentTuning.strings.forEach(stringMidi => {
|
||||
for (let fret = minFret; fret <= maxFret; fret++) {
|
||||
notes.add(stringMidi + fret);
|
||||
}
|
||||
});
|
||||
return Array.from(notes).sort((a, b) => a - b);
|
||||
}
|
||||
return [];
|
||||
}
|
||||
|
||||
if (config.type === 'specific_string') {
|
||||
if (currentTuning && config.stringIndex !== undefined) {
|
||||
const openNote = currentTuning.strings[config.stringIndex];
|
||||
if (openNote === undefined) return [];
|
||||
const notes = [];
|
||||
for (let i = 0; i <= 12; i++) {
|
||||
notes.push(openNote + i);
|
||||
}
|
||||
return notes;
|
||||
}
|
||||
return [];
|
||||
}
|
||||
|
||||
if (config.notes) return config.notes;
|
||||
if (config.min !== undefined && config.max !== undefined) {
|
||||
return Array.from({ length: config.max - config.min + 1 }, (_, i) => config.min! + i);
|
||||
}
|
||||
return [];
|
||||
};
|
||||
|
||||
if (rangeConfig) return getNotesFromConfig(rangeConfig);
|
||||
const fallbackRange = currentInstrumentDef.ranges[0];
|
||||
return getNotesFromConfig(fallbackRange);
|
||||
|
||||
}, [settings.difficulty, currentInstrumentDef, currentTuning, settings.customMinFret, settings.customMaxFret]);
|
||||
const pf = getPracticeFilter(settings);
|
||||
return computeTargetPool({
|
||||
instrumentId: settings.instrument,
|
||||
difficulty: settings.difficulty,
|
||||
tuningId: settings.tuningId,
|
||||
customMinFret: settings.customMinFret,
|
||||
customMaxFret: settings.customMaxFret,
|
||||
keyEnabled: pf.keyEnabled,
|
||||
keyTonic: pf.keyTonic,
|
||||
keyMode: pf.keyMode,
|
||||
strings: pf.strings,
|
||||
fretWindowEnabled: pf.fretWindowEnabled,
|
||||
fretMin: pf.fretMin,
|
||||
fretMax: pf.fretMax,
|
||||
});
|
||||
}, [settings]);
|
||||
|
||||
// --- Note Generation ---
|
||||
const generateNewNote = useCallback((keepFeedback = false) => {
|
||||
|
||||
@@ -18,6 +18,8 @@ import { buildSchedule, sanitizeBpm, type ScheduleEntry } from '../game/PhraseTr
|
||||
import { PPQ } from '../score/model';
|
||||
import { audioEngine } from '../audio/AudioEngine';
|
||||
import { subscribeRawFrames } from './rawFrameBus';
|
||||
import { phraseRunBus } from './phraseRunBus';
|
||||
import { phraseBeatBus } from './phraseBeatBus';
|
||||
|
||||
export type PhrasePhase = 'ready' | 'countIn' | 'playing' | 'paused' | 'preview' | 'done';
|
||||
|
||||
@@ -32,6 +34,9 @@ export interface PhraseTrainerConfig {
|
||||
readonly clickSound: boolean;
|
||||
readonly inputMode: 'mic' | 'virtual';
|
||||
readonly previewVolume?: number;
|
||||
/** Auto-start: begin without count-in as soon as the mic detects the
|
||||
* first note (step pace only — tempo needs the scheduled clock). */
|
||||
readonly autoStartOnNote?: boolean;
|
||||
}
|
||||
|
||||
export interface PhraseSummary {
|
||||
@@ -118,6 +123,12 @@ export function usePhraseTrainer(
|
||||
const setPhaseBoth = useCallback((p: PhrasePhase) => {
|
||||
phaseRef.current = p;
|
||||
setPhase(p);
|
||||
// Gate the shared metronome only in 'sync to metronome' (tempo) pace:
|
||||
// there the run owns the clock (clicks during count-in/playing only).
|
||||
// In 'at your pace' (step) the metronome ticks continuously when the
|
||||
// user has it switched on — like the single-note modes.
|
||||
const ownsClock = paceRef.current === 'tempo';
|
||||
phraseRunBus.set(ownsClock && (p === 'countIn' || p === 'playing' || p === 'preview'));
|
||||
}, []);
|
||||
|
||||
useEffect(() => { paceRef.current = config.pace; }, [config.pace]);
|
||||
@@ -197,7 +208,12 @@ export function usePhraseTrainer(
|
||||
countInBeatsRef.current > 0 &&
|
||||
e.at <= scoreStartRef.current + 0.02;
|
||||
if ((e.kind === 'count-in-beat' || e.kind === 'beat') && !isRedundantDownbeat && configRef.current.clickSound) {
|
||||
audioEngine.playClickAt(e.at, CLICK_VOLUME, e.kind === 'count-in-beat' ? e.beat === 0 : e.beat !== undefined);
|
||||
audioEngine.playClickAt(e.at, CLICK_VOLUME, e.kind === 'count-in-beat' ? e.beat === 0 : e.accent === true);
|
||||
}
|
||||
// Publish every beat to the shared bus so the metronome widget's
|
||||
// pendulum stays in sync with the run (click or no click).
|
||||
if (e.kind === 'count-in-beat' || e.kind === 'beat') {
|
||||
phraseBeatBus.emit(e.beat ?? 0, e.at);
|
||||
}
|
||||
clickIdxRef.current++;
|
||||
}
|
||||
@@ -264,7 +280,18 @@ export function usePhraseTrainer(
|
||||
const unsubscribe = subscribeRawFrames(frame => {
|
||||
if (configRef.current.inputMode !== 'mic') return;
|
||||
const p = phaseRef.current;
|
||||
if (p !== 'countIn' && p !== 'playing') return;
|
||||
// Auto-start: first detected note begins the run immediately
|
||||
// (no count-in) — only in step pace, only from 'ready', mic only.
|
||||
if (
|
||||
p === 'ready' &&
|
||||
configRef.current.autoStartOnNote &&
|
||||
paceRef.current === 'step'
|
||||
) {
|
||||
startRef.current?.();
|
||||
// Fall through: this frame feeds the fresh run below.
|
||||
}
|
||||
const p2 = phaseRef.current;
|
||||
if (p2 !== 'countIn' && p2 !== 'playing') return;
|
||||
// Pitched speaker output (preview, virtual instruments) must not
|
||||
// be scored; while audible the matcher sees no frames at all.
|
||||
if (audioEngine.isAudible()) return;
|
||||
@@ -478,6 +505,15 @@ export function usePhraseTrainer(
|
||||
if (!e.pitch) continue;
|
||||
audioEngine.scheduleNote(e.pitch.midi, t0 + (e.startTick / PPQ) * spb, (e.durationTicks / PPQ) * spb, vol, PREVIEW_GROUP);
|
||||
}
|
||||
// Tempo pace: click along with the preview so the rhythm is
|
||||
// audible while listening (quarter beats over the full span).
|
||||
if (paceRef.current === 'tempo' && configRef.current.clickSound) {
|
||||
const totalTicks = events.reduce((a, e) => Math.max(a, e.startTick + e.durationTicks), 0);
|
||||
const beats = Math.floor(totalTicks / PPQ);
|
||||
for (let i = 0; i < beats; i++) {
|
||||
audioEngine.playClickAt(t0 + i * spb, CLICK_VOLUME, i % 4 === 0);
|
||||
}
|
||||
}
|
||||
setPhaseBoth('preview');
|
||||
const totalSec = (events.reduce((a, e) => Math.max(a, e.startTick + e.durationTicks), 0) / PPQ) * spb;
|
||||
previewTimerRef.current = setTimeout(() => {
|
||||
|
||||
138
src/music/arpeggioGenerator.test.ts
Normal file
138
src/music/arpeggioGenerator.test.ts
Normal file
@@ -0,0 +1,138 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { generateArpeggio } from './arpeggioGenerator';
|
||||
import { scoreEvents, validateScore } from '../score/model';
|
||||
|
||||
const strings = [40, 45, 50, 55, 59, 64];
|
||||
const pool: number[] = [];
|
||||
for (const open of strings) for (let f = 0; f <= 12; f++) pool.push(open + f);
|
||||
|
||||
const base = {
|
||||
keyTonic: 'C',
|
||||
keyMode: 'major' as const,
|
||||
meter: { numerator: 4 as const, denominator: 4 as const },
|
||||
};
|
||||
|
||||
describe('generateArpeggio', () => {
|
||||
it('builds a C major arpeggio (up, one octave) from the pool', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'one-octave' }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const ev = scoreEvents(r.value);
|
||||
// C-E-G-C: 4 notes, quarter each (plus final extension)
|
||||
const midis = ev.map(e => e.pitch!.midi);
|
||||
expect(midis).toEqual([48, 52, 55, 60]);
|
||||
expect(validateScore(r.value)).toEqual([]);
|
||||
expect(r.value.title).toContain('Arpeggio');
|
||||
expect(r.value.title).toContain('C I');
|
||||
});
|
||||
|
||||
it('down pattern reverses the ascending cycle', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'down', coverage: 'one-octave' }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const midis = scoreEvents(r.value).map(e => e.pitch!.midi);
|
||||
expect(midis).toEqual([60, 55, 52, 48]);
|
||||
});
|
||||
|
||||
it('updown returns to the starting root without duplicating the top', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'updown', coverage: 'one-octave' }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const midis = scoreEvents(r.value).map(e => e.pitch!.midi);
|
||||
// ascend C-E-G-C then descend back to C; only one C5 (turning point)
|
||||
expect(midis).toEqual([48, 52, 55, 60, 55, 52, 48]);
|
||||
expect(midis.filter(m => m === 60)).toHaveLength(1);
|
||||
});
|
||||
|
||||
it('1235 pattern ends on the octave of the root', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: '1235', coverage: 'one-octave' }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const midis = scoreEvents(r.value).map(e => e.pitch!.midi);
|
||||
expect(midis).toEqual([48, 52, 55, 60]);
|
||||
});
|
||||
|
||||
it('two-octave coverage spans 12 semitones beyond the first cycle', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'two-octave' }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const midis = scoreEvents(r.value).map(e => e.pitch!.midi);
|
||||
expect(midis).toEqual([48, 52, 55, 60, 64, 67, 72]);
|
||||
});
|
||||
it('spells the dominant triad G-B-D in C major', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'V', pattern: 'up', coverage: 'one-octave' }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const pcs = scoreEvents(r.value).map(e => e.pitch!.midi % 12);
|
||||
expect(pcs).toEqual([7, 11, 2, 7]); // G B D G
|
||||
});
|
||||
|
||||
it('spells the leading-tone dim triad B-D-F in C major', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'vii0', pattern: 'up', coverage: 'one-octave' }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const pcs = scoreEvents(r.value).map(e => e.pitch!.midi % 12);
|
||||
expect(pcs).toEqual([11, 2, 5, 11]); // B D F B
|
||||
});
|
||||
|
||||
it('fails cleanly when the pool cannot fit a full triad cycle', () => {
|
||||
// Only frets 0-1 on all strings: no complete C-E-G cycle with octave
|
||||
const tiny: number[] = [];
|
||||
for (const open of strings) for (let f = 0; f <= 1; f++) tiny.push(open + f);
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'one-octave' }, tiny);
|
||||
expect(r.ok).toBe(false);
|
||||
});
|
||||
|
||||
it('is deterministic for identical configs', () => {
|
||||
const a = generateArpeggio({ ...base, degree: 'IV', pattern: 'updown', coverage: 'one-octave' }, pool);
|
||||
const b = generateArpeggio({ ...base, degree: 'IV', pattern: 'updown', coverage: 'one-octave' }, pool);
|
||||
expect(a).toEqual(b);
|
||||
});
|
||||
});
|
||||
|
||||
describe('generateArpeggio — sequence mode', () => {
|
||||
it('generates one chord per bar with functional progression', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'one-octave', bars: 4, progression: 'functional', seed: 7 }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
expect(r.value.measures.length).toBe(4);
|
||||
expect(r.value.chordSymbols?.length).toBe(4);
|
||||
expect(r.value.chordSymbols?.[0]).toBe('C'); // starts on I
|
||||
// bar boundaries: each bar starts on a chord root? (not enforced) — but ticks align
|
||||
const ev = scoreEvents(r.value);
|
||||
expect(validateScore(r.value)).toEqual([]);
|
||||
// 4 beats per bar × 4 bars = 16 notes in quarters mode
|
||||
expect(ev.length).toBe(16);
|
||||
// deterministic
|
||||
const r2 = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'one-octave', bars: 4, progression: 'functional', seed: 7 }, pool);
|
||||
expect(r2.ok && JSON.stringify(scoreEvents(r2.value)) === JSON.stringify(ev)).toBe(true);
|
||||
});
|
||||
|
||||
it('diatonic cycle walks I ii iii IV V vi vii', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'one-octave', bars: 7, progression: 'diatonic-cycle', seed: 1 }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const syms = r.value.chordSymbols!;
|
||||
expect(syms).toEqual(['C', 'Dm', 'Em', 'F', 'G', 'Am', 'Bdim']);
|
||||
});
|
||||
|
||||
it('eighths rhythm doubles notes per bar', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'one-octave', bars: 2, progression: 'functional', rhythm: 'eighths', seed: 3 }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const ev = scoreEvents(r.value);
|
||||
expect(ev.length).toBe(16); // 2 bars × 4 beats × 2
|
||||
expect(ev[0].durationTicks).toBe(240);
|
||||
expect(validateScore(r.value)).toEqual([]);
|
||||
});
|
||||
|
||||
it('random avoids repeating the same chord back-to-back', () => {
|
||||
const r = generateArpeggio({ ...base, degree: 'I', pattern: 'up', coverage: 'one-octave', bars: 6, progression: 'random', seed: 11 }, pool);
|
||||
expect(r.ok).toBe(true);
|
||||
if (!r.ok) return;
|
||||
const syms = r.value.chordSymbols!;
|
||||
for (let i = 1; i < syms.length; i++) {
|
||||
expect(syms[i]).not.toBe(syms[i - 1]);
|
||||
}
|
||||
});
|
||||
});
|
||||
378
src/music/arpeggioGenerator.ts
Normal file
378
src/music/arpeggioGenerator.ts
Normal file
@@ -0,0 +1,378 @@
|
||||
/**
|
||||
* Arpeggio generator (Phrase Mode material).
|
||||
*
|
||||
* Diatonic triads (I, ii, iii, IV, V, vi, vii°) built from the selected
|
||||
* key/mode; each practice run arpeggiates ONE chord — the user picks the
|
||||
* degree, the pattern (up / down / up-down / 1-2-3-5) and the coverage
|
||||
* (one or two octaves). Chord tones are spelled via the key's degree
|
||||
* spelling (same machinery as melody/scale generators), so spelling and
|
||||
* validation match the Phrase Mode invariants exactly.
|
||||
*/
|
||||
import { PPQ, validateScore, type Result, type Score, type ScoreEvent } from '../score/model';
|
||||
import { keyFor, type KeyContext, type ModeId } from './scales';
|
||||
import { STEP_LETTERS, LETTER_PC, type StepLetter } from '../score/model';
|
||||
import { makeRng } from './melodyGenerator';
|
||||
|
||||
export type ArpeggioDegree = 'I' | 'ii' | 'iii' | 'IV' | 'V' | 'vi' | 'vii0';
|
||||
export type ArpeggioPattern =
|
||||
| 'up' | 'down' | 'updown' | '1235'
|
||||
// Giuliani-style right-hand figures over one chord (indices into the
|
||||
// chord cycle per note; 1=root 2=third 3=fifth 4=octave):
|
||||
| 'giuliani-pim' // root-third-fifth (p-i-m)
|
||||
| 'giuliani-pima' // root-third-fifth-octave (p-i-m-a)
|
||||
| 'giuliani-pami' // root-octave-fifth-third (p-a-m-i)
|
||||
| 'giuliani-mami' // third-fifth-octave-third (inner-voice feel)
|
||||
| 'giuliani-ami' // octave-fifth-third (descending from the top)
|
||||
| 'giuliani-pimamim' // p-i-m-a-m-i (six-note rolling figure)
|
||||
| 'giuliani-papima' // p-a-p-i-m-a (bass-skip pattern)
|
||||
| 'giuliani-amipim' // a-m-i-p-i-m (tremolo-adjacent)
|
||||
| 'giuliani-miamim'; // m-i-a-m-i-m (triple-string figure)
|
||||
export type ArpeggioCoverage = 'one-octave' | 'two-octave';
|
||||
/** How the chord per bar is chosen in sequence mode. */
|
||||
export type ArpeggioProgression = 'random' | 'functional' | 'diatonic-cycle';
|
||||
/** Rhythm density for arpeggio sequences. */
|
||||
export type ArpeggioRhythm = 'quarters' | 'eighths';
|
||||
|
||||
export interface ArpeggioConfig {
|
||||
readonly keyTonic: string;
|
||||
readonly keyMode: ModeId;
|
||||
readonly degree: ArpeggioDegree;
|
||||
readonly pattern: ArpeggioPattern;
|
||||
readonly coverage: ArpeggioCoverage;
|
||||
readonly meter: { readonly numerator: 3 | 4; readonly denominator: 4 };
|
||||
/** Sequence mode: bars > 1 with chord progression selection. */
|
||||
readonly bars?: number;
|
||||
readonly progression?: ArpeggioProgression;
|
||||
readonly rhythm?: ArpeggioRhythm;
|
||||
/** Restrict progression chords to these degrees (empty = all 7).
|
||||
* The first bar always uses `degree` if it is in the selection. */
|
||||
readonly chordSelection?: readonly ArpeggioDegree[];
|
||||
/** Seed for progression selection (deterministic). */
|
||||
readonly seed?: number;
|
||||
}
|
||||
|
||||
/** Degree index (0-based) into the scale for each arpeggio degree label. */
|
||||
const DEGREE_INDEX: Record<ArpeggioDegree, number> = {
|
||||
I: 0, ii: 1, iii: 2, IV: 3, V: 4, vi: 5, 'vii0': 6,
|
||||
};
|
||||
|
||||
export const ARPEGGIO_DEGREE_LABELS: Record<ArpeggioDegree, string> = {
|
||||
I: 'I', ii: 'ii', iii: 'iii', IV: 'IV', V: 'V', vi: 'vi', 'vii0': 'vii°',
|
||||
};
|
||||
|
||||
export const PATTERN_LABELS: Record<ArpeggioPattern, string> = {
|
||||
up: 'Up', down: 'Down', updown: 'Up & down', '1235': '1-2-3-5',
|
||||
'giuliani-pim': 'Giuliani p-i-m',
|
||||
'giuliani-pima': 'Giuliani p-i-m-a',
|
||||
'giuliani-pami': 'Giuliani p-a-m-i',
|
||||
'giuliani-mami': 'Giuliani m-a-m-i',
|
||||
'giuliani-ami': 'Giuliani a-m-i',
|
||||
'giuliani-pimamim': 'Giuliani p-i-m-a-m-i',
|
||||
'giuliani-papima': 'Giuliani p-a-p-i-m-a',
|
||||
'giuliani-amipim': 'Giuliani a-m-i-p-i-m',
|
||||
'giuliani-miamim': 'Giuliani m-i-a-m-i-m',
|
||||
};
|
||||
|
||||
/**
|
||||
* Giuliani figures: chord-cycle index per note (0=root 1=third 2=fifth
|
||||
* 3=octave). Each figure repeats with an octave register shift per
|
||||
* repetition, mirroring the p-i-m-a feel of the 120 right-hand studies.
|
||||
*/
|
||||
const GIULIANI_FIGURES: Record<string, number[]> = {
|
||||
'giuliani-pim': [0, 1, 2],
|
||||
'giuliani-pima': [0, 1, 2, 3],
|
||||
'giuliani-pami': [0, 3, 2, 1],
|
||||
'giuliani-mami': [1, 2, 3, 1],
|
||||
'giuliani-ami': [3, 2, 1],
|
||||
'giuliani-pimamim': [0, 1, 2, 3, 2, 1],
|
||||
'giuliani-papima': [0, 3, 0, 1, 2, 3],
|
||||
'giuliani-amipim': [3, 2, 1, 0, 1, 2],
|
||||
'giuliani-miamim': [2, 1, 3, 2, 1, 2],
|
||||
};
|
||||
|
||||
/** pc of a spelled degree (step letter + alter). */
|
||||
const degreePc = (d: { step: StepLetter; alter: number }): number =>
|
||||
(LETTER_PC[STEP_LETTERS.indexOf(d.step)] + d.alter + 144) % 12;
|
||||
|
||||
/** Chord symbol name for a diatonic degree, e.g. "C", "Am", "Bdim". */
|
||||
function chordNameFor(key: KeyContext, degreeIndex: number): string {
|
||||
const d = key.degrees[degreeIndex];
|
||||
const third = key.degrees[(degreeIndex + 2) % 7];
|
||||
const fifth = key.degrees[(degreeIndex + 4) % 7];
|
||||
const root = d.step + (d.alter < 0 ? 'b'.repeat(-d.alter) : '#'.repeat(d.alter));
|
||||
const thirdInterval = (degreePc(third) - degreePc(d) + 12) % 12;
|
||||
const fifthInterval = (degreePc(fifth) - degreePc(d) + 12) % 12;
|
||||
let suffix = '';
|
||||
if (thirdInterval === 3 && fifthInterval === 7) suffix = 'm';
|
||||
else if (thirdInterval === 3 && fifthInterval === 6) suffix = 'dim';
|
||||
return root + suffix;
|
||||
}
|
||||
|
||||
/**
|
||||
* Chord-tone midi notes of the diatonic triad on `degreeIndex` (0-based),
|
||||
* intersected with the playable pool, ascending.
|
||||
*/
|
||||
function chordTonesInPool(key: KeyContext, degreeIndex: number, pool: number[]): number[] {
|
||||
const steps = [0, 2, 4].map(offset => (degreeIndex + offset) % 7);
|
||||
const pcs = new Set(steps.map(i => degreePc(key.degrees[i])));
|
||||
return pool.filter(midi => pcs.has(((midi % 12) + 12) % 12));
|
||||
}
|
||||
|
||||
/** Spelling (step, alter) for a midi note in the key — via scale degrees. */
|
||||
function spellInKey(key: KeyContext, midi: number): { step: StepLetter; alter: number; octave: number } | null {
|
||||
const pc = ((midi % 12) + 12) % 12;
|
||||
const d = key.degrees.find(deg => degreePc(deg) === pc);
|
||||
if (!d) return null;
|
||||
const octave = (midi - pc) / 12 - 1;
|
||||
return { step: d.step, alter: d.alter, octave };
|
||||
}
|
||||
|
||||
/**
|
||||
* Build the arpeggio path: chord tones ascending from the lowest available
|
||||
* root (degree tone) that allows a full one/two-octave cycle in the pool.
|
||||
* Pattern up/down/updown/1235 is applied to one octave cycle.
|
||||
*/
|
||||
function buildPath(key: KeyContext, degreeIndex: number, pool: number[], coverage: ArpeggioCoverage, pattern: ArpeggioPattern): number[] | null {
|
||||
const tones = chordTonesInPool(key, degreeIndex, pool);
|
||||
if (tones.length < 3) return null; // need at least one full triad
|
||||
|
||||
const rootPc = degreePc(key.degrees[degreeIndex]);
|
||||
const octaveCycle = (startMidi: number): number[] => {
|
||||
// chord tones >= startMidi within one octave, INCLUDING the octave
|
||||
// of the root (an arpeggio cycle conventionally ends on the octave)
|
||||
const cycle: number[] = [];
|
||||
for (const midi of tones) {
|
||||
if (midi >= startMidi && midi <= startMidi + 12) cycle.push(midi);
|
||||
}
|
||||
if (!cycle.includes(startMidi + 12) && pool.includes(startMidi + 12)) {
|
||||
cycle.push(startMidi + 12);
|
||||
}
|
||||
return cycle.sort((a, b) => a - b);
|
||||
};
|
||||
|
||||
const cycles: number[][] = [];
|
||||
// candidate roots: chord-tone midis with pc == rootPc, ascending
|
||||
const roots = tones.filter(m => ((m % 12) + 12) % 12 === rootPc);
|
||||
const octavesWanted = coverage === 'one-octave' ? 1 : 2;
|
||||
for (const root of roots) {
|
||||
const first = octaveCycle(root);
|
||||
if (first.length < 3 || first[0] !== root) continue;
|
||||
// second octave: tones one octave above `root`
|
||||
const second = first.map(m => m + 12).filter(m => pool.includes(m));
|
||||
if (octavesWanted === 2 && second.length < 4) continue;
|
||||
cycles.push(first);
|
||||
break; // lowest fitting root
|
||||
}
|
||||
if (cycles.length === 0) return null;
|
||||
|
||||
const base = cycles[0];
|
||||
const up = octavesWanted === 2
|
||||
// base already ends on the octave of the root — the second octave
|
||||
// continues from there (drop the duplicated turning root)
|
||||
? [...base, ...base.map(m => m + 12).slice(1)]
|
||||
: base;
|
||||
|
||||
switch (pattern) {
|
||||
case 'up': return up;
|
||||
case 'down': return [...up].reverse();
|
||||
case 'updown': {
|
||||
// ascend then descend, dropping the duplicated turning note
|
||||
const desc = [...up].reverse().slice(1);
|
||||
return [...up, ...desc];
|
||||
}
|
||||
case '1235': {
|
||||
// root, third, fifth, octave (of the root) — repeated per octave
|
||||
const path: number[] = [];
|
||||
for (let oct = 0; oct < octavesWanted; oct++) {
|
||||
path.push(base[0] + 12 * oct);
|
||||
path.push(base[1] + 12 * oct);
|
||||
path.push(base[2] + 12 * oct);
|
||||
path.push(base[0] + 12 * (oct + 1));
|
||||
}
|
||||
return path;
|
||||
}
|
||||
default: {
|
||||
// Giuliani figure: cycle [root, third, fifth, octave] with the
|
||||
// figure selecting cycle positions per note; the octave wraps so
|
||||
// e.g. p-i-m over C-E-G becomes C E G | C' E' G' ... (ascending
|
||||
// octave register shift every figure repetition).
|
||||
const figure = GIULIANI_FIGURES[pattern];
|
||||
// 4-note cycle per octave: root, third, fifth, octave
|
||||
const cycleOct = (oct: number): number[] => [
|
||||
base[0] + 12 * oct, base[1] + 12 * oct, base[2] + 12 * oct, base[0] + 12 * (oct + 1),
|
||||
];
|
||||
const path: number[] = [];
|
||||
for (let oct = 0; oct < octavesWanted; oct++) {
|
||||
const c = cycleOct(oct);
|
||||
for (const idx of figure) {
|
||||
const note = c[idx];
|
||||
if (note !== undefined && (oct === 0 || pool.includes(note))) path.push(note);
|
||||
}
|
||||
}
|
||||
return path;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
export function generateArpeggio(
|
||||
config: ArpeggioConfig,
|
||||
eligiblePitches: number[],
|
||||
): Result<Score> {
|
||||
const key = keyFor(config.keyTonic, config.keyMode);
|
||||
const pool = Array.from(new Set(eligiblePitches.filter(m => Number.isInteger(m)))).sort((a, b) => a - b);
|
||||
if (pool.length === 0) return { ok: false, error: 'No playable notes in the selected range — widen the note set.' };
|
||||
|
||||
const bars = config.bars ?? 1;
|
||||
const pattern = config.pattern;
|
||||
const coverage = config.coverage;
|
||||
|
||||
// ---- Chord sequence ----------------------------------------------------
|
||||
// Single-bar mode: the chosen degree for all bars (bars is 1).
|
||||
// Sequence mode: one chord per bar, chosen by progression rule.
|
||||
const degreeIndices: number[] = [];
|
||||
// Sequence chord pool: user selection (restricted to it) or all 7.
|
||||
const sel = (config.chordSelection ?? []).filter(d => d in DEGREE_INDEX).map(d => DEGREE_INDEX[d as ArpeggioDegree]);
|
||||
const pool7 = sel.length > 0 ? Array.from(new Set(sel)).sort((a, b) => a - b) : [0, 1, 2, 3, 4, 5, 6];
|
||||
if (bars === 1) {
|
||||
degreeIndices.push(DEGREE_INDEX[config.degree]);
|
||||
} else {
|
||||
const rng = makeRng(config.seed ?? 1);
|
||||
const pick = <T,>(arr: T[]): T => arr[Math.floor(rng() * arr.length)];
|
||||
for (let b = 0; b < bars; b++) {
|
||||
if (b === 0) {
|
||||
// First bar: a random chord FROM THE SELECTION — fixed starts
|
||||
// repeated the exercise identically every time.
|
||||
degreeIndices.push(pool7.includes(DEGREE_INDEX[config.degree]) && rng() < 0.34
|
||||
? DEGREE_INDEX[config.degree]
|
||||
: pick(pool7));
|
||||
continue;
|
||||
}
|
||||
const prev = degreeIndices[b - 1];
|
||||
if (config.progression === 'diatonic-cycle') {
|
||||
// Next diatonic degree that is also in the selection
|
||||
let next = (prev + 1) % 7;
|
||||
for (let tries = 0; tries < 7 && !pool7.includes(next); tries++) next = (next + 1) % 7;
|
||||
degreeIndices.push(next);
|
||||
} else if (config.progression === 'functional') {
|
||||
// Simple tonal function rules (major-oriented; mirrored for
|
||||
// minor by degree relationships below):
|
||||
// I -> {IV, V, vi}
|
||||
// ii -> {V}
|
||||
// iii -> {IV, vi}
|
||||
// IV -> {I, V}
|
||||
// V -> {I, vi}
|
||||
// vi -> {ii, IV}
|
||||
// vii -> {I}
|
||||
const NEXT: Record<number, number[]> = {
|
||||
0: [3, 4, 5],
|
||||
1: [4],
|
||||
2: [3, 5],
|
||||
3: [0, 4],
|
||||
4: [0, 5],
|
||||
5: [1, 3],
|
||||
6: [0],
|
||||
};
|
||||
// functional: honor the selection — filter rule targets by it
|
||||
const targets = (NEXT[prev] ?? [0]).filter(d => pool7.includes(d));
|
||||
degreeIndices.push(pick(targets.length > 0 ? targets : pool7.filter(d => d !== prev)));
|
||||
} else {
|
||||
// random: any selected degree, avoid immediate repetition
|
||||
const candidates = pool7.filter(d => d !== prev);
|
||||
degreeIndices.push(pick(candidates.length > 0 ? candidates : pool7));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// ---- Path per bar: one arpeggio cycle per chord -------------------------
|
||||
const full = config.meter.numerator * PPQ;
|
||||
const perBarSegments: { path: number[]; degreeIndex: number }[] = [];
|
||||
for (const di of degreeIndices) {
|
||||
const path = buildPath(key, di, pool, coverage, pattern);
|
||||
if (!path || path.length === 0) {
|
||||
return {
|
||||
ok: false,
|
||||
error: 'The arpeggio does not fit the playable range — widen the range (strings/fret window) or pick another degree.',
|
||||
};
|
||||
}
|
||||
perBarSegments.push({ path, degreeIndex: di });
|
||||
}
|
||||
|
||||
// ---- Events: fit each segment into one bar ------------------------------
|
||||
const rhythm = config.rhythm ?? 'quarters';
|
||||
const events: ScoreEvent[] = [];
|
||||
const chordSymbols: (string | null)[] = [];
|
||||
let cursor = 0;
|
||||
for (let b = 0; b < bars; b++) {
|
||||
const { path, degreeIndex } = perBarSegments[b];
|
||||
chordSymbols.push(chordNameFor(key, degreeIndex));
|
||||
if (bars === 1) {
|
||||
// Single-bar mode (v1 behavior): whole path, final note extended
|
||||
// to fill the bar — or, when the path overflows one bar, spread
|
||||
// over the needed bars with the overflow in the last note.
|
||||
const barCount = Math.max(1, Math.ceil((path.length * PPQ) / full));
|
||||
const totalTicks = barCount * full;
|
||||
const lastDuration = PPQ + (totalTicks - path.length * PPQ);
|
||||
path.forEach((midi, n) => {
|
||||
const spelled = spellInKey(key, midi);
|
||||
if (!spelled) return;
|
||||
const dur = n === path.length - 1 ? lastDuration : PPQ;
|
||||
events.push({ id: `a${b}-${n}`, startTick: cursor, durationTicks: dur, pitch: { midi, ...spelled } });
|
||||
cursor += dur;
|
||||
});
|
||||
continue;
|
||||
}
|
||||
// Sequence mode: fit the bar exactly. 'quarters': one note per beat,
|
||||
// cycling up/down through the path. 'eighths': two notes per beat.
|
||||
const slots = rhythm === 'quarters' ? 1 : 2;
|
||||
const notesPerBar = config.meter.numerator * slots;
|
||||
const line: number[] = [];
|
||||
let dir = 1;
|
||||
let lastIdx = 0;
|
||||
while (line.length < notesPerBar) {
|
||||
line.push(path[lastIdx]);
|
||||
if (lastIdx + dir < 0 || lastIdx + dir >= path.length) dir = -dir;
|
||||
else lastIdx += dir;
|
||||
}
|
||||
const noteDur = full / notesPerBar;
|
||||
line.forEach((midi, n) => {
|
||||
const spelled = spellInKey(key, midi);
|
||||
if (!spelled) return;
|
||||
events.push({
|
||||
id: `a${b}-${n}`,
|
||||
startTick: cursor,
|
||||
durationTicks: noteDur,
|
||||
pitch: { midi, ...spelled },
|
||||
});
|
||||
cursor += noteDur;
|
||||
});
|
||||
}
|
||||
|
||||
const chordName = `${key.tonic} ${ARPEGGIO_DEGREE_LABELS[config.degree]}`;
|
||||
const patternLabel = PATTERN_LABELS[config.pattern];
|
||||
// Single-bar paths can overflow into extra bars (updown/two-octave);
|
||||
// sequence mode always has exactly `bars` measures.
|
||||
const totalMeasures = bars === 1 ? Math.max(1, Math.ceil((cursor) / full)) : bars;
|
||||
const title = bars === 1
|
||||
? `Arpeggio — ${chordName} (${patternLabel}, ${coverage})`
|
||||
: `Arpeggio — ${key.tonic}: ${bars} bars (${config.progression ?? 'functional'})`;
|
||||
const score: Score = {
|
||||
version: 1,
|
||||
id: `arpeggio-${config.keyTonic}-${config.keyMode}-${config.degree}-${config.pattern}-${config.coverage}-${bars}b-${config.progression ?? 'single'}-${config.seed ?? 0}`,
|
||||
title,
|
||||
meter: config.meter,
|
||||
key: { tonic: config.keyTonic, mode: config.keyMode, signature: key.signature },
|
||||
measures: Array.from({ length: totalMeasures }, (_, i) => ({
|
||||
number: i + 1,
|
||||
startTick: i * full,
|
||||
durationTicks: full,
|
||||
})),
|
||||
chordSymbols: bars === 1
|
||||
// repeat the single chord symbol across the overflow bars
|
||||
? Array.from({ length: totalMeasures }, () => chordSymbols[0])
|
||||
: chordSymbols,
|
||||
voices: [{ id: 'melody', events }],
|
||||
};
|
||||
const problems = validateScore(score);
|
||||
if (problems.length > 0) return { ok: false, error: `Internal arpeggio error: ${problems[0]}` };
|
||||
return { ok: true, value: score };
|
||||
}
|
||||
29
src/music/melodyBarTotal.test.ts
Normal file
29
src/music/melodyBarTotal.test.ts
Normal file
@@ -0,0 +1,29 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { generateMelody } from './melodyGenerator';
|
||||
import { scoreEvents, validateScore } from '../score/model';
|
||||
|
||||
// Reproduce user report: "bei 4/4 manchmal 5 Viertel in bar 2"
|
||||
describe('melody generator bar totals (4/4, level 2)', () => {
|
||||
const strings = [40, 45, 50, 55, 59, 64];
|
||||
const pool: number[] = [];
|
||||
for (const open of strings) for (let f = 0; f <= 12; f++) pool.push(open + f);
|
||||
|
||||
it('bar 2 always totals exactly one bar', () => {
|
||||
let checked = 0;
|
||||
for (let seed = 1; seed <= 500; seed++) {
|
||||
const r = generateMelody(
|
||||
{ keyTonic: 'C', keyMode: 'major', bars: 2, meter: { numerator: 4, denominator: 4 }, rhythmLevel: 2, seed },
|
||||
pool,
|
||||
);
|
||||
if (!r.ok) continue;
|
||||
checked++;
|
||||
const ev = scoreEvents(r.value.score);
|
||||
const problems = validateScore(r.value.score);
|
||||
expect(problems, `seed ${seed}`).toEqual([]);
|
||||
const bar2 = ev.filter(e => e.startTick >= 1920 && e.startTick < 3840);
|
||||
const bar2Len = bar2.reduce((a, e) => a + e.durationTicks, 0);
|
||||
expect(bar2Len, `seed ${seed} bar2 ticks`).toBe(1920);
|
||||
}
|
||||
expect(checked).toBeGreaterThan(400);
|
||||
});
|
||||
});
|
||||
@@ -31,8 +31,9 @@ export interface MelodyConfig {
|
||||
/** 1..8 */
|
||||
readonly bars: number;
|
||||
readonly meter: { readonly numerator: 3 | 4; readonly denominator: 4 };
|
||||
/** 1 = Simple (quarters/halves/wholes), 2 = Mixed (+ paired eighths, dotted quarters). */
|
||||
readonly rhythmLevel: 1 | 2;
|
||||
/** 1 = Simple (quarters/halves/wholes), 2 = Mixed (+ paired eighths, dotted quarters),
|
||||
* 3 = Elaborate (+ sixteenths, sixteenth groups, dotted eighths). */
|
||||
readonly rhythmLevel: 1 | 2 | 3;
|
||||
readonly seed: number;
|
||||
}
|
||||
|
||||
@@ -53,6 +54,17 @@ const TEMPLATES_4_L2: number[][] = [
|
||||
[720, 720, 480],
|
||||
[720, 480, 240, 240, 240],
|
||||
];
|
||||
// Level 3: sixteenth-note material (120 = 16th, 360 = dotted 8th).
|
||||
const TEMPLATES_4_L3: number[][] = [
|
||||
[480, 480, 120, 120, 120, 120, 480],
|
||||
[120, 120, 120, 120, 480, 480, 480],
|
||||
[480, 120, 120, 120, 120, 480, 480],
|
||||
[360, 120, 480, 480, 480],
|
||||
[480, 360, 120, 480, 480],
|
||||
[120, 120, 120, 120, 120, 120, 120, 120, 480, 480],
|
||||
[480, 480, 360, 120, 480],
|
||||
[720, 120, 120, 120, 120, 480],
|
||||
];
|
||||
const TEMPLATES_3_L1: number[][] = [
|
||||
[480, 480, 480],
|
||||
[960, 480],
|
||||
@@ -65,11 +77,19 @@ const TEMPLATES_3_L2: number[][] = [
|
||||
[720, 720],
|
||||
[720, 240, 240, 240],
|
||||
];
|
||||
// Level 3 for 3/4: sixteenth material filling 1440 ticks.
|
||||
const TEMPLATES_3_L3: number[][] = [
|
||||
[480, 120, 120, 120, 120, 480, 480],
|
||||
[120, 120, 120, 120, 480, 480, 480],
|
||||
[360, 120, 480, 480, 480],
|
||||
[480, 360, 120, 480, 480],
|
||||
[120, 120, 120, 120, 120, 120, 120, 120, 480],
|
||||
];
|
||||
|
||||
function templatesFor(meter: { numerator: 3 | 4 }, level: 1 | 2): number[][] {
|
||||
function templatesFor(meter: { numerator: 3 | 4 }, level: 1 | 2 | 3): number[][] {
|
||||
const base = meter.numerator === 4
|
||||
? (level === 1 ? TEMPLATES_4_L1 : [...TEMPLATES_4_L1, ...TEMPLATES_4_L2])
|
||||
: (level === 1 ? TEMPLATES_3_L1 : [...TEMPLATES_3_L1, ...TEMPLATES_3_L2]);
|
||||
? (level === 1 ? TEMPLATES_4_L1 : level === 2 ? [...TEMPLATES_4_L1, ...TEMPLATES_4_L2] : [...TEMPLATES_4_L1, ...TEMPLATES_4_L2, ...TEMPLATES_4_L3])
|
||||
: (level === 1 ? TEMPLATES_3_L1 : level === 2 ? [...TEMPLATES_3_L1, ...TEMPLATES_3_L2] : [...TEMPLATES_3_L1, ...TEMPLATES_3_L2, ...TEMPLATES_3_L3]);
|
||||
return base.filter(t => t.reduce((a, b) => a + b, 0) === meter.numerator * PPQ);
|
||||
}
|
||||
|
||||
|
||||
@@ -12,6 +12,7 @@ import { keyFor, scalePitches, type KeyContext, type ModeId } from './scales';
|
||||
|
||||
export type DrillDirection = 'up' | 'down' | 'updown';
|
||||
export type DrillCoverage = 'one-octave' | 'two-octave' | 'position';
|
||||
export type DrillRhythm = 'quarters' | 'eighths';
|
||||
|
||||
export interface DrillConfig {
|
||||
readonly keyTonic: string;
|
||||
@@ -19,6 +20,8 @@ export interface DrillConfig {
|
||||
readonly direction: DrillDirection;
|
||||
readonly coverage: DrillCoverage;
|
||||
readonly meter: { readonly numerator: 3 | 4; readonly denominator: 4 };
|
||||
/** Note duration: quarters (default) or eighths. */
|
||||
readonly rhythm?: DrillRhythm;
|
||||
}
|
||||
|
||||
/** Complete ascending scale path from a starting tonic, if it fits the pool. */
|
||||
@@ -87,14 +90,15 @@ export function generateScaleDrill(
|
||||
else path = [...up, ...[...up].reverse().slice(1)]; // no duplicate turning tonic
|
||||
}
|
||||
|
||||
// ---- Quarter-note rhythm, final note extended to fill its bar ---------
|
||||
// ---- Rhythm: quarters (default) or eighths; final note extended ---------
|
||||
const noteDur = config.rhythm === 'eighths' ? PPQ / 2 : PPQ;
|
||||
const full = config.meter.numerator * PPQ;
|
||||
if (path.length * PPQ > 8 * full) {
|
||||
return { ok: false, error: 'This drill is longer than 8 bars — reduce the coverage or direction.' };
|
||||
if (path.length * noteDur > 8 * full) {
|
||||
return { ok: false, error: 'This drill is longer than 8 bars — reduce the coverage, direction or note duration.' };
|
||||
}
|
||||
const barCount = Math.max(1, Math.ceil((path.length * PPQ) / full));
|
||||
const barCount = Math.max(1, Math.ceil((path.length * noteDur) / full));
|
||||
const totalTicks = barCount * full;
|
||||
const lastDuration = PPQ + (totalTicks - path.length * PPQ);
|
||||
const lastDuration = noteDur + (totalTicks - path.length * noteDur);
|
||||
|
||||
const events: ScoreEvent[] = [];
|
||||
let cursor = 0;
|
||||
@@ -104,7 +108,7 @@ export function generateScaleDrill(
|
||||
return found ? found.pitch : sp[0]?.pitch ?? null;
|
||||
};
|
||||
for (let i = 0; i < path.length; i++) {
|
||||
const dur = i === path.length - 1 ? lastDuration : PPQ;
|
||||
const dur = i === path.length - 1 ? lastDuration : noteDur;
|
||||
const pitch = spellFor(path[i]);
|
||||
if (!pitch) return { ok: false, error: 'Internal error spelling drill pitch.' };
|
||||
events.push({ id: `s${i}`, startTick: cursor, durationTicks: dur, pitch });
|
||||
@@ -113,7 +117,7 @@ export function generateScaleDrill(
|
||||
|
||||
const score: Score = {
|
||||
version: 1,
|
||||
id: `scale-${config.keyTonic}-${config.keyMode}-${config.coverage}-${config.direction}-${config.meter.numerator}4`,
|
||||
id: `scale-${config.keyTonic}-${config.keyMode}-${config.coverage}-${config.direction}-${config.rhythm ?? 'quarters'}-${config.meter.numerator}4`,
|
||||
title: `${key.tonic} scale — ${config.coverage === 'position' ? 'within position' : config.coverage} ${config.direction}`,
|
||||
meter: config.meter,
|
||||
key: { tonic: config.keyTonic, mode: config.keyMode, signature: key.signature },
|
||||
|
||||
110
src/music/targetPool.test.ts
Normal file
110
src/music/targetPool.test.ts
Normal file
@@ -0,0 +1,110 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { computeTargetPool } from './targetPool';
|
||||
|
||||
const base = {
|
||||
instrumentId: 'guitar',
|
||||
difficulty: 'custom',
|
||||
tuningId: 'standard',
|
||||
customMinFret: 0,
|
||||
customMaxFret: 12,
|
||||
keyEnabled: false,
|
||||
keyTonic: 'C',
|
||||
keyMode: 'major',
|
||||
strings: [],
|
||||
fretWindowEnabled: false,
|
||||
fretMin: 0,
|
||||
fretMax: 4,
|
||||
};
|
||||
|
||||
describe('computeTargetPool', () => {
|
||||
it('returns the base playable range when no filters are active', () => {
|
||||
const pool = computeTargetPool(base);
|
||||
expect(pool).toContain(40); // low E
|
||||
expect(pool).toContain(64); // high e (12th fret)
|
||||
expect(pool.length).toBeGreaterThan(30);
|
||||
});
|
||||
|
||||
it('applies the fret window when enabled', () => {
|
||||
const pool = computeTargetPool({ ...base, fretWindowEnabled: true, fretMin: 0, fretMax: 4 });
|
||||
// guitar standard: open strings 40,45,50,55,59,64; window adds up to +4
|
||||
const lo = Math.min(...pool);
|
||||
const hi = Math.max(...pool);
|
||||
expect(lo).toBe(40);
|
||||
expect(hi).toBe(68); // 64 + 4
|
||||
expect(pool).toContain(59); // open g
|
||||
expect(pool).not.toContain(69);
|
||||
});
|
||||
|
||||
it('restricts to pitch classes of the key when key filter is on', () => {
|
||||
const pool = computeTargetPool({ ...base, keyEnabled: true, keyTonic: 'C', keyMode: 'major' });
|
||||
const pcs = new Set([0, 2, 4, 5, 7, 9, 11]); // C major
|
||||
for (const midi of pool) {
|
||||
expect(pcs.has(((midi % 12) + 12) % 12)).toBe(true);
|
||||
}
|
||||
// chromatic notes excluded
|
||||
expect(pool).not.toContain(61); // C# within range
|
||||
});
|
||||
|
||||
it('combines fret window and key filter', () => {
|
||||
const pool = computeTargetPool({ ...base, fretWindowEnabled: true, fretMin: 0, fretMax: 5, keyEnabled: true, keyTonic: 'E', keyMode: 'minor' });
|
||||
const pcs = new Set([4, 6, 7, 9, 11, 0, 2]); // E minor natural: E F# G A B C D
|
||||
expect(pool.length).toBeGreaterThan(0);
|
||||
for (const midi of pool) {
|
||||
expect(pcs.has(((midi % 12) + 12) % 12)).toBe(true);
|
||||
}
|
||||
// F (pc 5) and F# (pc 6 is in E minor; F natural is not) — F natural excluded
|
||||
expect(pool.some(m => ((m % 12) + 12) % 12 === 5)).toBe(false);
|
||||
});
|
||||
|
||||
it('supports modes beyond the single-note key signatures (dorian)', () => {
|
||||
const pool = computeTargetPool({ ...base, keyEnabled: true, keyTonic: 'D', keyMode: 'dorian' });
|
||||
const pcs = new Set([2, 4, 5, 7, 9, 11, 0]); // D dorian: D E F G A B C
|
||||
expect(pool.length).toBeGreaterThan(0);
|
||||
for (const midi of pool) {
|
||||
expect(pcs.has(((midi % 12) + 12) % 12)).toBe(true);
|
||||
}
|
||||
});
|
||||
|
||||
it('falls back to major for an invalid mode id', () => {
|
||||
const a = computeTargetPool({ ...base, keyEnabled: true, keyTonic: 'C', keyMode: 'major' });
|
||||
const b = computeTargetPool({ ...base, keyEnabled: true, keyTonic: 'C', keyMode: 'bogus' as string });
|
||||
expect(b).toEqual(a);
|
||||
});
|
||||
|
||||
it('returns an empty pool rather than crashing for a non-fretted window request', () => {
|
||||
const pool = computeTargetPool({ ...base, instrumentId: 'piano', fretWindowEnabled: true });
|
||||
// piano has no fret window: window is ignored, base pool returned
|
||||
expect(pool.length).toBeGreaterThan(0);
|
||||
});
|
||||
});
|
||||
|
||||
describe('computeTargetPool — string selection', () => {
|
||||
it('restricts to a single string when selected', () => {
|
||||
// String index 5 = high E (open midi 64)
|
||||
const pool = computeTargetPool({ ...base, strings: [5] });
|
||||
expect(pool).toContain(64); // open high E
|
||||
expect(pool).toContain(65); // fret 1
|
||||
expect(pool).not.toContain(45); // open A (string 1) — different string
|
||||
// all notes come from one string
|
||||
expect(pool.every(m => m >= 64)).toBe(true);
|
||||
});
|
||||
|
||||
it('combines strings + fret window', () => {
|
||||
// String index 1 (open A = 45), frets 0..2
|
||||
const pool = computeTargetPool({ ...base, strings: [1], fretWindowEnabled: true, fretMin: 0, fretMax: 2 });
|
||||
expect(pool).toEqual([45, 46, 47]);
|
||||
});
|
||||
|
||||
it('combines strings + fret window + key filter', () => {
|
||||
// Low E string (index 0), frets 0..4, C major → only natural notes in that range
|
||||
const pool = computeTargetPool({ ...base, strings: [0], fretWindowEnabled: true, fretMin: 0, fretMax: 4, keyEnabled: true, keyTonic: 'C', keyMode: 'major' });
|
||||
// Low E string frets 0-4: E(40) F(41) F#(42) G(43) G#(44); C major pcs: C D E F G A B
|
||||
expect(pool).toEqual([40, 41, 43]); // E, F, G (F#, G# excluded)
|
||||
});
|
||||
|
||||
it('treats empty strings array as all strings', () => {
|
||||
const a = computeTargetPool({ ...base, strings: [] });
|
||||
const b = computeTargetPool({ ...base, strings: [0, 1, 2, 3, 4, 5] });
|
||||
expect(a).toEqual(b);
|
||||
});
|
||||
});
|
||||
79
src/music/targetPool.ts
Normal file
79
src/music/targetPool.ts
Normal file
@@ -0,0 +1,79 @@
|
||||
/**
|
||||
* Unified target-note pool: base playable range (instrument + difficulty /
|
||||
* fret window) intersected with the optional key-filter (pitch classes of a
|
||||
* tonic+mode) and the optional fret window. Shared by all game modes so the
|
||||
* controls behave identically everywhere.
|
||||
*/
|
||||
import { keyFor, type ModeId } from './scales';
|
||||
import { computePlayableNotes, isFrettedInstrument } from './playableRange';
|
||||
import { TUNINGS } from './Tunings';
|
||||
|
||||
export interface TargetPoolInput {
|
||||
readonly instrumentId: string;
|
||||
readonly difficulty: string;
|
||||
readonly tuningId: string;
|
||||
readonly customMinFret?: number;
|
||||
readonly customMaxFret?: number;
|
||||
/** Key filter (target-note key, independent of the displayed key signature). */
|
||||
readonly keyEnabled: boolean;
|
||||
readonly keyTonic: string;
|
||||
readonly keyMode: string;
|
||||
/** Selected string indices (0 = lowest). Empty = all strings. */
|
||||
readonly strings: number[];
|
||||
/** Fret window (fretted instruments only). */
|
||||
readonly fretWindowEnabled: boolean;
|
||||
readonly fretMin: number;
|
||||
readonly fretMax: number;
|
||||
}
|
||||
|
||||
/**
|
||||
* Compute the effective target-note pool.
|
||||
*
|
||||
* Order of application:
|
||||
* 1. base playable range (instrument difficulty / custom frets)
|
||||
* 2. string selection ∩ fret window (fretted instruments only — replaces
|
||||
* the range, same semantics as Phrase Mode's fret window)
|
||||
* 3. key filter (pitch-class intersection, keeps spelling/degree data intact
|
||||
* via the same scale machinery Phrase Mode uses)
|
||||
*/
|
||||
export function computeTargetPool(input: TargetPoolInput): number[] {
|
||||
const tuning = TUNINGS[input.tuningId];
|
||||
const fretted = isFrettedInstrument(input.instrumentId) && !!tuning;
|
||||
|
||||
// 1+2: base pool with string selection and optional fret window.
|
||||
// For fretted instruments with NO filters the pool is the full fretboard
|
||||
// (frets 0–12) — the legacy difficulty ranges no longer apply since the
|
||||
// Note Set dropdown was removed.
|
||||
let base: number[];
|
||||
if (fretted) {
|
||||
const all = tuning!.strings;
|
||||
const selected = input.strings.length > 0
|
||||
? all.filter((_, i) => input.strings.includes(i))
|
||||
: all;
|
||||
if (selected.length === 0) return [];
|
||||
const lo = input.fretWindowEnabled ? Math.max(0, Math.min(24, input.fretMin)) : 0;
|
||||
const hi = input.fretWindowEnabled ? Math.max(lo, Math.min(24, input.fretMax)) : 12;
|
||||
base = selected.flatMap(open => {
|
||||
const notes: number[] = [];
|
||||
for (let fret = lo; fret <= hi; fret++) notes.push(open + fret);
|
||||
return notes;
|
||||
}).sort((a, b) => a - b);
|
||||
} else {
|
||||
base = computePlayableNotes({
|
||||
instrumentId: input.instrumentId,
|
||||
difficulty: input.difficulty,
|
||||
tuningId: input.tuningId,
|
||||
customMinFret: input.customMinFret,
|
||||
customMaxFret: input.customMaxFret,
|
||||
});
|
||||
}
|
||||
if (!input.keyEnabled) return base;
|
||||
|
||||
// 3: key/mode pitch-class filter
|
||||
const mode: ModeId = (['major', 'minor', 'dorian', 'phrygian', 'lydian', 'mixolydian', 'locrian', 'major-pentatonic', 'minor-pentatonic'] as const).includes(input.keyMode as ModeId)
|
||||
? input.keyMode as ModeId
|
||||
: 'major';
|
||||
const key = keyFor(input.keyTonic, mode);
|
||||
const pcs = new Set(key.pitchClasses);
|
||||
return base.filter(midi => pcs.has(((midi % 12) + 12) % 12));
|
||||
}
|
||||
@@ -91,6 +91,9 @@ export interface Score {
|
||||
readonly measures: ScoreMeasure[];
|
||||
/** Exactly one voice in v1. */
|
||||
readonly voices: ReadonlyArray<{ readonly id: 'melody'; readonly events: ScoreEvent[] }>;
|
||||
/** Optional chord symbol per bar index (0-based), e.g. "C", "Am", "G7".
|
||||
* Rendered above the staff; absent = no symbol for that bar. */
|
||||
readonly chordSymbols?: ReadonlyArray<string | null>;
|
||||
readonly source?: ScoreSource;
|
||||
}
|
||||
|
||||
|
||||
@@ -1,16 +1,48 @@
|
||||
export type Difficulty = string;
|
||||
|
||||
/** Unified target-note filter, shared by all game modes (v2 controls). */
|
||||
export interface PracticeFilter {
|
||||
/** Restrict target notes to pitch classes of this key/mode (null = off). */
|
||||
keyEnabled: boolean;
|
||||
keyTonic: string;
|
||||
keyMode: string; // ModeId
|
||||
/** Restrict target notes to selected strings (fretted instruments only).
|
||||
* Empty array = no restriction (all strings). Index 0 = lowest string. */
|
||||
strings: number[];
|
||||
/** Restrict target notes to a fret range (fretted instruments only). */
|
||||
fretWindowEnabled: boolean;
|
||||
fretMin: number;
|
||||
fretMax: number;
|
||||
}
|
||||
|
||||
export const DEFAULT_PRACTICE_FILTER: PracticeFilter = {
|
||||
keyEnabled: false,
|
||||
keyTonic: 'C',
|
||||
keyMode: 'major',
|
||||
strings: [],
|
||||
fretWindowEnabled: false,
|
||||
fretMin: 0,
|
||||
fretMax: 4,
|
||||
};
|
||||
|
||||
export function getPracticeFilter(s: AppSettings): PracticeFilter {
|
||||
return { ...DEFAULT_PRACTICE_FILTER, ...(s.practice ?? {}) };
|
||||
}
|
||||
|
||||
export interface RhythmSettings {
|
||||
mode: 'bpm' | 'seconds';
|
||||
bpm: number;
|
||||
seconds: number;
|
||||
active: boolean;
|
||||
/** Phrase Mode: sync the metronome to the exercise (starts with it,
|
||||
* silent while idle) instead of running freely. */
|
||||
syncToExercise: boolean;
|
||||
autoAdvance: boolean;
|
||||
sound: boolean;
|
||||
volume: number;
|
||||
}
|
||||
|
||||
export type PhraseMaterial = 'melody' | 'scale' | 'library' | 'import';
|
||||
export type PhraseMaterial = 'melody' | 'scale' | 'arpeggio' | 'library' | 'import';
|
||||
export type PhrasePace = 'step' | 'tempo';
|
||||
|
||||
export interface PhraseSettings {
|
||||
@@ -21,9 +53,20 @@ export interface PhraseSettings {
|
||||
keyMode: string; // ModeId
|
||||
bars: number; // 1–8 (melodies)
|
||||
meterNumerator: 3 | 4;
|
||||
rhythmLevel: 1 | 2;
|
||||
rhythmLevel: 1 | 2 | 3;
|
||||
scaleCoverage: 'one-octave' | 'two-octave' | 'position';
|
||||
scaleDirection: 'up' | 'down' | 'updown';
|
||||
scaleRhythm: 'quarters' | 'eighths';
|
||||
/** Arpeggio settings (material = 'arpeggio'). */
|
||||
arpeggioDegree: string; // ArpeggioDegree (single-chord mode / sequence start)
|
||||
/** Sequence mode: which degrees may appear (empty = all 7). */
|
||||
arpeggioChordSelection: string[];
|
||||
arpeggioPattern: 'up' | 'down' | 'updown' | '1235';
|
||||
arpeggioCoverage: 'one-octave' | 'two-octave';
|
||||
/** Sequence mode: bars per exercise (1 = single chord). */
|
||||
arpeggioBars: number;
|
||||
arpeggioProgression: 'random' | 'functional' | 'diatonic-cycle';
|
||||
arpeggioRhythm: 'quarters' | 'eighths';
|
||||
/** Phrase-local fret window (guitar/bass) — overrides the note set. */
|
||||
fretWindowEnabled: boolean;
|
||||
fretMin: number;
|
||||
@@ -38,6 +81,9 @@ export interface PhraseSettings {
|
||||
* new melody) after a short pause — keeps practice flowing. */
|
||||
autoContinue: boolean;
|
||||
inputMode: 'mic' | 'virtual';
|
||||
/** Begin the run as soon as the first note is played (no count-in,
|
||||
* step pace only). */
|
||||
autoStartOnNote: boolean;
|
||||
}
|
||||
|
||||
export const DEFAULT_PHRASE_SETTINGS: PhraseSettings = {
|
||||
@@ -50,6 +96,14 @@ export const DEFAULT_PHRASE_SETTINGS: PhraseSettings = {
|
||||
rhythmLevel: 1,
|
||||
scaleCoverage: 'one-octave',
|
||||
scaleDirection: 'updown',
|
||||
scaleRhythm: 'quarters',
|
||||
arpeggioDegree: 'I',
|
||||
arpeggioChordSelection: [],
|
||||
arpeggioPattern: 'up',
|
||||
arpeggioCoverage: 'one-octave',
|
||||
arpeggioBars: 1,
|
||||
arpeggioProgression: 'functional',
|
||||
arpeggioRhythm: 'quarters',
|
||||
fretWindowEnabled: false,
|
||||
fretMin: 0,
|
||||
fretMax: 4,
|
||||
@@ -60,6 +114,7 @@ export const DEFAULT_PHRASE_SETTINGS: PhraseSettings = {
|
||||
clickSound: false,
|
||||
autoContinue: true,
|
||||
inputMode: 'mic',
|
||||
autoStartOnNote: false,
|
||||
};
|
||||
|
||||
export function getPhraseSettings(s: AppSettings): PhraseSettings {
|
||||
@@ -68,12 +123,16 @@ export function getPhraseSettings(s: AppSettings): PhraseSettings {
|
||||
|
||||
export interface AppSettings {
|
||||
phrase?: PhraseSettings;
|
||||
/** Unified target-note filter (key + fret window), all modes. */
|
||||
practice?: PracticeFilter;
|
||||
difficulty: Difficulty;
|
||||
showHint: boolean;
|
||||
showFretboard: boolean;
|
||||
showTuningMeter: boolean;
|
||||
tuningId: string;
|
||||
keySignature: string;
|
||||
/** Keep display key signature in sync with the target-key control. */
|
||||
keyFollowsTarget?: boolean;
|
||||
instrument: string;
|
||||
rhythm: RhythmSettings;
|
||||
zenMode: boolean;
|
||||
@@ -101,12 +160,14 @@ export const DEFAULT_SETTINGS: AppSettings = {
|
||||
showTuningMeter: false,
|
||||
tuningId: 'standard',
|
||||
keySignature: 'C',
|
||||
keyFollowsTarget: false,
|
||||
instrument: 'guitar',
|
||||
rhythm: {
|
||||
mode: 'bpm',
|
||||
bpm: 60,
|
||||
seconds: 5,
|
||||
active: false,
|
||||
syncToExercise: true,
|
||||
autoAdvance: false,
|
||||
sound: true,
|
||||
volume: 0.5
|
||||
@@ -114,6 +175,7 @@ export const DEFAULT_SETTINGS: AppSettings = {
|
||||
zenMode: false,
|
||||
gameMode: 'sight_reading',
|
||||
phrase: DEFAULT_PHRASE_SETTINGS,
|
||||
practice: DEFAULT_PRACTICE_FILTER,
|
||||
customMinFret: 0,
|
||||
customMaxFret: 12,
|
||||
autoPlaySightReading: false,
|
||||
|
||||
Reference in New Issue
Block a user