QVAC-24061 chore: release @qvac/inference 0.20.0 - #4562
opaninakuffo wants to merge 9 commits into
Conversation
License compliance — cleanNo new dependency license findings in this PR. Warn-only (shadow) mode — this check does not block merges yet. Updated automatically by the canonical license compliance workflow. NOTICE presence (advisory)Missing NOTICE (advisory, does not block):
|
Review StatusCurrent Status: ❌ PENDING Pending reviews: Needs 1 Management or Team Lead, and 1 more from Management, Team Lead, or Member. |
QVAC E2E — base run recordedBase: run 35407621343 · suite
All recorded platforms are green — nothing for |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
… prettier" This reverts commit 8bb4c85.
#4406 shipped the edit pipeline with understand/remake; the notes only showed those two.
0194db1
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
lauripiisang
left a comment
There was a problem hiding this comment.
Four changelog issues, all in packages/inference/changelog/0.20.0/.
1. splitMode: 'row' entry names the wrong key and enum for completion models.
breaking.md and CHANGELOG_LLM.md say "modelConfig.splitMode is 'none' or 'layer'".
There are two keys with two enums in src/schemas/llamacpp-config.ts:
llmConfigBaseSchema→'split-mode', now['none', 'layer', 'tensor']embedConfigBaseSchema→splitMode, now['none', 'layer']
As written, someone running 'split-mode': 'tensor' reads this as tensor parallelism
being dropped, and someone hitting the validation error can't find the key they set.
Neither spelling appears in the SDK source or the generated API page, so this entry is
the only public documentation of it. Suggested replacement:
Fabric 10549.1.0 dropped llama.cpp's unused row split.
- Completion models:
modelConfig['split-mode']is'none','layer'or'tensor'.'row'is rejected.- Embedding models:
modelConfig.splitModeis'none'or'layer'.'row'is rejected.
'tensor'on completion models is unaffected.
2. The embed removal needs one clause on what was lost.
Before this cut, the completion description called 'row' "legacy; degrades to 'layer'"
while the embed description called it "tensor parallelism". Please confirm against the
Fabric change which is accurate and say so — embedding models have no 'tensor' to move
to, so "use 'layer'" is either a rename or a downgrade.
3. Parakeet language validation tightening is undocumented.
src/schemas/transcription-config.ts added
/^(|auto|[a-zA-Z]{2,3}(-[a-zA-Z]{2,4})?)$/ to a field that was plain z.string().
Configs that used to load now fail at loadModel — e.g. zh-Hans-CN, or a language
name rather than a code. This belongs in breaking.md.
4. No ABot-World section, though the package exports the API.
src/index.ts exports worldCreateScene and worldStep, and the SDK 0.20.0 notes carry
"ABot-World Sessions" under New APIs. The two changelogs mirror each other section for
section otherwise.
Note (1), (3) and (4) also apply to packages/sdk/changelog/0.20.0/ in #4543, plus
docs/website/content/docs/reference/release-notes/v0.20.x.mdx, and both backmerges.
packages/inference/CHANGELOG.md is currently byte-identical to CHANGELOG_LLM.md —
worth keeping that true.
Separately: CHANGELOG.md still goes 0.20.0 → 0.17.0, while npm has 0.17.1, 0.18.0,
0.18.1, 0.18.2, 0.19.0 and 0.19.1. Since this PR backfills changelog/0.17.0/, adding
one-line stub sections for those six would close the gap for good.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
QVAC E2E —
|
QVAC E2E —
|
QVAC E2E —
|
QVAC E2E —
|
QVAC E2E —
|
🎯 What problem does this PR solve?
Cuts
@qvac/inference0.20.0 from currentmainso the engine can publish before@qvac/sdk0.20.0. After #4338, the SDK published lane installs^0.20.0from npm; this package has to be on the registry first.Metadata only: version, changelog, NOTICE. No SDK bump. No publish.
📝 How does it solve it?
packages/inferenceto0.20.0.4dbb66d8e/--base-version=0.19.1), plus the main-only PRs that sat between the previous.0and that patch (#4010,#4234,#4342,#4384).NOTICE(TranslatePsy AfriSLM, Nemotron 3.5 ASR, JS dep versions).Companion SDK PR #4543 still lockstep-bumps inference on the SDK changelog branch. That bump belongs here; do not merge #4543 until this is published and the SDK PR drops the in-tree inference version change.
Do not add
skip-sdk-pod-checks.🧪 How was it tested?
--package=inference --base-commit=4dbb66d8e --base-version=0.19.1prettier --checkon inference changelog markdowngenerate-notice.js inference(models + JS)💥 Breaking Changes
Documented in
packages/inference/changelog/0.20.0/breaking.md:clip_on_cpu/vae_on_cpu/control_net_cpu→params_backend/backendkvCachesplitMode: 'row'rejected; use'layer'🔌 API Changes
TurboVec
createVectorIndex/loadVectorIndex, MiniMax-H3video(), Parakeet Nemotron, AudioGen understand/remake/audioEdit, TTS sampleRate/cancel, tool grammartool_choice/toolErrors. Seechangelog/0.20.0/api.md.📦 Models
16 added, 4 removed. Full lists in
changelog/0.20.0/models.md.Added models
Removed models