Skip to content

fix(cif): handle mixed zero-fire batches with CPU/ONNX regression CI - #3857

Merged
LauraGPT merged 2 commits into
mainfrom
codex/pr3854-cif-boundary-regressions-20261107
Nov 7, 2026
Merged

LauraGPT merged 2 commits into
mainfrom
codex/pr3854-cif-boundary-regressions-20261107

Conversation

@LauraGPT

@LauraGPT LauraGPT commented Nov 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

Integrates #3854 while preserving its original commit and authorship. Empty CIF samples have no output frame to reset; filtering their reset offsets avoids out-of-bounds indexing when a mixed batch ends in zero-fire samples. Both eager and export functions are covered. Existing rounding and thresholds are unchanged.

Adds read-only exact-head CPU CI with required ONNX dependencies and a zero-skip assertion. The signed companion commit synchronizes retained main content without rewriting contributor history.

Maintainer validation

  • Same submitted suite on unchanged main: 45 errors out of 98 subcases across 6 methods. Candidate: 98/98 subcases, 6/6 methods, zero skips, including 14 dynamic ONNX Runtime CPU batch patterns.
  • Actual NVIDIA L20Z: 34 FP32 analytic cases, 2551 CUDA kernels, independent sequential frame oracle, finite gradients, and real CifPredictorV2/export callers. Exact outputs, checkpoint hash, trace and source hashes are retained.
  • The intended pytest CI command additionally passed all 6 methods with zero skips. Workflow structure and shell syntax validated; new exact-head hosted CI remains required before merge.
  • Private HOME/cache and kernel network isolation throughout. No new pretrained weights, training, production changes, BF16 claim, full Paraformer export, recognition-quality benchmark or full-suite claim.

This is not a new release. Nano remains deferred.

BillVan1945 and others added 2 commits November 8, 2026 01:32
Preserve the original PR #3854 commit. Synchronize retained main content to the exact reviewed integration tree and add fail-closed CPU/ONNX regression CI. No history rewrite, pretrained download, or protection bypass.

Signed-off-by: LauraGPT <zhifu.gzf@alibaba-inc.com>
@LauraGPT
LauraGPT merged commit b5d0fd9 into main Nov 7, 2026
16 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.

2 participants