已合并
FE Log Rectification #4482
李颜戎创建于 12 天前
FE Log Rectification #4482
已合并
李颜戎创建于 12 天前
李颜戎
李颜戎
12 天前

Pull Request

描述

EcoC15版本日志易用性整改(FE责任田)

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:

  1. ut st通过

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 李颜戎 的贡献)
李颜戎李颜戎
12 天前 添加了label:enhancement
李颜戎李颜戎
12 天前 创建了 pull request,commit c93f2763
atomgit-bot
atomgit-bot
12 天前 评论:

变更摘要

该 PR 是一次纯日志修正(FE Log Rectification),不涉及功能逻辑改动,仅针对编译器中 FE_LOGEFE_LOGWFE_LOGDGELOGEKLOGE 及测试打印等日志文本进行纠错与补全:修正拼写错误(如 vecorvectorconctrlcontrolBuidlerBuilderpirvateprivateQeq_scaleReq_scale)、更正表述不准确的内容(如溢出告警中 "addition"→"multiplication"、空日志补全为带节点信息的报错)、补齐缺失的格式化参数,并将测试用例中的输出文案一并修正,主要涉及 ConcatTileFusionPassOpAxisUpdateDescGenerateCMOInvalidTaskGenerateCMOPrefetchTaskFftsTaskBuilderFeGraphUtilsOpSliceUtil 等模块。

主要改动

  • 补齐空日志并补充上下文信息: fe_graph_utils.ccFeGraphUtils::GetNextInAnchorsOfSubNetOutput 将原先空的 FE_LOGE("") 改为输出节点名、属性名与输入索引的完整错误日志;tbe_json_parse_impl.ccParseFatbin 修正日志格式串,补上缺失的 tiling_key_num 参数(%zu)。
  • 修正溢出告警措辞: op_axis_update_desc.ccGetFractalZNewShape 将两处溢出告警中的 "addition"(加法)更正为 "multiplication"(乘法),使其与实际执行的乘法运算一致。
  • 修正日志拼写与术语错误: concat_tile_fusion_pass.cc 修正 vecor_core_numconctrl 拼写;ffts_task_builder.cc 修正 FFTSPlusTaskBuidlerFFTSPlusTaskBuilderpyobj_assemble_utils.cc 修正 pirvateprivatefusion_quant_util_impl.cc 修正 Qeq_scaleReq_scalegenerate_cmo_invalid_task.ccgenerate_cmo_prefetch_task.cc 将 "for mode" 更正为 "for node"。
  • 更正表述不准确的告警文案: aclnn_node_converter.ccBuildOpInputTensors 将错误日志由 "is not null"(否定逻辑颠倒)改为 "is null";atomic_ffts_update_kernel.ccFFTSCalcAtomicOutputShapeSize 将 "is over then slice size" 改为 "exceeds slice size"。
  • 丰富切片不支持日志信息: op_slice_util.ccFillupResizeSliceInfoFillupScatterSliceInfoFillupSegmentSliceInfo 移除 (void)op_desc_ptr 空转语句,日志中区分 resize/scatter/segment 切片模式并补充节点名;测试文件(如 expand_dims_ut.cctbe_ub_fusion_ut.ccte_fusion_base.ccte_fusion_test.ccfusion_engine_dsa_graph_optimizer_stest.cc)同步修正输出中的拼写与中文标点问题。
likedislike
atomgit-bot
atomgit-bot
12 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
12 天前 添加了label:cann-cla/yes
此处折叠了198条消息 查看更多
zhanj
zhanj成员
9 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
9 天前 添加了label:lgtm
张德鹏成员
8 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:approved
CANN-robotCANN-robot成员
8 天前 合入了pull request