sgl-project/sglang · 作者 alphabetc1 · 合并时间 2026-03-22 12:54
utils/common.py 中累积了多个没有调用者的函数，例如 get_float_env_var、decode_video_base64 等，这些函数在之前的 PR（如 #14501、#5888）中被重构或移除调用点后成为死代码。移除它们可以减少代码噪音，保持模块的简洁性和可维护性，避免未来维护时的混淆。
该 PR 变更简单直接，适合快速浏览以了解代码清理的最佳实践。对于新贡献者，可以学习如何识别和移除死代码；无需精读，除非关注代码维护流程。
review 中仅有一个 bot 评论（gemini-code-assist[bot]）表示支持，认为移除死代码是好的重构实践，变更直接且无问题。没有其他讨论或争议，变更被迅速接受。
在唯一变更文件 python/sglang/srt/utils/common.py 中，删除了以下未使用的符号：函数 get_float_env_var、decode_video_base64、pytorch_profile、dump_to_file、maybe_torch_compile、is_triton_3，变量 step_counter，以及相关导入语句 from torch.profiler import ...。通过全仓库 grep 验证这些符号在其他 *.py 文件中无引用。
| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
python/sglang/srt/utils/common.py |
sglang/srt/utils | modified | 4.0 |
分析完成后，这里会展示 LLM 生成的相对完整源码片段和详细注释。
gemini-code-assist[bot] 评论认为移除死代码是好的重构实践，变更直接且无问题。
结论：变更被接受，无争议。 · 已解决
风险极低：删除的是已验证无调用者的死代码，不会影响现有功能。唯一潜在风险是如果未来有代码错误地引用了这些符号，但已通过全仓库 grep 验证无引用，确保了安全性。无回归、性能、安全或兼容性问题。
对用户和系统无直接影响，仅提升代码库的整洁度。对团队来说，减少了不必要的代码维护，提高了模块的可读性和可维护性，影响范围限于 utils/common.py 模块。
当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。
本 PR 移除了 utils/common.py 中的死代码，包括多个未使用的函数和导入，以减少代码噪音并提升可维护性。变更已验证安全，无任何功能影响，属于常规维护性质。
sglang/srt/utils/common.py 模块中累积了多个在先前 PR 中失去调用者的函数，例如 get_float_env_var 和 decode_video_base64，这些代码成为死代码，增加了维护负担。移除它们可以保持模块精简，避免未来开发时的混淆，并遵循代码清洁的最佳实践。
在 python/sglang/srt/utils/common.py 中，删除了以下未使用的符号：
| 符号 | 用途 | 移除原因 |
|---|---|---|
get_float_env_var |
环境变量处理 | 已迁移到 environ.py (#14501) |
decode_video_base64 |
视频解码 | 被 load_video 替代 (#5888) |
pytorch_profile 和 step_counter |
性能分析 | 分析路径已移除 (#1652) |
dump_to_file |
数据转储 | 被 tensor dump hook 替代 (#10566) |
maybe_torch_compile 和 is_triton_3 |
Torch 编译 | 直接使用 torch.compile (#3039) |
同时移除了未使用的导入 from torch.profiler import ...。通过全仓库 grep 验证无残留引用。
review 中仅有一个 bot 评论支持此变更：
"This pull request removes several unused functions and variables from
utils/common.py, which is a good refactoring practice to reduce code noise and improve maintainability."
无其他讨论或争议，变更被迅速接受。
风险极低：删除的是已验证无调用者的死代码，不会影响现有功能。已通过全仓库 grep 验证无引用，确保安全。对用户无感知，仅提升代码库质量；对团队减少维护开销，提高模块可读性。
本 PR 清理的死代码源于以下历史 PR 的变更，显示了代码库的持续演进：
environ.py，使 get_float_env_var 成为死代码。load_video 替代 decode_video_base64。pytorch_profile 和 step_counter 失效。dump_to_file。torch.compile，废弃 maybe_torch_compile 和 is_triton_3。这些关联 PR 共同促进了代码现代化，本 PR 是清理遗留代码的后续步骤。
参与讨论