Skip to content

[Docathon] add cn doc for paddle.utils.require_version and paddle.Tensor.create_tensor #6233

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

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

ooooo-create
Copy link
Collaborator

@paddle-bot
Copy link

paddle-bot bot commented Oct 19, 2023

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-6233.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明,请参考:飞桨文档预览工具

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Oct 20, 2023
@SigureMo
Copy link
Member

SigureMo commented Dec 6, 2023

create_tensor 不应该是一个 Tensor 的方法,应该是错误的 patch,create_tensor 的第一个参数并不是 Tensor,而是 dtype,不能作为 Tensor 的方法,需要将其从 tensor_method_func 移除

@SigureMo
Copy link
Member

SigureMo commented Dec 6, 2023

paddle.utils.require_version 我没啥问题,孙师傅看看呢

@SigureMo
Copy link
Member

SigureMo commented Dec 6, 2023

create_tensor 不应该是一个 Tensor 的方法,应该是错误的 patch,create_tensor 的第一个参数并不是 Tensor,而是 dtype,不能作为 Tensor 的方法,需要将其从 tensor_method_func 移除

这个不阻塞本 PR 合入,与本 PR 无关

Copy link
Collaborator

@sunzhongkai588 sunzhongkai588 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM~写的很好

@luotao1 luotao1 merged commit 3511e7d into PaddlePaddle:develop Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants