-
Notifications
You must be signed in to change notification settings - Fork 5
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
GraphPPL version 4.0.0 #204
Conversation
50% faster model construction (zoom)
Use same unique call for partition check and constraint materialization
Performance review
Type stability of nested models
Add compile time keys for the extra properties of graph nodes
Migration guide
…d-patch1 A small patch for the documentation version 4.0.0
…d-patch-2 Minor changes to the syntax guide
Adjusted documentation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #204 +/- ##
=======================================
Coverage ? 91.82%
=======================================
Files ? 15
Lines ? 1932
Branches ? 0
=======================================
Hits ? 1774
Misses ? 158
Partials ? 0 ☔ View full report in Codecov by Sentry. |
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 @wouterwln
This PR reimplements GraphPPL to explicitly construct a factor graph representation of models. This allows nested models and a nicer model macro. This PR also adds tests and documentation to the new version