Skip to content

Tracking: live keys and guards blocked on a reference implementation #7

Description

@florian6973

This issue exists to stay open. Once the other audit issues close, the repository looks covered — but
the highest-ranked risk in the audit is still entirely unexercised, and nothing on the board would say so.

The blocker

A generated repo ships a stub model.py, so no chain has ever run training or prediction. The rendered
conformance suite gates this whole tier behind skip_if_stub. Closing it needs a reference implementation
living outside the payload — an examples/ tree that a test renders a profile and drops in. That is
the agreed follow-up recorded in CLAUDE.md's "Known gaps", and it is not a reason to put a model back into
template/.

Until it exists, none of the following can be tested, and an agent that picks one up will do one of the
two wrong things: skip_if_stub the test (reproducing the exact gap) or start filling in model.py.

Blocked items

  • do_resume end-to-end — that a resumed run actually inherits the checkpoint, and that resuming from
    an unrelated run yields a different model. The path/decision logic is testable now and is covered in
    the torch-free tests issue; only this half is blocked.
  • seed — two runs at one seed agree; seed=null skips seed_everything. The settings ship in the
    reproducibility issue; proving them needs two real training runs.
  • attach_labels=false — output shape, and that meds_evaluation rejects it as expected.
  • _persist_checkpoint's final_weights fallback (train.py:300) — when no ModelCheckpoint
    monitored anything, the last epoch's weights are published instead of a selected best. Recorded in
    the manifest, never asserted.
  • load_pretrained_weights's zero-match RuntimeError (train.py:280) — triage first: this may
    need only torch and a two-parameter nn.Module, not a trained artifact. If so it moves to the
    torch-free issue.
  • CoverageError (predict.py:175) — reachable in normal use; nothing exercises it.
  • InferenceKind.scores / MaterializedPredictCommand — reachable only through
    zero_shot_materialized, which skip_if_stubs.
  • test_meds_dev_e2e's final assertions — everything up to the model has been executed for real;
    the assertions never have.

Why one issue and not eight

Eight unactionable issues get re-triaged every time someone reads the list, and each one is an invitation
for an agent to work around the blocker rather than respect it. One issue has a single place to record why
nothing here can move yet.

When the reference implementation lands, this splits into real issues.

Related

Ranked #1 (and, with determinism, #2) in the audit's "how quietly it fails" list — the only silent
failure modes in it. A resume that picks up an unrelated run's checkpoint yields a model that trains,
predicts, and looks entirely plausible.

From the Hydra configuration audit, §2 (docs/CONFIG-AUDIT.md).

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    blockedCannot proceed until a dependency landstestingTest coverage

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions