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

Add comprehensive documentation to Python via sphinx #3993

Merged
merged 16 commits into from
May 19, 2019

Conversation

lramati
Copy link
Contributor

@lramati lramati commented May 16, 2019

Inline help is now much more comprehensive. Additionally, useful HTML docs can be generated with sphinx

@lramati
Copy link
Contributor Author

lramati commented May 16, 2019

Note: Generating the documentation generates a lot of warnings and creates the folder wrappers/python/docs/_generated in the main tree (NOT the build tree).
I'm not sure how to resolve these issues, but coherent documentation DOES get generated.
The documentation isn't as cross linked as one might like, but it is functional.

Copy link
Contributor

@bfulkers-i bfulkers-i left a comment

Choose a reason for hiding this comment

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

I stopped reviewing python.cpp once I realized the comments were generated from the c/c++ headers

include/librealsense2/h/rs_types.h Outdated Show resolved Hide resolved
include/librealsense2/h/rs_types.h Outdated Show resolved Hide resolved
include/librealsense2/hpp/rs_frame.hpp Outdated Show resolved Hide resolved
@@ -407,7 +407,7 @@ namespace rs2
void keep() { rs2_keep_frame(frame_ref); }

/**
* Parenthesis operator check internal frame handle is valid.
* Parenthesis operator check if internal frame handle is valid.
Copy link
Contributor

Choose a reason for hiding this comment

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

The parenthesis operator checks if the

include/librealsense2/hpp/rs_frame.hpp Outdated Show resolved Hide resolved
wrappers/python/python.cpp Outdated Show resolved Hide resolved
wrappers/python/python.cpp Outdated Show resolved Hide resolved
wrappers/python/python.cpp Outdated Show resolved Hide resolved
wrappers/python/python.cpp Outdated Show resolved Hide resolved
wrappers/python/python.cpp Outdated Show resolved Hide resolved
lramati pushed a commit to lramati/librealsense that referenced this pull request May 19, 2019
Copy link
Collaborator

@ev-mp ev-mp left a comment

Choose a reason for hiding this comment

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

@lramati , LGTM.
This PR will definitely facilitate Python development without the need for constant cross-referencing.
@bfulkers-i , thank you for the rigorous review 👍

@ev-mp ev-mp merged commit 4dad00e into IntelRealSense:development May 19, 2019
@lramati lramati deleted the pydocs branch June 16, 2019 13:17
achirkin pushed a commit to achirkin/librealsense that referenced this pull request Nov 26, 2020
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