Repository navigation
fix edge case for repetition count threshold - #28311
SergeySklyarov wants to merge 1 commit into
Conversation
|
Hi @SergeySklyarov, thanks for your contribution! Per our contribution guidelines, the automated PR checker found the following issue(s) that need your attention:
Please note that maintainers reserve the right to make final decisions on PRs. If you believe there is a mistake, please comment below. |
|
Hi @ggerganov could you please check this PR when you have time? |
|
Hi @ggerganov, following up on this PR. It has no merge conflicts and #28279 has already been merged. Could you please review it and approve the pending CI workflows when convenient? |
Overview
Fixes an edge case where a repetition count of exactly 2000 was rejected by the parser, despite values above 2000 being handled as unbounded. Repetition counts equal to the threshold of 2000 are now accepted, while nested repetitions exceeding the expansion limit remain restricted.
Additional information
This PR revisits the fix from #27177 (which stalled in Draft due to template and AI disclosure requirements) and presents the solution along with tests.
Requirements