Repository navigation
fix: correct EMA RVQ exports and verify trained-model parity - #75
Merged
Merged
Conversation
apage224
marked this pull request as ready for review
October 6, 2026 00:46
Contributor
Author
|
Integrated the updated prerequisite and resolved the stack integration issues. CI now runs on stacked PRs. A fresh independent review of the combined changes found no remaining actionable findings and independently passed 16 targeted tests. At |
apage224
changed the base branch from
issue-64-multiprecision-encoder-release
to
main
October 6, 2026 11:48
9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
EMA RVQ exports included embedding-sum accumulators as additional codebooks. Two-level models consequently advertised four levels, doubling table storage and fixed-width token payloads while changing reconstruction. This repair exports only the trained codebooks and checks deployment against an independently restored training quantizer.
Closes #74.
Prerequisite #65 is merged. This PR now targets
mainand includes its merge commit without changing the reviewed code. Fresh CI and documentation checks passed against the updated base atd6359e7. The integration commit has the same source tree as the independently reviewedcd01a0c.Changes
maintainer-notes/rvq-export-audit.md.Validation
d6359e7.Compatibility and scope
Older RVQ packages without independent training references require re-export for strict validation. Corrected level counts change packet interpretation; old packets remain bound to their original bundle. Audit artifacts are git-ignored and unpublished. The audit establishes deployment parity, not a new physiological-quality result; two-stage entropy priors were not requalified.