search

TuringLang / DynamicPPL.jl

Implementation of domain-specific language (DSL) for dynamic probabilistic programming
https://turinglang.org/DynamicPPL.jl/
MIT License
157 stars 26 forks source link

Miscellaneous style and docs improvements #622

Closed mhauru closed 1 week ago

mhauru commented 2 months ago

Small typo fixes and a couple of docstring improvements. These are things I picked up as I was reading the code to understand it.

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9563441562

Details

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/context_implementations.jl 0 5 0.0%
<!-- Total: 5 10 50.0% -->
Totals Coverage Status
Change from base Build 9495604846: -0.7%
Covered Lines: 2743
Relevant Lines: 3445
💛 - Coveralls
coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9563441562

Details

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/context_implementations.jl 0 5 0.0%
<!-- Total: 5 10 50.0% -->
Totals Coverage Status
Change from base Build 9495604846: -0.03%
Covered Lines: 2756
Relevant Lines: 3431
💛 - Coveralls
torfjelde commented 2 months ago

Note that there's quite a rework on the transformation side in the works: #575 . I started working on this again today. After that, vectorize will be replaced + not used as extensively. You can see a description in the doc-preview: https://turinglang.org/DynamicPPL.jl/previews/PR575/internals/transformations/

mhauru commented 1 week ago

I rebase this on master and left only the relevant changes. Ready for review now.

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 10669973054

Details

Files with Coverage Reduction New Missed Lines %
src/model.jl 1 88.46%
src/threadsafe.jl 18 49.57%
<!-- Total: 19 -->
Totals Coverage Status
Change from base Build 10652412131: 0.0%
Covered Lines: 2712
Relevant Lines: 3499
💛 - Coveralls