Skip to content

Add reusable echo embedding replacement manifest path - #88

Open
kevzho wants to merge 1 commit into
mainfrom
codex/e12-pooling-ablation-manifest
Open

kevzho wants to merge 1 commit into
mainfrom
codex/e12-pooling-ablation-manifest

Conversation

@kevzho

@kevzho kevzho commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add replace-echo to rebuild a joined manifest for EchoJEPA pooling ablations while preserving the existing paired cohort, labels, splits, and ECG embeddings
  • cover the replacement path with a regression test
  • supports E12 controlled pooled-vs-clip experiments without re-reading the large source HuBERT parquet

Validation

  • PYTHONPATH=src python -m pytest -q: 133 passed, 1 skipped
  • Ruff check passed for the changed files

E12 result

The measured clip-16 condition restored non-degenerate ECG-to-Echo attention, but did not improve fused validation MAE (10.5861 clip-16 vs 10.4499 pooled). Pooled remains canonical; controlled-data manifests, checkpoints, and metric logs remain local and gitignored per the data policy.

Implements E12 (#77).

@kevzho
kevzho requested a review from duckyquang September 11, 2026 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant