Skip to content

Prove and document mixed response domains - #1281

Open
AlexanderFengler wants to merge 2 commits into
codex/1265-response-domainsfrom
codex/1265-mixed-response-proof
Open

Prove and document mixed response domains#1281
AlexanderFengler wants to merge 2 commits into
codex/1265-response-domainsfrom
codex/1265-mixed-response-proof

Conversation

@AlexanderFengler

Copy link
Copy Markdown
Member

Summary

  • Prove the dependency-free mixed-response lifecycle for three- and four-column observations, including literal physical ordering, an asymmetric likelihood oracle, prior/posterior prediction, safe-mode chunking, sample_do, DataFrame projection, seeded streams, and cloudpickle reconstruction.
  • Document ModelConfig.response as ordered physical DataFrame columns and response_domains as per-column annotations, with endpoint rules, legacy choices migration, generator obligations, and current limits in one public source of truth.
  • Keep this final slice behavior-neutral: it adds no runtime production logic, abstraction, dependency, model registration, or producer bridge.

Stack

Complexity ledger

  • Runtime production files/LOC: 0; src/hssm/hssm.py changes docstrings only.
  • New public symbols, helpers, conditional branches, modules, or state: 0.
  • Fixed-width cleanup required: none; the generic proof passes through the single resolver, validator, and width path already in the sink.
  • Compatibility retained: exact one-/two-column shapes and names, choice-only scalar output, safe/unsafe prediction, RNG advancement, and serialized state.
  • Deliberately still narrow: plotting, KDE/training, legacy simulate_data, RT-less multi-column data, and producer-schema projection.

Validation

  • 993 passed, 201 deselected, 10 subtests passed in the full non-slow suite.
  • 21 passed in the focused observation-dimension lifecycle suite.
  • Strict MkDocs build and docs-weight check.
  • Ruff check/format, mypy, Pyrefly, and git diff --check.
  • Independent lifecycle and documentation reviews passed on the exact head.

Refs #1265

@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: bfa3dac1-deec-4283-9eeb-d477fb939739

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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