已合并
bugfix: aicpu交付件导出增加deviceId,保持和python解析结果一致 #497
wangzixuan创建于 21 小时前
bugfix: aicpu交付件导出增加deviceId,保持和python解析结果一致 #497
已合并
wangzixuan创建于 21 小时前
wangzixuan成员
21 小时前

PR 合入模板

注:经过自检不涉及的可标注“不涉及”或直接打勾,特殊情况请文字备注。不符合规范的 PR 不允许合入,请(后备)commit 注意。


1. 修改描述

  • 修改原因:

  • 1、aicpu导出新增c化流程,但是表头未与python侧交付件对齐,缺少deviceId字段

  • 2、原导出 aicpu的timestamp字段未加\t,导致csv中的时间被按数字处理,精度丢失。

  • 修改内容:

  • 1、正确适配aicpu导出c化流程,增加deviceId

  • 2、统一整改csv相关的导出类的\t使用,规范使用逻辑。


2. 功能验证


3. 分支合并要求


3. 代码检视

  • 要求:

    • 合入代码超过 200 行,需三人以上会议检视。
    • 检视密度≥1个/100行。
    • 检视缺陷密度未达要求需提供说明。
    • 大于 1000 行代码原则上不允许合入,需进行备案。
  • 检视意见数:____ 条 (请填写本次检视的意见总数,用于commit合入前审视)


4. 安全自检

Python、C++

C++


5. 变更知会


likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wangzixuan 的贡献)
Wwangzixuan成员
21 小时前 创建了 pull request,commit 56908e61
atomgit-bot
atomgit-bot
21 小时前 评论:

变更摘要

本 PR 为 aicpu 交付件导出增加 deviceId,使 CSV 导出结果与 python 侧解析结果保持一致。核心改动是在 AicpuDpDataAicpuMiData 数据实体中新增 deviceId 字段,在 AicpuProcessor 数据加载阶段将单设备处理时的 deviceId 写入该字段,并由 AicpuAssembler 在导出 aicpu/dp/mi 三类 CSV 时将该字段作为首列 Device_id 输出,同时同步更新了对应 UT 用例。

主要改动

  • 数据实体新增 deviceId 字段:在 aicpu_summary_data.hAicpuDpDataAicpuMiData 结构体中各新增 uint16_t deviceId = UINT16_MAX; 字段,为导出提供数据来源。
  • 加载流程填充 deviceIdAicpuProcessor::LoadDpDataLoadMiData 的函数签名增加 uint16_t deviceId 入参(aicpu_processor.h 声明同步修改),在解析数据行时执行 data.deviceId = deviceIdProcessSingleDevice 调用处将本机 deviceId 传入。
  • CSV 导出增加 Device_id 首列AicpuAssembler::WriteAicpuCsvWriteDpCsvWriteMiCsvheaders_ 均新增 "Device_id" 作为第一列,对应数据行每行以 std::to_string(item.deviceId) 开头输出。
  • UT 用例同步更新aicpu_assembler_utest.cpp 更新三类 CSV 的表头与预期行内容(新增 deviceId 列),aicpu_processor_utest.cpp 新增对加载结果 deviceId 取值的断言。
likedislike
不准确?
AtlasAccount
AtlasAccount成员
21 小时前 评论:
atomgit-bot
atomgit-bot
21 小时前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
ascend-robotascend-robot成员
21 小时前 添加了label:ascend-cla/yes
此处折叠了49条消息 查看更多
panyj1993
panyj1993成员
1 小时前 评论:

/approve

likedislike
ascend-robotascend-robot成员
1 小时前 添加了label:approved
ascend-robotascend-robot成员
1 小时前 合入了pull request
ascend-robot
ascend-robot成员
1 小时前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
ascend-robot
ascend-robot成员
1 小时前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike