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

[DOC] API docs for cucim.CuImage not rendering #237

Closed
charlesbluca opened this issue Mar 21, 2022 · 3 comments · Fixed by #275
Closed

[DOC] API docs for cucim.CuImage not rendering #237

charlesbluca opened this issue Mar 21, 2022 · 3 comments · Fixed by #275
Assignees
Labels
doc Improvements or additions to documentation
Milestone

Comments

@charlesbluca
Copy link
Member

Report incorrect documentation

Location of incorrect documentation
The auto-generated API docs for cucim.CuImage:

The corresponding code for this:

Describe the problems or issues found in the documentation
The API docs for this class are not rendering:

image

Steps taken to verify documentation is incorrect
Comparing this page to the last stable docs build, I can see that these API docs used to get rendered properly for 21.12:

Suggested fix for documentation
A local build could potentially clear up the issue here - has CuImage been relocated or removed?

@charlesbluca charlesbluca added the doc Improvements or additions to documentation label Mar 21, 2022
@grlee77
Copy link
Contributor

grlee77 commented Mar 21, 2022

Thanks for reporting this. In addition to the missing cucim.clara docs, it looks like we are also missing docs for the functions under cucim.core.operations.

@grlee77
Copy link
Contributor

grlee77 commented Mar 21, 2022

The heading "CuImage Submodules" should probably be "Clara Submodules" and the functions to be documented are the ones here

@grlee77
Copy link
Contributor

grlee77 commented Mar 21, 2022

or maybe the ones under 'core.operations' are still to be considered experimental as these are not automatically loaded into the top-level cucim namespace. we can discuss in the next community call

@gigony gigony self-assigned this Apr 5, 2022
@gigony gigony added this to the v22.04.01 milestone Apr 5, 2022
@grlee77 grlee77 modified the milestones: v22.04.01, v22.06.00 May 11, 2022
@grlee77 grlee77 assigned grlee77 and unassigned gigony May 12, 2022
@rapids-bot rapids-bot bot closed this as completed in #275 May 20, 2022
rapids-bot bot pushed a commit that referenced this issue May 20, 2022
closes #237
closes #224
closes #119

Populate the API docs for the `cucim.clara` and `cucim.core` submodules. A few sphinx warnings were also resolved.

Authors:
  - Gregory Lee (https://github.com/grlee77)

Approvers:
  - Gigon Bae (https://github.com/gigony)
  - https://github.com/jakirkham

URL: #275
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants