Skip to content

Improve public API docs and expand the manual.#708

Open
giordano wants to merge 11 commits into
JuliaGPU:mainfrom
giordano:mg/docs
Open

Improve public API docs and expand the manual.#708
giordano wants to merge 11 commits into
JuliaGPU:mainfrom
giordano:mg/docs

Conversation

@giordano

@giordano giordano commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Add missing docstrings for backends, kernel handles, and reflection macros; expand quickstart, kernels, and API pages with examples.

Mostly made by Cursor, I haven't reviewed all changes in detail (also because there are some areas I'm not familiar with), so please review everything very carefully!

Comment thread src/KernelAbstractions.jl
Comment thread src/KernelAbstractions.jl Outdated
Comment thread src/nditeration.jl
Add missing docstrings for backends, kernel handles, and reflection macros;
expand quickstart, kernels, and API pages with examples.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.39%. Comparing base (b179e7d) to head (226135b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #708      +/-   ##
==========================================
+ Coverage   52.31%   52.39%   +0.08%     
==========================================
  Files          22       22              
  Lines        1688     1691       +3     
==========================================
+ Hits          883      886       +3     
  Misses        805      805              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vchuravy vchuravy added this to the 0.10.0 milestone Jun 10, 2026
@giordano

Copy link
Copy Markdown
Contributor Author

I added a step to the docs CI job to always upload the docs build as an artefact, so that can be always inspected also when preview isn't available. For the latest version of the docs you can find the build at https://github.com/JuliaGPU/KernelAbstractions.jl/actions/runs/27298019329/artifacts/7544902664.

Comment thread docs/src/kernels.md Outdated
Comment thread docs/src/kernels.md Outdated
Comment thread docs/src/kernels.md Outdated
Comment thread src/KernelAbstractions.jl
Comment thread src/KernelAbstractions.jl
Comment thread src/KernelAbstractions.jl
Comment thread src/KernelAbstractions.jl
Comment thread src/nditeration.jl Outdated
giordano and others added 3 commits June 10, 2026 22:28
Clarify in the kernels guide that local memory allocation sizes must be
known at compile time.

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread docs/src/kernels.md Outdated
giordano and others added 3 commits June 10, 2026 23:01
Suggested-by: Christian Guinard <28689358+christiangnrd@users.noreply.github.com>
Suggested-by: Christian Guinard <28689358+christiangnrd@users.noreply.github.com>
@giordano

Copy link
Copy Markdown
Contributor Author

I turned most codeblocks in "Quickstart" and "Writing kernels" into proper doctests and runnable examples, to make sure they actually work correctly, otherwise there's the risk they'll bitrot. The doctest CI job hasn't been doing anything so far, as there currently are no doctests

giordano and others added 2 commits June 11, 2026 16:26
Co-authored-by: Christian Guinard <28689358+christiangnrd@users.noreply.github.com>
…ction

Also, make it crystal clear that internals are not meant to be directly used by
end-users.
@giordano

Copy link
Copy Markdown
Contributor Author

Ok, I should have addressed all comments above (I hope I didn't miss any!). The added doctests/@examples should make the docs more robust going forward.

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