Skip to content
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

贡献指南 / 开发 C++ 算子 中 算子描述及定义 链接错误 #7082

Open
cangtianhuang opened this issue Mar 6, 2025 · 7 comments
Assignees

Comments

@cangtianhuang
Copy link
Contributor

文档链接&描述 Document Links & Description

贡献指南/开发 C++ 算子 中 算子描述及定义 链接错误,指向的文件不存在。包括:

位于:开发 C++ 算子

请提出你的建议/提个PR修复这个问题 Please give your suggestion/update a PR to fix it

正确的 yaml 文件路径应该位于paddle\phi\ops\yaml\ops.yaml 与 paddle\phi\ops\yaml\backward.yaml

@luotao1
Copy link
Collaborator

luotao1 commented Mar 6, 2025

正确的 yaml 文件路径应该位于paddle\phi\ops\yaml\ops.yaml 与 paddle\phi\ops\yaml\backward.yaml

可以提个PR修复下哦

@sunzhongkai588
Copy link
Collaborator

sunzhongkai588 commented Mar 6, 2025

@cangtianhuang 这俩目录已经被迁移了,参考 PaddlePaddle/Paddle#64446

目录改一下吧,新目录在 paddle/phi/ops/yaml/backward.yamlpaddle/phi/ops/yaml/ops.yaml

@cangtianhuang
Copy link
Contributor Author

@cangtianhuang 这俩目录已经被迁移了,参考 PaddlePaddle/Paddle#64446

目录改一下吧,新目录在 paddle/phi/ops/yaml/backward.yamlpaddle/phi/ops/yaml/ops.yaml

感谢您的说明!我正在研究源码,查阅文档时发现了该错误。稍后我会提交PR修改它。

@DrRyanHuang
Copy link
Member

@cangtianhuang 还有一些格式问题也可以顺手修复一下😉,比如:

`backward.yaml` 中反向算子的配置规则如下:

@cangtianhuang
Copy link
Contributor Author

@cangtianhuang 还有一些格式问题也可以顺手修复一下😉,比如:

`backward.yaml` 中反向算子的配置规则如下:

好的😂我现在还没有完全弄懂算子与内核的实现方式,正在顺藤摸瓜的找。

@DrRyanHuang
Copy link
Member

btw,我发现几处关于 fluid 代码的描述,位于 八、更多信息 。若 fluid 将完全废弃,此处也应该被修改。此PR中未解决该问题😶‍🌫️。

@cangtianhuang 同学感兴趣后续也可以继续优化这部分内容 😉

@DrRyanHuang DrRyanHuang self-assigned this Mar 7, 2025
@cangtianhuang
Copy link
Contributor Author

@cangtianhuang 同学感兴趣后续也可以继续优化这部分内容 😉

这可能需要几天后😂我正在向paddle库中提交issue与PR

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

No branches or pull requests

4 participants