-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[OpCompat] Remove redundant mappings #55448
[OpCompat] Remove redundant mappings #55448
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
Sorry to inform you that 3e1369a's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
@Asthestarsfalll hello,重新看了下这里的脚本,有个问题
这个检查是为了确认是否全部参数都是冗余的吗?如果是的话,应该没有必要。只有一对参数或某几个映射冗余,也应该一起删除 |
这里是判断是不是只有inputs或outputs,如果有其他信息比如extra等就跳过,现在去除了 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
* [OpCompat] Remove redundant mappings * update op_compat.yaml
* [OpCompat] Remove redundant mappings * update op_compat.yaml
PR types
Others
PR changes
Others
Description
去除paddle/phi/api/yaml/op_compat.yaml中类似冗余的映射逻辑
脚本代码: