Skip to content

Reorganize cuml.accel docs#7026

Merged
rapids-bot[bot] merged 1 commit intorapidsai:branch-25.08from
jcrist:reorg-docs
Jul 22, 2025
Merged

Reorganize cuml.accel docs#7026
rapids-bot[bot] merged 1 commit intorapidsai:branch-25.08from
jcrist:reorg-docs

Conversation

@jcrist
Copy link
Copy Markdown
Member

@jcrist jcrist commented Jul 21, 2025

This is a first pass at improving our docs organization (mostly targeting cuml.accel). It:

  • Moves all cuml.accel docs under a cuml-accel folder, removing the zero-code-change-* prefix most of these files had.
  • Sets up redirects so all old URLs continue to work
  • Moves the FAQ list out of the main cuml-accel page and into its own page.
  • Simplifies the initial cuml-accel page to provide a quickstart example and some pointers into other useful pages for followup. It's less busy now and should hopefully be easier for users to navigate.
  • Hides the toctrees, these appear in the sidebar already and don't need to take up extra vertical space.
  • Removes the very long titles from all cuml.accel docs pages. The user is already in the cuml.accel docs tree, IMO they don't need to see "cuml.accel: Zero Code Change Acceleration ..." as a title prefix for every page. This makes the sidebar and top bar much more legible.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@jcrist jcrist added doc Documentation non-breaking Non-breaking change labels Jul 21, 2025
@jcrist
Copy link
Copy Markdown
Member Author

jcrist commented Jul 21, 2025

This will conflict with #7021. I have a slight preference to merge that one first and fix the conflicts here, but either works.

@jcrist jcrist requested a review from csadorf July 21, 2025 18:39
@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Jul 21, 2025

This will conflict with #7021. I have a slight preference to merge that one first and fix the conflicts here, but either works.

#7021 should be good to merge.

This is a first pass at improving our docs organization (mostly
targeting cuml.accel). It:

- Moves all `cuml.accel` docs under a `cuml-accel` folder, removing the
  `zero-code-change-*` prefix most of these files had.
- Sets up redirects so all old URLs continue to work
- Moves the FAQ list out of the main `cuml-accel` page and into its own
  page.
- Simplifies the initial `cuml-accel` page to provide a quickstart
  example and some pointers into other useful pages for followup. It's
  less busy now and should hopefully be easier for users to navigate.
- Hides the toctrees, these appear in the sidebar already and don't need
  to take up extra vertical space.
- Removes the very long titles from all `cuml.accel` docs pages. The
  user is already in the `cuml.accel` docs tree, they don't need to see
  "cuml.accel: Zero Code Change Acceleration" as a title prefix for
  every page.
@jcrist
Copy link
Copy Markdown
Member Author

jcrist commented Jul 21, 2025

Conflicts fixed, this is ready for review.

@betatim
Copy link
Copy Markdown
Member

betatim commented Jul 22, 2025

https://downloads.rapids.ai/ci/cuml/pull-request/7026/8fb7da5/docs/cuml/html/cuml-accel/ docs preview for easy access (you need to be on the nvidia VPN to be able to see it)

Copy link
Copy Markdown
Member

@betatim betatim left a comment

Choose a reason for hiding this comment

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

LGTM

@jcrist
Copy link
Copy Markdown
Member Author

jcrist commented Jul 22, 2025

/merge

@rapids-bot rapids-bot Bot merged commit 65e301c into rapidsai:branch-25.08 Jul 22, 2025
55 checks passed
@jcrist jcrist deleted the reorg-docs branch July 22, 2025 13:22
@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Jul 22, 2025

I'm mostly good with the changes in this PR except for the "cuml.accel" as the only section header. I think it would be better and looks better to use "Zero Code Change Accelerator" in the section title and then be less repetitive within the body of the text. I can pushup a small PR with that change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Documentation non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants