已关闭
aclnnUnique和aclnnUnique2接口相关文件误放在uniqueConsecutive算子目录下 #5427
wkx12138创建于  26 天前关闭于  13 天前
wkx12138成员
26 天前 创建

aclnnUnique和aclnnUnique2为多个L0拼接而成,无同名L0接口,但是aclnnUnique和aclnnUnique2接口相关文件都放在了uniqueConsecutive算子目录下,se建议应该新增一个单独的目录。

likedislike
Wwkx12138成员
26 天前 将 yourealize 设为负责人
yuning_chen
yuning_chen成员
25 天前 评论:

您好,目前 @yourealize 正在处理

likedislike
yuning_chenyuning_chen成员
25 天前 将 wkx12138 设为负责人
陈展熹成员
17 天前 评论:

不负责此接口

likedislike
陈陈展熹成员
17 天前 添加了label:wait-feedback
wkx12138成员
17 天前 评论:

根据提交记录来看,aclnnUnique和aclnnUnique2接口是 @yourealize 将此接口迁移到uniqueConsecutive算子目录下,希望相关责任人可以恢复uniqueConsecutive算子原有的目录结构

likedislike
Wwkx12138成员
17 天前 删除了label:wait-feedback
陈展熹成员
17 天前 评论:

image.png

该aclnn接口profiling实测数据只包含uniqueConsecutive算子调用。 且分析aclnnUnique/aclnnUnique2 源码实现。该算子调用的接口包括UniqueCommon::FlattenAndSort、l0op::UniqueConsecutive、UniqueCommon::ComputeInverseIndices、l0op::UniqueWithCountsAndSorting。

其中,UniqueCommon::FlattenAndSort、UniqueCommon::ComputeInverseIndices接口位于uniqueConsecutive目录下,l0op::UniqueConsecutive为该算子l0接口。UniqueWithCountsAndSorting为aicpu算子,对应uniqueConsecutive算子全局排序去重版本,为aicore不支持场景的兜底实现。

aclnnUnique/aclnnUnique2 接口放置在 uniqueConsecutive 算子目录下符合当前ops-nn仓目录规范。
若要将aclnnUnique/aclnnUnique2接口迁移至独立目录,需考虑通过ops-nn仓双周例会评审

likedislike
陈陈展熹成员
17 天前 添加了label:wait-feedback
wkx12138成员
16 天前 评论:

aclnnunique/aclnnunique2接口内调用多个算子的l0接口,并不存在只包含uniqueConsecutive算子的情况。分析aclnnUnique/aclnnUnique2 源码实现该算子调用的接口包括UniqueCommon::FlattenAndSort、l0op::UniqueConsecutive、UniqueCommon::ComputeInverseIndices、l0op::UniqueWithCountsAndSorting。
其中,UniqueCommon::FlattenAndSort、UniqueCommon::ComputeInverseIndices接口在unique_common.cpp文件中,该文件在@yourealize 迁移aclnnunique/aclnnunique2接口之前也并不位于uniqueConsecutive目录下,https://gitcode.com/cann/ops-nn/pull/8848/diffs 是迁移接口的pr。@yourealize在未告知我的情况下将aclnnunique/aclnnunique2接口以及相关文件迁移到uniqueConsecutive算子目录下,我只要求恢复我uniqueConsecutive算子原有的目录结构,至于后续aclnnunique/aclnnunique2接口应该怎么处理与我无关,我并不是aclnnunique/aclnnunique2接口的负责人。

likedislike
Wwkx12138成员
16 天前 删除了label:wait-feedback
陈陈展熹成员
16 天前 关联了pull request:refactor(unique): 将全局去重接口迁移到独立目录
CANN-robotCANN-robot成员
13 天前 关闭了 issue
CANN-robotCANN-robot成员
13 天前 添加了label:resolved