-
Notifications
You must be signed in to change notification settings - Fork 14
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
Optimize rule/marginalrule dispatch #299
Conversation
Codecov ReportPatch coverage:
📣 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@@ Coverage Diff @@
## master #299 +/- ##
==========================================
+ Coverage 62.57% 64.32% +1.74%
==========================================
Files 197 197
Lines 7394 7363 -31
==========================================
+ Hits 4627 4736 +109
+ Misses 2767 2627 -140
... and 5 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Thanks for fixing the docs @bartvanerp . Are you okay with the remaining changes? Or did you plan to change text in other places? |
This PR improves upon #290 and addresses #284 even further.
A better version of the #292 without breaking changes. The PR improves execution time and memory allocation profile in 100% test-cases by various degree. Particularly useful for small models and real-time inference at a high rate, but also improves the performance of variational iterations.
Timings
Memory profile
This PR also removes the
@symmetrical
macro (was unused really) and removes options from therule
macro (was unused too).EDIT: This PR also adds documentation/docstrings for the
@rule
,@call_rule
,@marginalrule
and@call_marginalrule
.