-
Notifications
You must be signed in to change notification settings - Fork 68
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
Add Turing integration tests #1813
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1813 +/- ##
==========================================
+ Coverage 67.50% 69.92% +2.41%
==========================================
Files 31 32 +1
Lines 12668 13645 +977
==========================================
+ Hits 8552 9541 +989
+ Misses 4116 4104 -12 ☔ View full report in Codecov by Sentry. |
I changed the dependency from Turing to DynamicPPL and Distributions since we don't do any sampling. I pinnned those two and their major depencencies: AbstractMCMC, AbstractPPL, Bijectors, LogDensityProblems, LogDensityProblemsAD. Can pin all the dependencies if you'd like (BangBang, OrderedCollections, etc.), but I wonder if in that case it would make more sense to just commit the Manifest.toml. |
Makes log density functions out of a bunch of Turing.jl models and checks that Enzyme can differentiate them, both in Forward and Reverse modes.
Related issues that should probably be sorted before merging this: