执行摘要
此PR修复了sglang仓库中多模态DeepStack嵌入路径的一个索引对齐错误，通过添加简单的else分支确保模态列表严格对齐，避免下游索引越界问题，提升混合模态输入的鲁棒性。变更影响有限，但解决了潜在的系统崩溃风险。
功能与动机
动机源于多模态DeepStack嵌入路径中的簿记问题：原逻辑在 use_deepstack.get(modality, None) 未启用或 embedding is None 时，deepstack_embeddings 列表未附加元素，导致其与 modalities、embeddings 和 masks 列表长度不一致，引发下游索引错误。PR目标是通过显式对齐列表，使DeepStack路径对混合模态输入更健壮。
实现拆解
实现仅修改 python/sglang/srt/managers/mm_utils.py 文件的 embed_mm_inputs 函数。关键代码变更如下：
if use_deepstack.get(modality, None) and embedding is not None:
embedding, deepstack_embedding = multimodal_model.separate_deepstack_embeds(embedding)
deepstack_embeddings += [deepstack_embedding]
else:
deepstack_embeddings += [None] # 新增分支
此改动确保在所有模态分支中，
deepstack_embeddings 列表与其他列表一对一对齐，而下游逻辑保持不变。
评论区精华
review讨论较少，主要集中在流程管理：
- hnyls2002 请求 yuan-luo 进行代码审查，并询问 xq25478 修复lint问题。
- 无深入技术交锋，最终通过CI测试后合并，表明变更被团队接受。
风险与影响
风险分析：
- 低风险：变更仅涉及Python列表操作，不触及核心计算或性能。
- 潜在风险：下游代码若依赖
deepstack_embeddings 中非None值，可能引入bug；但PR描述指出下游已处理None。
- 测试覆盖：PR未添加新测试，依赖现有回归测试，可能存在未覆盖边缘情况。
影响评估：
- 影响范围：限于多模态DeepStack嵌入路径，对混合模态输入场景。
- 影响程度：修复索引错误，避免系统异常，提升内部一致性，对用户透明。
关联脉络
从同仓库近期历史PR看，本PR属于一系列bugfix和优化的一部分，但无直接关联PR修改相同文件或功能。例如，PR 21664 修复多模态生成模型，但涉及不同模块。本PR聚焦于多模态管理工具中的对齐问题，反映了团队对系统鲁棒性的持续改进。
参与讨论