-
Notifications
You must be signed in to change notification settings - Fork 30
Feature/doc infra improvements #397
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
Feature/doc infra improvements #397
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.
Looks good! Thank you!
To build using Python ``setuptools``, the following packages should be | ||
installed: | ||
|
||
- ``cython`` |
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.
Let us provide the specific versions of each of these dependencies if not a range of versions
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.
I will add a requirements.txt to our repo and reference that in the quick start guide.
- Remove breathe and exhale requirements and instead use Doxyrest for the C API documentation. - Add cmake changes related to Doxyrest. - Provide two separate modes of documentation generation: both C and Python API and only Python API. - Update README, gitignore.
- Changes name of all files to use dpctl instead of dpCtl. - Remove toc_pyapi.rst
- All Python API classes are now documented under separate pages. - Add an autosummary to each class documentation page. - Rename dpCtl to dpctl in few places.
7ea7e4f
to
0d61611
Compare
Various improvements to dpctl documentation infrastructure.
doxyrest
in place ofbreathe
andexhale
to generate C API rst files.docfiles
sub-directory.Supersedes #341.