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

[Docs] add some Tensor API en doc #55958

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

zhwesky2010
Copy link
Contributor

@zhwesky2010 zhwesky2010 commented Aug 3, 2023

PR types

Bug fixes

PR changes

APIs

Description

Pcard-73362

add some Tensor API en doc.

@paddle-bot
Copy link

paddle-bot bot commented Aug 3, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@zhwesky2010 zhwesky2010 force-pushed the add_en_doc1 branch 3 times, most recently from 7551a3e to e352e21 Compare August 3, 2023 12:06
@zhwesky2010 zhwesky2010 changed the title add some Tensor API en doc [Docs] add some Tensor API en doc Aug 3, 2023
paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
Comment on lines 2124 to 2125
Input SparseCooTensor: returns True
Input DenseTensor/SparseCsrTensor: returns False
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Input SparseCooTensor: returns True
Input DenseTensor/SparseCsrTensor: returns False
Input SparseCooTensor: returns True
Input DenseTensor/SparseCsrTensor: returns False

paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
Comment on lines 2197 to 2198
**Notes**:
**This API is only available for DenseTensor/SparseCooTensor.**
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
**Notes**:
**This API is only available for DenseTensor/SparseCooTensor.**
Note:
**This API is only available for DenseTensor/SparseCooTensor.**

paddle/fluid/pybind/eager_method.cc Outdated Show resolved Hide resolved
@zhwesky2010 zhwesky2010 force-pushed the add_en_doc1 branch 3 times, most recently from 3b2a3ec to 13c9f70 Compare August 4, 2023 12:32
sunzhongkai588
sunzhongkai588 previously approved these changes Aug 7, 2023
Copy link
Contributor

@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

@paddle-ci-bot
Copy link

paddle-ci-bot bot commented Aug 12, 2023

Sorry to inform you that 13c9f70's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

wanghuancoder
wanghuancoder previously approved these changes Aug 14, 2023
Copy link
Contributor

@wanghuancoder wanghuancoder left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@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 for docs

@wanghuancoder wanghuancoder merged commit fd765f6 into PaddlePaddle:develop Aug 17, 2023
risemeup1 pushed a commit that referenced this pull request Aug 17, 2023
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

Successfully merging this pull request may close these issues.

3 participants