ColmTalbot / gwpopulation

CPU/GPU agnostic gravitational-wave population inference
https://colmtalbot.github.io/gwpopulation/
MIT License
31 stars 46 forks source link

Add convergence check for default __call__ behavior in vt #68

Closed jacobgolomb closed 1 year ago

jacobgolomb commented 1 year ago

I realized that when we changed the ResamplingVT object to return the mu and var by default (rather than the vt_factor), we got rid of the ability to enforce the n_eff > 4*n_obs condition. This PR adds a flag to include this check before returning the mu, var values in the call.