-
Notifications
You must be signed in to change notification settings - Fork 219
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
Generate API docs #2347
Generate API docs #2347
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2347 +/- ##
==========================================
- Coverage 86.67% 0.00% -86.68%
==========================================
Files 24 22 -2
Lines 1599 1533 -66
==========================================
- Hits 1386 0 -1386
- Misses 213 1533 +1320 ☔ View full report in Codecov by Sentry. |
dda13b9
to
2efffe2
Compare
Pull Request Test Coverage Report for Build 11252307560Details
💛 - Coveralls |
8c368c5
to
8d408b4
Compare
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
I think this is finally ready :) Most of the changes can be seen via the preview docs (https://turinglang.org/Turing.jl/previews/PR2347/) but also please do check over the docstrings I added. (There are no source code changes in this PR) |
@sunxd3 can you help review @penelopeysm's PR? |
1.11 tests failing because of compintell/Mooncake.jl#180 |
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.
Great work @penelopeysm
# API: `Turing.Inference` | ||
|
||
```@autodocs | ||
Modules = [Turing.Inference] |
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.
ideally, this page is more structured. but I think we should merge this now and fix it later.
Co-authored-by: Xianda Sun <5433119+sunxd3@users.noreply.github.com>
Just saw this; really like this @penelopeysm :) |
Description
This PR generates API docs for Turing.jl itself. This exposes documentation for important things like samplers which are otherwise undocumented.
Many of the exported identifiers are defined in other packages (DynamicPPL, etc.) and re-exported and/or extended by Turing. Rather than inserting the actual docs (which can actually be quite difficult sometimes as it requires locating the right method), I've generally opted to put them in a table with links to the corresponding package documentation.
This PR also adds docstrings to exported identifiers where necessary, and fixes some inaccuracies in the
MH()
docstrings.Closes #2344.
Preview the docs
https://turinglang.org/Turing.jl/previews/PR2347/