-
Notifications
You must be signed in to change notification settings - Fork 2
Description
When resampling with a priority_fn
that outputs log priority scores, the function update_weights!
(called after resampling) currently does something that is only correct if we're interested in unbiasedly estimating the normalizing constant / log marginal likelihood estimate, and is not correct for other test functions.
In general, the proper unnormalized weight
However, this is not the weight update currently implemented in update_weights!
.
In addition to fixing this, the tests will have to be revised to check for genuine proper weighting. This will require Monte Carlo estimation, since the correct weight update creates variance in the LML estimate and other expectations.
*If I did the math right.