... | @@ -20,9 +20,9 @@ This review wiki covers the changes to the code based in order to incorporate po |
... | @@ -20,9 +20,9 @@ This review wiki covers the changes to the code based in order to incorporate po |
|
# Review Checklist
|
|
# Review Checklist
|
|
|
|
|
|
* [ ] Formalism described in [LIGO-P2000216](https://dcc.ligo.org/LIGO-P2000216) / [arXiv:2007.01372](https://arxiv.org/abs/2007.01372) is correct
|
|
* [ ] Formalism described in [LIGO-P2000216](https://dcc.ligo.org/LIGO-P2000216) / [arXiv:2007.01372](https://arxiv.org/abs/2007.01372) is correct
|
|
* [ ] Monte-Carlo point-estimates and error estimates derived in [LIGO-P2000216](https://dcc.ligo.org/LIGO-P2000216) / [arXiv:2007.01372](https://arxiv.org/abs/2007.01372) are correct
|
|
* [ ] Monte-Carlo point-estimates and error estimates derived in [monte-carlo-derivations.pdf](uploads/06f0131a22f824afdf9acebd5460981a/monte-carlo-derivations.pdf) are correct
|
|
* *Note!* Reed has since found smally typos in the error estimates published in these works (associated with covariance between "f" and "g", etc).
|
|
* *Note!* Reed found smally typos in the error estimates published on arXiv (v1 and v2) associated with covariance between "f" and "g", etc. The attached notes correct this (and it has been corrected in the code).
|
|
* **ACTION ITEM** (for Reed): update derivations to show correct error estimation and to include formulas for Bayes factor.
|
|
* The attached notes also provide explicit formulas for the estimation of the bayes factor, which are only suggested in the arxiv versions. The code implements what is shown in the attached notes.
|
|
* [ ] Monte-Carlo algorithms are correctly implemented within calls to [utils.prob_odds](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/mmax_model_selection/utils.py#L434) (or [utils.post_prior_bayes](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/mmax_model_selection/utils.py#L476) as appropriate).
|
|
* [ ] Monte-Carlo algorithms are correctly implemented within calls to [utils.prob_odds](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/mmax_model_selection/utils.py#L434) (or [utils.post_prior_bayes](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/mmax_model_selection/utils.py#L476) as appropriate).
|
|
* [ ] population priors are correctly implemented within calls to [utils.population_weights](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/mmax_model_selection/utils.py#L107)
|
|
* [ ] population priors are correctly implemented within calls to [utils.population_weights](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/mmax_model_selection/utils.py#L107)
|
|
* [ ] reweighting of single-event posterior samples is by population priors is correctly done within the [mmax-model-selection executable](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/bin/mmax-model-selection#L153)
|
|
* [ ] reweighting of single-event posterior samples is by population priors is correctly done within the [mmax-model-selection executable](https://git.ligo.org/reed.essick/mmax-model-selection/-/blob/master/bin/mmax-model-selection#L153)
|
... | | ... | |