Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
T
Tidal
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • PLN
  • Tidal
  • Repository

Switch branch/tag
  • Tidal
  • tools
  • tests
  • test_preload_plan.py
Find file
BlameHistoryPermalink
  • PLN (Algolia)'s avatar
    preload: a bank named 808bd killed the whole plan, and the checker said ok · 3b518774
    Last night's WORD fix made five banks visible for the first time — 808bd,
    808cy, 808hc, 808sd, 90s_synatm — and the emitter wrote them as `\808bd`.
    A SuperCollider bare symbol cannot start with a digit, so preload.scd died
    at parse time and warmed 0 banks where it had warmed 132 the day before:
    
        ERROR: syntax error, unexpected NAME, expecting ']'
                  [ \808bd, 25, ".../Dirt-Samples/808bd" ],
        ERROR: Command line parse failed
    
    One bad row costs the whole file, and nothing in the rig says so — with
    doNotReadYet restored to true, all 132 banks just fall back to lazy reads.
    It was only visible in the boot journal.
    
    Quoting is the fix: '808bd' and \808bd are the same symbol, and only names
    that need quoting get one, so an existing plan diffs by exactly the broken
    rows. Now 137/137 banks warm in 3.8 s.
    
    Fixing it exposed the second half. The row shape had four readers and
    writers — emit_sc, check-preload's union_plan, and its banks()/bank_counts()
    shell greps for `[ \name`, which cannot match a quoted row. So the checker
    reported ok on a plan genuinely short by five banks: want came out 132
    instead of 137. A checker that cannot read what the emitter writes is worse
    than no checker, so all four now go through plan_row / plan_row_name /
    plan_rows, and the shape is written down once.
    
    43 new tests, and they were watched failing: with the quoting reverted, 21
    of them break and check-preload goes blind again.
    PLN (Algolia) authored Sep 24, 2026
    3b518774
test_preload_plan.py 5.25 KB
EditWeb IDE
×

Replace test_preload_plan.py

Attach a file by drag & drop or click to upload


Cancel
A new branch will be created in your fork and a new merge request will be started.