Remove VariableOrderAccumulator and subset and merge on accs#1005
Conversation
Benchmark Report for Commit 1d93529Computer InformationBenchmark Results |
|
DynamicPPL.jl documentation for PR #1005 is available at: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1005 +/- ##
==========================================
+ Coverage 81.91% 82.16% +0.24%
==========================================
Files 38 38
Lines 4025 3935 -90
==========================================
- Hits 3297 3233 -64
+ Misses 728 702 -26 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Pull Request Test Coverage Report for Build 16680558902Details
💛 - Coveralls |
There was a problem hiding this comment.
As @yebai requested #901 (comment) I'll add some extra elaboration in the changelog about why we no longer need num_produce / order / VariableOrderAccumulator. Leaving this comment here as a reminder to myself to do this next week.
(I'm happy to do this since the corresponding Turing PR was mine.)
|
num_produce / order / pMCMC section in changelog has been expanded. |
84af272 to
9b2bee0
Compare
penelopeysm
left a comment
There was a problem hiding this comment.
There's a set_retained_vns_del! in src/threadsafe.jl that should also be removed, but otherwise very happy
|
Probably that method never needed to exist because in Turing's particle MCMC code threadsafe evaluation is always disabled: I actually wonder whether threadsafe execution would work now with the changes we have... my guess is not but idk. |
These are no longer needed in Turing.jl.
To be merged after #901. Merging that will also clean up the diff of this. Please don't try to review before that.