openLiBing可接管PR, PR详情链接:
https://www.openlibing.com/apps/transitionPrDetail?owner=Ascend&repo=pytorch&number=40002&platform=gitcode&codeHostingPlatformFlag=gitcode

compile

Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here

PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

💡 Tip:

• Committer can comment /approve or /lgtm
• Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

hhz0, thanks for your pull request. All authors of the commits have signed the CLA. 👍

当前仓库存在以下 保护分支 ：

评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支（创建同步 PR）：
a) 如果当前 PR 是 Open 状态，同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged，将立即执行同步操作

注意：

1. /sync 命令可以指定同步到多个分支，仅最后一个 /sync 命令生效
2. 如果创建的同步 PR 不正确，可通过向同步 PR 的源分支提交轻量级 PR 完善，或使用 /close 命令关闭

• compile、compile_inductor、compile_torchair : 运行流水线
• retry : 重试流水线所有失败子任务
• retry <任务名> : 仅重试指定失败子任务
• stop : 停止流水线

ascend docs pipeline is running...

ascend docs pipeline is running...

✅ 跳过 docs ci 检查，没有需要检查的文档文件

✅ 跳过 docs ci 检查，没有需要检查的文档文件

变更摘要

此 PR 修复了 profiler 离线解析多卡场景下，当 max_process_number 小于总 ascend_pt 数量时，进程池复用导致 ProfilerConfig 的 _is_load 标记未能重置、配置无法被更新的问题。核心改动是在 ProfilerConfig 中新增 reset_is_load() 方法，并在 ProfilingParser 解析流程中加载配置前调用该方法重置标记位，确保每次解析都能正确加载 rank 信息。

主要改动

• 新增 ProfilerConfig.reset_is_load() 方法：在 _profiler_config.py 中为 ProfilerConfig 类添加 reset_is_load() 方法，将 _is_load 标记重置为 False，使进程复用时能够重新加载配置。
• 解析流程中调用 reset_is_load()：在 _profiling_parser.py 的 ProfilingParser 初始化中，于 ProfilerConfig().load_info() 之前增加 ProfilerConfig().reset_is_load() 调用，确保每次解析前重置加载标记。
• 新增单元测试：在 test_profiler_config.py 中添加 test_reset_is_load_will_reset_is_load_to_false 测试用例，验证 reset_is_load() 正确将 _is_load 置为 False 且不影响 _rank_id 等其他字段。

• compile、compile_inductor、compile_torchair : 运行流水线
• retry : 重试流水线所有失败子任务
• retry <任务名> : 仅重试指定失败子任务
• stop : 停止流水线

代码审查

审查总结

我对全部 3 个变更文件逐一进行了审查：

• P0: 0
• P1: 0
• P2: 0
• P3: 1

整体风险评估：低风险。

这个 DIFF 修复了一个真实存在的 bug：当进程池复用时，ProfilerConfig 单例的 _is_load 标记未重置，导致 load_info 直接返回而不重新加载配置，最终导致 DB 文件中的 rank ID 与 profiler_info.json 中的不一致。修复方案正确且精准：在 analyse_profiling_data 中调用 load_info 之前通过 reset_is_load() 清除标记位。新增的 reset_is_load 方法实现简洁，新增的单元测试覆盖了该方法的核心行为。唯一的瑕疵是测试未清理单例状态（P3），对当前测试运行无实际影响。

⚠️ 已识别出整体风险，但无法提取行内评论，请参考整体评估。

/approve

compile

ascend docs pipeline is running...

✅ 跳过 docs ci 检查，没有需要检查的文档文件

• compile、compile_inductor、compile_torchair : 运行流水线
• retry : 重试流水线所有失败子任务
• retry <任务名> : 仅重试指定失败子任务
• stop : 停止流水线

/approve