Closed st-- closed 1 year ago
The deprecations in #361 had one flaw: previously,
dtc(VFE(fz), fx, y)
was technically correct (though it would have been better form to use the DTC alias defined as const DTC = VFE instead).
DTC
const DTC = VFE
However, the deprecations passed this on to approx_log_evidence, which then would just compute the VFE ELBO.
approx_log_evidence
This PR catches this special case.
Patch coverage is 100.00% of modified lines.
100.00%
:loudspeaker: Thoughts on this report? Let us know!.
The deprecations in #361 had one flaw: previously,
was technically correct (though it would have been better form to use the
DTC
alias defined asconst DTC = VFE
instead).However, the deprecations passed this on to
approx_log_evidence
, which then would just compute the VFE ELBO.This PR catches this special case.