执行摘要
该PR修复了多阶段去噪（multi-stage denoising）场景下性能分析器无法正确记录步骤时序的问题，通过引入record_as_step标志和重构步骤记录逻辑，确保去噪步骤的执行时间能被准确记录。变更涉及多个去噪阶段文件和核心性能日志工具，属于针对特定场景的bug修复，对系统性能监控有积极影响。
功能与动机
在多阶段去噪场景中，某些去噪阶段的名称可能不以denoising_step_前缀开头，导致原性能分析器（StageProfiler）无法将其识别为去噪步骤，从而漏记录执行时间。PR通过显式标记这些阶段为步骤，修复了性能数据缺失问题。从代码变更看，原逻辑依赖"denoising_step_" in self.stage_name的字符串包含匹配，现改为更精确的前缀匹配结合显式标志。
实现拆解
实现分为两个层面：
- 调用方适配：在四个去噪阶段文件（
denoising.py、denoising_av.py、denoising_dmd.py、mova.py）的forward方法中，调用StageProfiler时添加record_as_step=True参数。
- 核心工具重构：在
perf_logger.py中：
- 新增
record_as_step实例变量和_should_record_as_step()方法
- 将
record_steps(index, duration_s)重命名为record_step(duration_s)，移除index参数
- 修改
__enter__和__exit__中的同步逻辑，使用_should_record_as_step()判断
- 修改步骤记录逻辑，同样使用新判断条件
关键代码片段：
def _should_record_as_step(self) -> bool:
return self.record_as_step or self.stage_name.startswith("denoising_step_")
评论区精华
review中唯一评论来自gemini-code-assist[bot]，指出一个潜在行为变更：
"The logic in _should_record_as_step uses startswith("denoising_step_"). The previous implementation used "denoising_step_" in self.stage_name. While startswith is generally more precise, this is a slight change in behavior for any legacy custom stage names that might have contained but not started with that string."
该评论提醒从包含匹配改为前缀匹配可能影响历史自定义阶段名称的自动归类。作者未回复此评论，最终代码仍采用startswith方案，可能认为这是可接受的精确化改进，或此类自定义名称不存在。
风险与影响
风险：
- 行为变更风险：若存在历史自定义阶段名称包含但不以
denoising_step_开头（如custom_denoising_step_1），将不再被自动记录为步骤，导致性能数据缺失。
- 参数传递风险：新增
record_as_step参数需确保在所有多阶段去噪场景正确传递，否则仍可能漏记录。
影响：
- 正面：多阶段去噪的性能分析数据更准确，有助于调试优化。
- 中性：变更仅影响性能监控记录，不影响核心推理逻辑。
- 潜在负面：团队需注意行为变更可能影响现有自定义阶段名称的监控。
关联脉络
从近期历史PR看，该PR属于diffusion和multimodal相关改进的一部分。虽然未直接关联其他PR，但体现了对复杂生成场景（多阶段去噪）性能监控的持续完善。同仓库近期有多个涉及性能分析、测试和扩散模型的PR（如#21740、#21408），显示团队正加强对多模态生成和扩散模型的支持与优化。
参与讨论