已关闭
[Bug-Report|缺陷反馈]: `auto_sync_log=None` 仍生成编译器日志参数 #80
KaranocaVe创建于  10月3日关闭于  14 天前
KaranocaVe
KaranocaVe
10月3日 创建

[Bug-Report|缺陷反馈]: auto_sync_log=None 仍生成编译器日志参数

问题描述

CompileOptions.auto_sync_log 是可选路径,但在默认开启自动同步时传入 None,生成的毕昇命令仍包含 -cce-auto-sync-log=None,把字面量 None 当作日志路径。

环境

cann/pyasc@076338c4a626090446413d98e0af835948c2e69c。当前证据覆盖 Host 侧编译命令构造,未运行毕昇编译或 NPU。

重现步骤

from asc.runtime.compiler import CompileOptions

options = CompileOptions(auto_sync_log=None)

自动同步开启时,基于该选项生成的毕昇命令包含 -cce-auto-sync-log=None。

预期结果

None 和空字符串都应省略日志参数;非空路径应原样传入。

日志 / 截图

-cce-auto-sync-log=None

关联 PR: https://gitcode.com/cann/pyasc/merge_requests/202

likedislike
KaranocaVeKaranocaVe
10月3日 关联了pull request:`auto_sync_log=None` 仍生成编译器日志参数
KaranocaVeKaranocaVe
10月3日 修改了issue 的描述
kong0808成员
25 天前 评论:

您好,感谢关注与贡献!若该issue的处理PR已验证并提交,建议先执行compile构建,确保CI流水线通过(打上系统标签ci-pipeline-passed)。
待CI流水线通过后,麻烦您及时艾特Committer(@qin437231,@kong0808,@wuzhenqing,@vla5924,@hyolee)或模块特性田主(@vla5924)进行检视。
如果有检视意见,麻烦先完成检视意见修改。待检视通过后,我们会有Committer进行加分合入~

likedislike
Lli-yuanjie-da成员
23 天前 将 KaranocaVe 设为负责人
KaranocaVe
KaranocaVe
14 天前 评论:

@li-yuanjie-da 您好,这个 Issue 对应的修复已经合入:https://gitcode.com/cann/pyasc/merge_requests/202

开发者计划反馈,Issue 需要有 Accepted 标签才能计入积分,仅关闭或标记 resolved 还不够。目前这条 Issue 尚未标记 Accepted,麻烦您结合问题和已合入的修复复核一下;确认符合受理条件后,帮忙补上 Accepted 标签。

感谢您的支持!

likedislike
li-yuanjie-da成员
14 天前 评论:

感谢反馈,问题已经确认并修复。

原来的判断只排除了空字符串,auto_sync_log为None时仍会进入拼接逻辑,最终把字面量None传给毕昇,生成-cce-auto-sync-log=None这样的无效参数。

我核对了 https://gitcode.com/cann/pyasc/merge_requests/202 合入master后的最终代码。现在None和空字符串都会省略自动同步日志参数,只有传入非空路径时才会生成对应的日志参数。新增测试覆盖了None、空字符串和非空路径三种场景,文档也同步说明了这两种空值都表示不落盘。从合入内容和测试覆盖看,反馈的问题已经解决,没有发现明显的遗留问题。

这个问题已按贡献流程复核,我先关闭issue。如果后续使用中自动同步日志参数仍有异常,欢迎补充新的复现信息,我们会继续跟进。

likedislike
Lli-yuanjie-da成员
14 天前 issue状态由 进行中 改变为 已完成
Lli-yuanjie-da成员
14 天前 关闭了 issue
CANN-robotCANN-robot成员
14 天前 添加了label:resolved