Skip to content

Refresh web-roundtrip fixtures to current .pat format (CRC-16 trailer) #127

Description

@mbreiser

The web→MATLAB roundtrip fixtures in maDisplayTools/patterns/web_generated/ (+ web_generated_manifest.json) predate the per-frame CRC-16/CCITT-FALSE trailer added to the encoder (webDisplayTools commit 9eaae5d). On-disk sizes match the 2026-02-18 manifest (e.g. gs16 2x10 = 81298) but are 40 bytes short (20 frames × 2) of what the current encoder emits (81338). So the fixtures don't exercise the current file format.

They're still self-consistent (files match manifest), so validate_web_roundtrip.m passes as-is — this is hygiene, not a live bug.

Task

  • Re-run node tests/generate-roundtrip-patterns.js --outdir ../maDisplayTools/patterns/web_generated
  • Update web_generated_manifest.json (file sizes change)
  • Re-run MATLAB maDisplayTools/tests/validate_web_roundtrip.m to confirm the round-trip still decodes with the CRC trailer present
  • Optional: the generator hardcodes stretchValues: new Array(tp.numFrames).fill(1) (tests/generate-roundtrip-patterns.js:355). Duty is immaterial to a roundtrip check, but could bump to 0x80 for realism.

Note: these are deterministic test vectors, not user-facing examples; the current duty=1 is harmless.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions