-
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
Adding the rst file to render IO under Fluid in paddlepaddle.org #7470
Conversation
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.
Can you paste the generated result like #7445 (comment) to help check the correct format?
doc/api/v2/fluid/io.rst
Outdated
|
||
|
||
|
||
isParameter |
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.
is_parameter
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.
Fixed.
python/paddle/v2/fluid/io.py
Outdated
var : The input variable. | ||
|
||
Returns: | ||
boolean result whether the variable is a Parameter |
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.
Is line 14 and line 22 lacking periods in the end?
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.
Thank you! I will fix these things. Just wanted to check if it renders the comment at all.
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.
Fixed.
@@ -11,6 +11,16 @@ | |||
|
|||
|
|||
def is_parameter(var): |
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.
As is_parameter
is not in __all__
, could it show correctly on the website?
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.
Yes, see screenshot below
From the screenshot, there is no heading |
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.
LGTM!!
This PR solves the issue : #7335
As of now, I have added comments in just the
is_parameter
function ofio.py
and made sure this works and renders fine, and added the required rst file.Once merged, I can add comments to other functions of the file as well in order for them to render.