-
Notifications
You must be signed in to change notification settings - Fork 229
Update variational inference interface to match AdvancedVI@0.5
#2699
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
base: breaking
Are you sure you want to change the base?
Conversation
|
Turing.jl documentation for PR #2699 is available at: |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## breaking #2699 +/- ##
=============================================
- Coverage 86.45% 58.56% -27.90%
=============================================
Files 21 20 -1
Lines 1418 1395 -23
=============================================
- Hits 1226 817 -409
- Misses 192 578 +386 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Thanks @Red-Portal.
Since this changes the user-interface, could you please set the target branch of this PR to be breaking? We use that branch to collect breaking changes into a single, larger release. Currently breaking points at the same commit as main, so you shouldn't have to mess with git history for this.
In that same process, could you also please add an entry in HISTORY.md describing to users all interface changes (arguments, return values, behaviour) and what they can do to adapt to them. Maybe a code example of "this is how you used to call vi, this is how you do the same thing now", that sort of thing.
… package test, (keep existing compat) (#2701) Co-authored-by: CompatHelper Julia <compathelper_noreply@julialang.org>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
I'm actually sorry to butt in on this, but are there new features available in 0.5, or is it just an interface change? |
|
@penelopeysm No new features. |
|
Okay. The reason why I asked is because this is an interface change for Turing and that is kind of annoying for end users if they don't get any benefit. I wonder in that case if it is worth doing either:
Other than that, I second everything that @mhauru brought up here, so I won't also review the details of this. |
Co-authored-by: Markus Hauru <markus@mhauru.org>
Co-authored-by: Markus Hauru <markus@mhauru.org>
|
@penelopeysm That's reasonable. I think we could go with the following for now:
|
… into bump_advancedvi_0.5
This updates the interface to
AdvancedVIto match the new v0.5 version. It also removes the previously-deprecated interface.