Skip to content

Fix build issues with documentation#7122

Merged
miscco merged 15 commits intoNVIDIA:mainfrom
miscco:fix_documentation
Jan 12, 2026
Merged

Fix build issues with documentation#7122
miscco merged 15 commits intoNVIDIA:mainfrom
miscco:fix_documentation

Conversation

@miscco
Copy link
Contributor

@miscco miscco commented Jan 8, 2026

Our docs currently do not build.

I tried to address as many probloms as possible, however there are still some strange errors complains about a missing :end-after: string, but only sometimes 🤷

Also for whatever reason doxygen sometimes does not find the right module label of the any_resource files?

@miscco miscco requested review from a team as code owners January 8, 2026 12:08
@miscco miscco requested review from gonidelis and shwina January 8, 2026 12:08
@github-project-automation github-project-automation bot moved this to Todo in CCCL Jan 8, 2026
@miscco miscco requested a review from bernhardmgruber January 8, 2026 12:08
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Jan 8, 2026
Below, ``segmented_sort`` is used to perform a segmented sort. It also rearranges the values according to the keys' order.

.. literalinclude:: ../../python/cuda_cccl/tests/parallel/examples/sort/segmented_sort_basic.py
.. literalinclude:: ../../python/cuda_cccl/tests/compute/examples/sort/segmented_sort_basic.py
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is giving a warning about missing :end-after:

The same for radix_sort, maybe because there are 2 examples after each other? @shwina

@github-actions

This comment has been minimized.

Comment on lines +61 to 62
#if !_CCCL_COMPILER(NVRTC) && !defined(_CCCL_DOXYGEN_INVOKED)
inline ::std::ostream& operator<<(::std::ostream& os, RadixSortStoreAlgorithm algo)
Copy link
Contributor

Choose a reason for hiding this comment

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

Question: I don't understand why Doxygen cannot handle operator<<. I think it should be publicly documented it would be sad to see it go. What's the problem here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I dont know either, from the error message I guess it is that it tries to putt the << into the section header and that does things

Copy link
Contributor

Choose a reason for hiding this comment

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

Super annoying! I hope we can bring the docs back at some point.

@github-actions

This comment has been minimized.

@miscco miscco enabled auto-merge (squash) January 12, 2026 13:37
@github-actions
Copy link
Contributor

🥳 CI Workflow Results

🟩 Finished in 3d 05h: Pass: 100%/190 | Total: 6d 11h | Max: 5h 28m | Hits: 71%/264166

See results here.

Copy link
Contributor

@shwina shwina left a comment

Choose a reason for hiding this comment

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

Approving Python changes - thank you for catching this!

@miscco miscco merged commit 5da471c into NVIDIA:main Jan 12, 2026
402 of 405 checks passed
@miscco miscco deleted the fix_documentation branch January 12, 2026 15:58
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants