ReactiveBayes / RxInfer.jl

Julia package for automated Bayesian inference on a factor graph with reactive message passing
MIT License
259 stars 24 forks source link

docs(): improve wording in README & add statement of need #66

Closed bvdmitri closed 1 year ago

bvdmitri commented 1 year ago

This PR fixes #65 and #62 .

@bertdv Do you mind reviewing the Why RxInfer section in the documentation. That is supposed to be a "Statement of need" and is important for the JOSS publication. It should not be too long though.

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

codecov-commenter commented 1 year ago

Codecov Report

Base: 79.70% // Head: 79.70% // No change to project coverage :thumbsup:

Coverage data is based on head (6064cb8) compared to base (01c6fb4). Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #66 +/- ## ======================================= Coverage 79.70% 79.70% ======================================= Files 11 11 Lines 1163 1163 ======================================= Hits 927 927 Misses 236 236 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=biaslab). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=biaslab)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.