Skip to content

Commit 3474656

Browse files
committed
test: Update ChangeBeamWidth test to use same return sequence count as beam width
- Modified the ChangeBeamWidth test to use the same number of return sequences as the beam width, improving test consistency. Signed-off-by: Robin Kobus <[email protected]>
1 parent c5cc688 commit 3474656

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

cpp/tests/e2e_tests/executor/executorTest.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2467,8 +2467,10 @@ void doTokenComparisonChangeBeamWidth(bool enableReuse, SizeType32 maxWaitMs)
24672467
beamResult.contextLogitsFile = resultsPath / PathUtil::FP16_PLUGIN_PACKED_PAGED_CONTEXT_LOGITS_FILE();
24682468
beamResult.genLogitsFile = resultsPath / PathUtil::FP16_PLUGIN_PACKED_PAGED_GENERATION_LOGITS_FILE();
24692469

2470+
auto const numReturnSequences = beamWidth;
2471+
24702472
runTest(executor, inputPath, modelIds, flakyTestInfo, streaming, vocabSizePadded, beamResult, outConfig,
2471-
isSpeculativeDecoding, maxWaitMs, false, 1, false, 1);
2473+
isSpeculativeDecoding, maxWaitMs, false, numReturnSequences, false, 1);
24722474
}
24732475
}
24742476

0 commit comments

Comments
 (0)