Changes
Page history
Update PESummary Review
authored
Nov 20, 2020
by
Charlie Hoy
Hide whitespace changes
Inline
Side-by-side
PESummary-Review.md
View page @
cb675bc6
...
...
@@ -234,4 +234,28 @@ 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: |
## 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