Skip to content

Changes

Page history
Update O4 review authored by Simon Stevenson's avatar Simon Stevenson
Show whitespace changes
Inline Side-by-side
O4-review.md
View page @ 6af232db
......@@ -6,16 +6,174 @@ Many of these features have unit tests verifying their performance, others have
## Significant changes
| Description | Relevant changes | Link | Status | Reviewed version |
|-------------|------------------|------|--------|------------------|
| Optimized Cython geometric functions | [bilby-cython](https://git.ligo.org/colm.talbot/bilby-cython) | [issues](https://git.ligo.org/colm.talbot/bilby-cython/-/issues/?sort=updated_desc&state=closed&first_page_size=20) | :white_check_mark: Reviewed by @matthew-pitkin 11/04/23 | [0.4.0](https://git.ligo.org/colm.talbot/bilby-cython/-/merge_requests/14#note_670691) |
| Sampling in aligned and planar spin components | https://git.ligo.org/lscsoft/bilby/-/merge_requests/868 | [review page](O4-review/spin-parameters) | :white_check_mark: Reviewed by @simon-stevenson 29/03/23 | 2.0.2 |
| Various improvements to ROQ likelihood | https://git.ligo.org/lscsoft/bilby/-/merge_requests/1093, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1216, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1196 | [review page](O4-review/roq-improvements) | Ready for review | |
| multibanding likelihood | https://git.ligo.org/lscsoft/bilby/-/merge_requests/972, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1171 | [review wiki](https://git.ligo.org/pe/O4/multiband-likelihood-review/-/wikis/home) | | |
| relative binning (heterodyning) likelihood | https://git.ligo.org/lscsoft/bilby/-/merge_requests/1105, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1186, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1198, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1211 | [review page](O4-review/relative-binning) | Ready for review | |
| Likelihood calibration marginalization | https://git.ligo.org/lscsoft/bilby/-/merge_requests/856, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1201, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1234 | [review page](O4-review/calibration-marginalized-likelihood) | Ready for review | |
| ROQ likelihood time marginalization | (https://git.ligo.org/lscsoft/bilby/-/merge_requests/1040) | [review page](O4-review/roq-time-marginalization) | Ready for review | |
| `dynesty` sampling | https://git.ligo.org/lscsoft/bilby/-/merge_requests/1138 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1187, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1190, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1203, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1222, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1225 | [review page](O4-review/dynesty-updates) | Ready for review | |
| Likelihood reweighting | https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/493 https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/508 https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/522 https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/509 https://git.ligo.org/lscsoft/bilby/-/merge_requests/776 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1234 | [review page](O4-review/likelihood-reweighting) | Ready for review | |
| bilby-mcmc |https://git.ligo.org/lscsoft/bilby/-/merge_requests/905 https://git.ligo.org/lscsoft/bilby/-/merge_requests/945 https://git.ligo.org/lscsoft/bilby/-/merge_requests/985 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1001 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1004 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1242 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1079 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1141 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1155 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1156 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1210 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1222 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1237 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1238 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1242| | | |
<table>
<tr>
<th>Description</th>
<th>Relevant changes</th>
<th>Link</th>
<th>Status</th>
<th>Reviewed version</th>
</tr>
<tr>
<td>Optimized Cython geometric functions</td>
<td>
[bilby-cython](https://git.ligo.org/colm.talbot/bilby-cython)
</td>
<td>
[issues](https://git.ligo.org/colm.talbot/bilby-cython/-/issues/?sort=updated_desc&state=closed&first_page_size=20)
</td>
<td>
:white_check_mark: Reviewed by @matthew-pitkin 11/04/23
</td>
<td>
[0.4.0](https://git.ligo.org/colm.talbot/bilby-cython/-/merge_requests/14#note_670691 "FEAT: expose the calculation of the rotation matrix")
</td>
</tr>
<tr>
<td>Sampling in aligned and planar spin components</td>
<td>
https://git.ligo.org/lscsoft/bilby/-/merge_requests/868
</td>
<td>
[review page](O4-review/spin-parameters)
</td>
<td>
:white_check_mark: Reviewed by @simon-stevenson 29/03/23
</td>
<td>2.0.2</td>
</tr>
<tr>
<td>Various improvements to ROQ likelihood</td>
<td>
https://git.ligo.org/lscsoft/bilby/-/merge_requests/1093, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1216, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1196
</td>
<td>
[review page](O4-review/roq-improvements)
</td>
<td>Ready for review</td>
<td>
</td>
</tr>
<tr>
<td>multibanding likelihood</td>
<td>
https://git.ligo.org/lscsoft/bilby/-/merge_requests/972, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1171,
https://git.ligo.org/lscsoft/bilby/-/merge_requests/1253
</td>
<td>
[review wiki](https://git.ligo.org/pe/O4/multiband-likelihood-review/-/wikis/home)
</td>
<td>Ready for review by @rory-smith</td>
<td>
</td>
</tr>
<tr>
<td>relative binning (heterodyning) likelihood</td>
<td>
https://git.ligo.org/lscsoft/bilby/-/merge_requests/1105, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1186, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1198, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1211
</td>
<td>
[review page](O4-review/relative-binning)
</td>
<td>Ready for review</td>
<td>
</td>
</tr>
<tr>
<td>Likelihood calibration marginalization</td>
<td>
https://git.ligo.org/lscsoft/bilby/-/merge_requests/856, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1201, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1234
</td>
<td>
[review page](O4-review/calibration-marginalized-likelihood)
</td>
<td>Ready for review</td>
<td>
</td>
</tr>
<tr>
<td>ROQ likelihood time marginalization</td>
<td>
(https://git.ligo.org/lscsoft/bilby/-/merge_requests/1040)
</td>
<td>
[review page](O4-review/roq-time-marginalization)
</td>
<td>Ready for review</td>
<td>
</td>
</tr>
<tr>
<td>
`dynesty` sampling
</td>
<td>
https://git.ligo.org/lscsoft/bilby/-/merge_requests/1138 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1187, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1190, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1203, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1222, https://git.ligo.org/lscsoft/bilby/-/merge_requests/1225
</td>
<td>
[review page](O4-review/dynesty-updates)
</td>
<td>Ready for review</td>
<td>
</td>
</tr>
<tr>
<td>Likelihood reweighting</td>
<td>
https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/493 https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/508 https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/522 https://git.ligo.org/lscsoft/bilby_pipe/-/merge_requests/509 https://git.ligo.org/lscsoft/bilby/-/merge_requests/776 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1234
</td>
<td>
[review page](O4-review/likelihood-reweighting)
</td>
<td>Ready for review</td>
<td>
</td>
</tr>
<tr>
<td>bilby-mcmc</td>
<td>
https://git.ligo.org/lscsoft/bilby/-/merge_requests/905 https://git.ligo.org/lscsoft/bilby/-/merge_requests/945 https://git.ligo.org/lscsoft/bilby/-/merge_requests/985 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1001 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1004 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1242 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1079 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1141 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1155 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1156 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1210 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1222 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1237 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1238 https://git.ligo.org/lscsoft/bilby/-/merge_requests/1242
</td>
<td>
</td>
<td>
</td>
<td>
</td>
</tr>
</table>