mirror of
https://github.com/garrytan/gbrain.git
synced 2026-08-14 00:48:18 +00:00
fix(test): recipe-ollama-dims pins a truncated width — native-width requests now omit the param (#3699)
Master's #1072 test asserted bare qwen3-embedding@1024 threads dimensions:1024; wave PR #3699 suppresses the param when the request equals the native width (1024 for the bare id), so the same input now correctly returns undefined — pinned by dims-qwen3-native.test.ts. Moved this test to 512 to keep its actual intent (bare id recognized as Matryoshka-capable) without contradicting the suppression. Third composition defect of the wave; caught by CI shard 8.
This commit is contained in:
@@ -28,8 +28,12 @@ describe('dims: ollama Matryoshka models', () => {
|
||||
});
|
||||
|
||||
test('bare qwen3-embedding (no quant tag) also recognized', () => {
|
||||
expect(dimsProviderOptions('openai-compatible', 'qwen3-embedding', 1024))
|
||||
.toEqual({ openaiCompatible: { dimensions: 1024 } });
|
||||
// 512 not 1024: bare qwen3-embedding's native width is 1024, and a
|
||||
// request equal to the native width omits the param entirely (fixed-dim
|
||||
// vLLM backends 400 on it) — pinned by dims-qwen3-native.test.ts. This
|
||||
// test's job is only that the bare id is recognized as Matryoshka-capable.
|
||||
expect(dimsProviderOptions('openai-compatible', 'qwen3-embedding', 512))
|
||||
.toEqual({ openaiCompatible: { dimensions: 512 } });
|
||||
});
|
||||
|
||||
test('unrelated openai-compat model returns undefined (regression guard)', () => {
|
||||
|
||||
Reference in New Issue
Block a user