Update PESummary Review authored by Charlie Hoy's avatar Charlie Hoy
...@@ -235,3 +235,27 @@ To perform the review tasks, we are using the sample release associated with GW1 ...@@ -235,3 +235,27 @@ To perform the review tasks, we are using the sample release associated with GW1
| `network_matched_filter_snr` bug | [`network_matched_filter_snr` bug](https://git.ligo.org/charlie.hoy/pesummary_bug/-/wikis/Review-of-network_matched_filter_snr-bug-script#sign-off) | @charlie.hoy | @vivien | :white_check_mark: | [376af2c0](https://git.ligo.org/lscsoft/pesummary/-/merge_requests/428/diffs?commit_id=376af2c0537d07be2d658e1705c20535260375a0) | :heavy_check_mark: | | `network_matched_filter_snr` bug | [`network_matched_filter_snr` bug](https://git.ligo.org/charlie.hoy/pesummary_bug/-/wikis/Review-of-network_matched_filter_snr-bug-script#sign-off) | @charlie.hoy | @vivien | :white_check_mark: | [376af2c0](https://git.ligo.org/lscsoft/pesummary/-/merge_requests/428/diffs?commit_id=376af2c0537d07be2d658e1705c20535260375a0) | :heavy_check_mark: |
## Review Telecons ## Review Telecons
# Amendment 10
# Code version
* `pesummary`: [0.10.0](https://git.ligo.org/lscsoft/pesummary/-/tree/v0.10.0)
# Links to tests
> Mark complete with `:white_check_mark:` and incomplete with `:x:`
| Test | Related Resources | Person Responsible | Reviewer | Status | Sign off hash | Final Sign off |
| ---- | ----------------- | ------------------ | -------- | ------ | ------------- | --------------- |
| `summaryreview` | [review page](https://git.ligo.org/lscsoft/pesummary/-/wikis/v0.10.0#summaryreview-script) | @charlie.hoy | | :incomplete: | | |
| Reading and writing to an SQL database | [review page](https://git.ligo.org/lscsoft/pesummary/-/wikis/v0.10.0#sql-database-review) | @charlie.hoy | @philip.relton | :white_check_mark: | | |
| Reading and writing to a CSV file | [review page](https://git.ligo.org/lscsoft/pesummary/-/wikis/v0.10.0#csv-file-format) | @charlie.hoy | @philip.relton | :white_check_mark | | |
| Watermark on html pages | [review page](https://git.ligo.org/lscsoft/pesummary/-/wikis/v0.10.0#preliminary-watermark) | @charlie.hoy | @philip.relton | :incomplete: | | |
| Fetching GWTC-2 posterior samples | [review page](https://git.ligo.org/lscsoft/pesummary/-/wikis/v0.10.0#fetch-gwtc-2-result-files) | @charlie.hoy | | :incomplete: | | |
| File summary | [review page](https://git.ligo.org/lscsoft/pesummary/-/wikis/v0.10.0#file-summary) | @charlie.hoy | | :incomplete: | | |
| Converted parameters property | [review page](https://git.ligo.org/lscsoft/pesummary/-/wikis/v0.10.0#converted-parameters) | @charlie.hoy | | :incomplete: | | |
| summarypipe with `pbilby` result files | [!459](https://git.ligo.org/lscsoft/pesummary/-/merge_requests/459) [!460](https://git.ligo.org/lscsoft/pesummary/-/merge_requests/460) | @charlie.hoy | @david.keitel | :white_check_mark: | | |
| Conversion with source frame masses | [!461](https://git.ligo.org/lscsoft/pesummary/-/merge_requests/461) | @charlie.hoy | @david.keitel | :white_check_mark: | | |
## Review Telecons
\ No newline at end of file