v0.6.0 docs: threshold status table + deferral rationale - #17
Open
SeamusMullan wants to merge 1 commit into
Open
v0.6.0 docs: threshold status table + deferral rationale#17SeamusMullan wants to merge 1 commit into
SeamusMullan wants to merge 1 commit into
Conversation
…min/C6-GR Folds the v0.6.0 milestone work into the threshold doc as a status table: 8 of 14 items shipped (UI6-C1, C2, M1, M3, M4, M6, M7, M8 + half each of M2 and C6), 4 deferred to v0.6.1 with rationale (UI6-C3, C4, the true 720x620 minimum for M2, and the GR overlay half of C6), and 2 blocked on Blender assets (UI6-C5 + dependent M5). Also lists each shipped PR with its branch so the ship gate is auditable at tag time. Refs v0.6.0-threshold.md. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the v0.6.0 threshold milestone document to reflect current completion status, including what shipped, what was partially delivered, what’s deferred to v0.6.1, and what’s blocked on missing assets—so the v0.6.0 ship gate can be audited at tag time.
Changes:
- Replaces the prior “Blocking/Required” lists with status tables (Done / Partial / Deferred / Blocked).
- Adds a “v0.6.0 Release Status” section with a Mermaid status pie chart and an auditable shipped-PR table (PR + branch).
- Documents deferral and blockage rationale for remaining items.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
416
to
+425
| ### Blocking Issues (must fix before v0.6.0 ships) | ||
|
|
||
| | ID | Category | Summary | | ||
| |----|----------|---------| | ||
| | UI6-C1 | Theme | DirektDSP_GUI not adopted as LookAndFeel; raw JUCE controls everywhere | | ||
| | UI6-C2 | Theme | No BandGate accent in DirektDSP_GUI palette | | ||
| | UI6-C3 | Theme | No themeable spectral display primitive; `SpectrumVisualizer` is bespoke draw code | | ||
| | UI6-C4 | Layout | Layout does not reflect band-split / spectral operation | | ||
| | UI6-C5 | Visual | No Blender-rendered chrome — UI is pure `fillRect`/`drawRoundedRectangle` | | ||
| | UI6-C6 | Interaction | Spectrum is read-only; no direct manipulation | | ||
| | ID | Category | Status | Summary | | ||
| |----|----------|--------|---------| | ||
| | UI6-C1 | Theme | ✅ Done (#11) | DirektDSP_GUI not adopted as LookAndFeel; raw JUCE controls everywhere | | ||
| | UI6-C2 | Theme | ✅ Done (DirektDSP_GUI#57 + #10) | No BandGate accent in DirektDSP_GUI palette | | ||
| | UI6-C3 | Theme | ⏸ Deferred to v0.6.1 | No themeable spectral display primitive; `SpectrumVisualizer` is bespoke draw code | | ||
| | UI6-C4 | Layout | ⏸ Deferred to v0.6.1 | Layout does not reflect band-split / spectral operation | | ||
| | UI6-C5 | Visual | ⏸ Blocked on Blender assets | No Blender-rendered chrome — UI is pure `fillRect`/`drawRoundedRectangle` | | ||
| | UI6-C6 | Interaction | 🟡 Partial (#16) | Spectrum is read-only; no direct manipulation | |
Comment on lines
+469
to
+472
| - **UI6-C3 (DirektSpectrumDisplay primitive)** — requires lifting the 793-line `SpectrumVisualizer` into a reusable submodule primitive with theme-driven colours, freq/dB rulers, draggable handles, and a GR overlay slot. Roughly a one-week submodule project; out of scope for the UI-cohesion milestone. | ||
| - **UI6-C4 (spectral-axis layout with per-band columns)** — removes the `ACTIVE_BAND` swap and instantiates N parallel `GateControls` / `RelayStrip` columns tied to crossover positions. Major UX shift; needs the v0.6.0 LookAndFeel + decomposition to settle first, then can land cleanly in v0.6.1. | ||
| - **UI6-M2 (true 720×620 minimum)** — needs `DirektFlexContainer` adoption across the strip layout; same dependency chain as C4. | ||
| - **UI6-C6 (GR overlay + threshold snap)** — needs a per-band reduction aggregate that `fetchSpectralVisualData()` does not expose. Either a small DSP plumbing change or a post-fetch averaging pass; pairs naturally with C3 / C4. |
| | #15 | `v0.6.0/ui6-m2-resize` | UI6-M2 (partial) | | ||
| | #16 | `v0.6.0/ui6-c6-hover-readout` | UI6-C6 (partial) | | ||
|
|
||
| Stacked: each PR is based on the previous one. Merge in PR-number order. |
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Folds the v0.6.0 milestone work into
v0.6.0-threshold.mdas a status table and a release-status section:DirektSpectrumDisplayprimitive), UI6-C4 (per-band columnar layout), the true 720x620 minimum half of UI6-M2, and the GR overlay half of UI6-C6.Each shipped PR is listed with its branch so the ship gate is auditable at tag time.
Dependencies
Test plan
Bump VERSION to 0.6.0 + release commit).🤖 Generated with Claude Code