Skip to content

CheckSeamAnchor proves a seam anchor's POSITION and not its MAGNITUDE: a zero-width anchor passes green #1884

Description

@localai-bot

CheckSeamAnchor proves a seam anchor's POSITION and not its MAGNITUDE, so a zero-width anchor passes it green.

Found while landing the three anchors #1668 owes, by mutating the gate that holds them. Filed by the change that introduced the gate, against its own work.

What the gate does prove

CheckSeamAnchor (tests/vllm/multimodal/test_ltx2_video.cpp) holds load.dit_config and artifacts.mux to a POSITION, deliberately carrying no constant, because .agents/specs/ltx25-phase-residue.md ## Design 3 withdrew the instrument-relative bound after three fresh reviews measured its tail. Its clauses:

  • the anchor is emitted exactly once, is not a span, is not nested;
  • it opens at or after its predecessor's end and closes at or before its successor's start;
  • no other leaf lies inside the window it claims;
  • for artifacts.mux, which has no successor, nothing follows it — it is the LAST leaf of its render.

Six mutations red against that, including both call-site deletions.

What it does NOT prove, measured

Mutation M7, NOT DETECTED. Close artifacts.mux immediately after opening it, at the head of the tail, and drop the late mux_phase.Close():

  phase::Scope mux_phase("artifacts.mux");
  mux_phase.Close();          // the whole tail is once again un-named
  result.frame_count = rendered_frames;
[doctest] assertions: 796 | 796 passed | 0 failed |
[doctest] Status: SUCCESS!

Every clause survives it. The count is 1; it is neither span nor nested; it still opens after artifacts.audio ends; nothing follows it, so it is still the last leaf; and a zero-width window [s, s] contains no other leaf vacuously, because every interval satisfies end <= s + 1e-9 || start >= s - 1e-9. So the anchor reports ~0 s, the region it was added to name is back in unaccounted_seconds, and the table reads green with a name on nothing.

The same argument applies to load.dit_config.

Why this is a class and not an oversight

It is the third appearance of one shape, and the file already records the other two:

A magnitude clause is what would close it, and one family of magnitude clause is what this row has already measured shut twice: any floor whose denominator is the INSTRUMENT, the wall, or a written-down number of seconds. ## Design 3: a seam's honest share is a property of the box, residue <= 2 * instrument went red 4 times in 45 at the table bound and 28 in 160 on a standalone probe, and a 20-run sample of that quantity does not see the tail that decides the gate. A floor of that family must not be added to make this issue go away.

That ban does NOT reach every magnitude clause, and an earlier revision of this issue said it did. Those three denominators all ask how many seconds a seam SHOULD hold, and the honest answer is a property of the box. A ratio whose denominator is the seam being named, measured in the same run, asks a different question. See the candidate below.

What would actually close it

The same two shapes the row already names, neither of which is a tolerance:

  1. The seam-extent ratio, which is where to start. The anchor's own extent against the seam its two neighbours define, both measured in the same run: (end - start) against (next_start - previous_end). Honest that is near 1, because a correct anchor covers the seam except for two instrument boundaries; collapsed it is near 0. Wide separation, of the kind the decode.video.vae floor already exploits, and its denominator is the region being named rather than the instrument — so it is outside the banned family. It still needs a constant and a cross-box distribution, so it is a candidate and not a conclusion, and it needs its own red-first measurement and fresh review.
  2. An anchor INSIDE the callee, so the window cannot be narrower than the work — the repair the decode.audio note asks for and the one LTX-2.5: denoise.step and denoise.update are open to a seconds transfer that no assertion in the file can see #1568 needs.
  3. A bound on a quantity the scheduler cannot move, which is the open question of LTX-2.5: nothing bounds the instrument's own share of a leaf, so both residue gates can widen silently #1570 and, one level up, of The LTX-2.5 phase-table SUM gate's 95% floor sits inside its own measurement, and it is RED on main #1439.

Do not close this by widening, and do not close it with a floor from the banned family above. Read .agents/specs/ltx25-phase-residue.md ## Design 3 and ### M7 first.

Scope

Not fixed in flow: closing it needs an anchor inside a callee or a scheduler-independent bound, each of which is its own row with its own red-first tests and fresh review. The anchors themselves are correct and are where the gap decomposition said the time was; what is unheld is that they stay wrapped around it.

Owned by row LTX25-PHASE-RESIDUE, listed under ## Owed in ltx25-phase-residue.md.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions