Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

部分算子的参数命名不规范 #1153

Open
zrr1999 opened this issue Jan 17, 2023 · 3 comments
Open

部分算子的参数命名不规范 #1153

zrr1999 opened this issue Jan 17, 2023 · 3 comments
Labels

Comments

@zrr1999
Copy link
Member

zrr1999 commented Jan 17, 2023

在 transpose 等算子中,vector axis 被作为一个传入参数,而在一般情况下,axis应该是一个整数,只表示某一个维度。在Paddle等其他项目中这个参数通常会被命名为perm。

这个问题在使用时通常不会造成太大的问题,但是在开发pass中有可能遇到一些潜在的BUG,例如在我开发 公共子表达式消除的PR中,需要开发一个可扩展的接口,这个接口需要根据参数名进行一些预处理,例如需要将负数的axis处理成正数,但是transpose的axis参数混淆了这个过程,现阶段只能增加判断特定算子名的方式来解决这个问题。

参考文档

  1. https://www.paddlepaddle.org.cn/documentation/docs/zh/api/paddle/transpose_cn.html#transpose
@zrr1999
Copy link
Member Author

zrr1999 commented Jan 17, 2023

@SunNy820828449 @luotao1

@SunNy820828449
Copy link
Collaborator

好的 谢谢。我们现在内部正在重新整理算子,不过因为优先级不高,估计会在一季度末或者二季度开始完成算子体系的更新,我们会就你提出的问题进行调整。

@zrr1999
Copy link
Member Author

zrr1999 commented Jan 17, 2023

好的 谢谢。我们现在内部正在重新整理算子,不过因为优先级不高,估计会在一季度末或者二季度开始完成算子体系的更新,我们会就你提出的问题进行调整。

好的了解了

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants