Conversation
janfb
left a comment
There was a problem hiding this comment.
Looks good, thanks for fixing this! 🙏
But where was the bug then here? Was it because the min_acceptance_rate was returned, which was the min over the batch of xs in case of log_prob_batched?
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1355 +/- ##
===========================================
- Coverage 89.40% 78.42% -10.99%
===========================================
Files 118 118
Lines 8715 8749 +34
===========================================
- Hits 7792 6861 -931
- Misses 923 1888 +965
Flags with carried forward coverage won't be shown. Click here to find out more.
|
The bug was that different xs in batched sampling lead to different posteriors. But the leakage correction scaled every "posterior" with the "min_acceptance_rate" i.e. the posterior which leaks most. But different posteriors, have different leakage. |
This should fix the issue #1350.
Changes