已合并
[Bugfix][log]Unify MindIE SD logging and improve diagnostics #328
guowenna1创建于 6月3日
[Bugfix][log]Unify MindIE SD logging and improve diagnostics #328
已合并
guowenna1创建于 6月3日
guowenna1成员
6月3日

Which issue(s) this PR fixes or accomplishes

Fix part of https://gitcode.com/Ascend/MindIE-SD/issues/158

Purpose

本 PR 修复 MindIE SD 日志问题,主要包括:

  1. 统一 MindIE SD Python 模块日志出口,避免 compilation、share_memory 等模块直接使用标准库 logging.getLogger(__name__),导致日志格式、落盘路径、过滤级别和开关行为不一致。
  2. 优化日志模块默认输出格式,确保默认与 verbose 模式均包含 MindIE SD 组件标识。
  3. 精简默认 INFO 场景日志,将正常流程、调试态信息降级为 DEBUG,避免默认运行场景产生不必要日志。
  4. 增强 WARNING/ERROR 日志内容,补充问题描述、可能根因、参数期望值/实际值和进一步排查建议。
  5. 修复 pre-commit 暴露的日志格式、pylint、bandit、typos 等问题,包括日志参数数量不匹配、拼写错误、动态 API 静态检查误报标注和 EPLB scheduler 嵌套层级过深问题。eplb_scheduler、greedy_algorithm中此类改动较多,多是形式改动,无实际影响。

Test Plan

  1. 执行 pre-commit 全量检查,覆盖 ruff、pylint、bandit、typos 等静态质量门禁。
  2. 执行 Python 编译检查,确认本次修改未引入语法错误。
  3. 执行 git diff 空白检查,确认无行尾空白、格式污染。
  4. 白盒检查 mindiesd 正式代码中默认 INFO 日志是否清理完成。
  5. 白盒检查除日志模块本体外,是否仍存在直接使用标准库 logging.getLogger(__name__) 的模块。

Test Report

已执行并通过:
image.png

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 guowenna1 的贡献)
Gguowenna1成员
6月3日 创建了 pull request,commit 1b6aedc7
ascend-robotascend-robot成员
6月3日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
6月3日 评论:

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

module lgtm status approve status
repo-Ascend/MindIE-SD mazhixin00_00, changetheway (2/2) mazhixin00_00 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
6月3日 添加了label:ci-pipeline-running
Gguowenna1成员
6月3日 修改了pull request 的描述
此处折叠了82条消息 查看更多
ascend-robot
ascend-robot成员
6月3日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
KexiongYu

likedislike
ascend-robotascend-robot成员
6月3日 添加了label:lgtm
lanwangli成员
6月3日 评论:

/merge

likedislike
ascend-robotascend-robot成员
6月3日 添加了label:keeper_approved
ascend-robotascend-robot成员
6月3日 合入了pull request