Remove C128 req slot allocation reset - #620
zhangxiaolei123456 merged 1 commit into
Conversation
There was a problem hiding this comment.
Code Review
The pull request removes the _clear_c128_req_state_for_new_slots helper function and its usage from alloc_req_slots in common.py. In deepseek_v4_memory_pool.py, the batch clearing method clear_c128_req_states is removed, and the single-slot clearing method clear_c128_req_state is simplified to perform direct in-place tensor operations. Additionally, the associated unit test file has been deleted. I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
f5c0073
into
sync-pr-601-to-main
Motivation
Modifications
Accuracy Tests
Speed Tests and Profiling
Checklist
Review and Merge Process
/tag-and-rerun-ci,/tag-run-ci-label,/rerun-failed-ciCI States
Latest PR Test (Base): ❌ Run #28293208004
Latest PR Test (Extra): ❌ Run #28293207904