Skip to content

Integrate fixture CTS workflow with verified OTT solver contract - #4

Merged
jadenfix merged 13 commits into
mainfrom
codex/oct14-ott-contract
Oct 14, 2026
Merged

jadenfix merged 13 commits into
mainfrom
codex/oct14-ott-contract

Conversation

@jadenfix

@jadenfix jadenfix commented Oct 14, 2026 •

Copy link
Copy Markdown
Owner

OTT-JAX 0.6 rejects sinkhorn_kwargs; the previous broad exception handler silently substitutes balanced transport, including for unbalanced requests. Use its actual solve_kwargs contract, propagate execution failures, and allow the balanced fallback only when optional imports are unavailable. Explicit unbalanced requests fail when OTT is unavailable or disabled.

Current head: e502a2d5d288f6ed22000d3a5a673e243e1e7a80; exact local/published tree: 8da22ac2c4e7c21b9a32dd5bf2fc35b9d1f53a11. Integrated owner content through 339abbbbe3d846bcb08a3196536427dd52ba58fa, preserving its truthful replay/matrix diagnostics and real CPU optimizer/artifact CI. No owner branch edits or merges performed.

Regression validation: 14 loss tests pass, including actual installed OTT 0.6 balanced/unbalanced values and finite gradients with fallback forbidden, solver failure propagation, and missing-dependency behavior. Actual two-step CPU CTS run with use_ott=true passes: finite loss 5.198169708251953, parameter delta 14.129130254499614, generated shape [2,2]. Ruff and format pass; pyright has 0 errors and 7 missing optional dependency warnings. Historical source/current blob integrity and full GIF/MP4 decode pass. Exact integrated full suite is running after shared-memory resource exhaustion; merge remains gated on hosted CI run 38623740606.

Published GIF/MP4 remain explicitly historical receipts of the prior source and use_ott=false; they do not establish the new OTT path. Verify captured source blobs and the changed current source blob separately; CI fetches history for that check. No model-quality, live teacher, ALFWorld, scale, or benchmark claim.

Stacked narrow followup to #3. Earlier transient head a316bda was replaced by e502a2d to preserve concurrent owner changes; review and merge only the current exact head.

Copy link
Copy Markdown
Owner Author

Exact published tree 8da22ac2c4e7c21b9a32dd5bf2fc35b9d1f53a11 (head e502a2d5d288f6ed22000d3a5a673e243e1e7a80) now passes the full local suite: 53 tests, 1 JAX/fork warning, 227.07s. Ruff, format, pyright (0 errors; 7 optional-import warnings), historical/current source integrity and full 60-frame GIF/MP4 decoding pass. Hosted CI run 38623740606 has passed the test step and is executing the real local optimizer/archive-diagnostic lane. Merge remains held until all current-head checks finish. Published media retain their historical use_ott=false scope.

Copy link
Copy Markdown
Owner Author

Current-head hosted CI 38623740606 passes all steps. Repeated the actual two-step CTS optimizer with use_ott=true against exact integrated source tree 8da22ac2c4e7c21b9a32dd5bf2fc35b9d1f53a11: finite loss 5.198169708251953, L1 parameter delta 14.129130254499614, generated [2,2], group [2,3,8]. This closes the whole-tree provenance gap after concurrent owner integration. Review: APPROVE the scoped OTT contract/error-handling repair. Research quality, trained-model evaluation, live teacher/ALFWorld/scale and paper reproduction remain unverified; the checked-in video remains a historical fallback-only receipt. No merges performed by this QA lane.

Copy link
Copy Markdown
Owner Author

Final owner integration and QA freeze: head 3f3ac1b7b1352856ff49c741a503cc96a8091618, tree f7bfb736d07dcb42aa19e8cbf2103ba1e085310a. This is a proper two-parent commit retaining both prior tested OTT head e502a2d and latest owner f64b34c ancestry. Owner delta changes only matrix fairness budget, comparator table label and one regression; no training/model/OTT/CI source changes. All latest owner content is preserved.

At this exact combined tree, 17 targeted tests pass (3 matrix/evidence tests and 14 loss tests including real installed OTT values/gradients and failures); Ruff passes, pyright 0 errors/7 optional-import warnings, and source integrity/full media decode pass. Prior full-suite 53 tests and real OTT-enabled two-step optimizer pass belong to ancestor e502a2d; they are not relabeled as full-suite runs on this new head. New hosted CI 38625132872 is running. Merge HOLD until it passes and the shared coordinator confirms the owner freeze. No further source writes planned by this QA lane.

@jadenfix jadenfix changed the title Fix installed OTT solver dispatch and stop hiding solver failures Integrate fixture CTS workflow with verified OTT solver contract Oct 14, 2026
@jadenfix
jadenfix changed the base branch from codex/fixture-rl-demo-20261013 to main October 14, 2026 19:41
@jadenfix
jadenfix merged commit cdc2145 into main Oct 14, 2026
2 checks passed
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