Rename the strategy-output param namespace exposure -> bias (instance + exposure.scale knob + manifest label) #134

New Issue

2026-06-24T13:11:46+02:00

Brummel commented

2026-06-24 13:11:46 +02:00

Deferred from the cycle-0065 rename iteration (#126): the typed exposure_sign_flips metric was renamed to bias_sign_flips, but the strategy-output param-namespace cluster was kept uniformly as "exposure" because renaming it is a coherent ~30-site, sweep-pinned change the single-run rename should not have smuggled in.

Rename together (one consistent unit), all driven by the Bias node's instance name:

Bias::builder().named("exposure") -> .named("bias") (the ~6 harness builders + fixtures);
the derived knob path exposure.scale -> bias.scale (Composite::param_space derives the knob from the node name) — touches the sweep axis/.range(...) bindings (blueprint.rs), the walkforward ParamSpecs (walkforward.rs), the member_key dir names, and the JSON byte-pins in cli_run.rs (:602, :1030-1031);
the exposure_scale manifest param label -> bias_scale (the single-run manifest builders).

No on-disk back-compat break (the knob path is a runtime param address; recorded params in old runs.jsonl/families.jsonl are inert data strings). NOT renamed (deliberate, #117): SimBroker's pre-reframe exposure concept (slot 0 + prev_exposure + the exposure · price-return integral) and the on-disk exposure tap label (the recorded bias series feeding chart --tap exposure).

Deferred from the cycle-0065 rename iteration (#126): the typed `exposure_sign_flips` metric was renamed to `bias_sign_flips`, but the **strategy-output param-namespace cluster** was kept uniformly as "exposure" because renaming it is a coherent ~30-site, sweep-pinned change the single-run rename should not have smuggled in. Rename together (one consistent unit), all driven by the Bias node's instance name: - `Bias::builder().named("exposure")` -> `.named("bias")` (the ~6 harness builders + fixtures); - the derived knob path `exposure.scale` -> `bias.scale` (`Composite::param_space` derives the knob from the node name) — touches the sweep axis/`.range(...)` bindings (`blueprint.rs`), the walkforward `ParamSpec`s (`walkforward.rs`), the `member_key` dir names, and the JSON byte-pins in `cli_run.rs` (`:602`, `:1030-1031`); - the `exposure_scale` manifest param label -> `bias_scale` (the single-run manifest builders). No on-disk back-compat break (the knob path is a runtime param address; recorded params in old `runs.jsonl`/`families.jsonl` are inert data strings). NOT renamed (deliberate, #117): SimBroker's pre-reframe exposure concept (slot 0 + `prev_exposure` + the `exposure · price-return` integral) and the on-disk `exposure` **tap** label (the recorded bias series feeding `chart --tap exposure`).

Brummel added the idea label 2026-06-24 13:11:46 +02:00

Brummel referenced this issue from a commit

2026-06-24 14:23:47 +02:00

chore(stage1-r): finalize cycle 0065 — milestone closed, ephemeral spec/plans removed

Brummel referenced this issue from a commit

2026-06-24 14:52:25 +02:00

refactor(stage1-r): rename the strategy-output param namespace exposure -> bias

Brummel closed this issue

2026-06-24 14:52:25 +02:00

Sign in to join this conversation.