| ... | @@ -26,3 +26,51 @@ |
... | @@ -26,3 +26,51 @@ |
|
|
> Note: only four known events were selected for closer examination, the other seven events from GWTC1 were not inspected. A PE-group publication is planned producing results for all events in GWTC1.
|
|
> Note: only four known events were selected for closer examination, the other seven events from GWTC1 were not inspected. A PE-group publication is planned producing results for all events in GWTC1.
|
|
|
|
|
|
|
|
- AOB
|
|
- AOB
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
# Minutes
|
|
|
|
|
|
|
|
Bilby review call -- 190923
|
|
|
|
|
|
|
|
Ashton, Romero-Shaw, Pitkin, Galaudage, Stevenson, Lasky
|
|
|
|
|
|
|
|
Minutes: Lasky
|
|
|
|
|
|
|
|
https://git.ligo.org/lscsoft/bilby_pipe/wikis/03-review/minutes/190923
|
|
|
|
|
|
|
|
|
|
|
|
* Fixes/changes:
|
|
|
|
- difference in constants between LI and Bilby. Hadn’t noticed previously because chirp mass wasn’t measured as accurately as with GW170817. Fixed to LI values (although they’re outdated)
|
|
|
|
- conversion in times wrt GMT. Again, reverted to handling things the way LI does
|
|
|
|
- Boundaries: reflective still working, and are still being used and work (i.e., match with LI). Not clear if this is right, but there is now a bilby_pipe flag for choosing boundary conditions.
|
|
|
|
|
|
|
|
* GW170817 (https://ldas-jobs.ligo.caltech.edu/~gregory.ashton/bilby_pe_event_samples/GW170817/comparison_lowspin/home.html)
|
|
|
|
- LALInference MCMC02, LALInferenceNest03, Bilby
|
|
|
|
- LALInferenceNest03 and Bilby use same custom-made ROQ (from Rory Smith)
|
|
|
|
- This is low-spin run with fixed sky. ROQ Rory built goes up to 0.8-0.9 ish. LI and Bilby have an issue using this ROQ at high spins. Is a problem with the ROQ, not Bilby or LI. Rory will fix this.
|
|
|
|
- known bug in times, but this is present in both LI and Bilby. It’s being looked at/fixed on LALSuite.
|
|
|
|
- Proposal: this run is used for review, and other runs will be done and can be looked at later.
|
|
|
|
|
|
|
|
SS: All looks really good. Great agreement.
|
|
|
|
MP: Same. Masses now look correct. Timing is a known bug, so all looks great.
|
|
|
|
SNR_angle’s look different. Assume this is a difference in definitions in PESummary; clearly not affecting the parameters in any way.
|
|
|
|
GA: will touch base with Charlie about these definitions. Will put in an issue on git to PESummary.
|
|
|
|
|
|
|
|
* Proposed review statement - appears on front minutes page
|
|
|
|
- Good work being done getting Bilby to run online. Hopefully by end of O3a (although looking increasingly unlikely), but definitely by start of O3b. Would therefore like review statement that explains what has and has not been reviewed.
|
|
|
|
- would be good to get high-spin and all-sky prior cases for GW170817. But, don’t want other things to get held up, because those cases will take a long time (in just compute time…). But this should be made explicit in review statement.
|
|
|
|
|
|
|
|
MP: Suggestion sounds good. Happy for statement to be written, and then MP to sign off. Agree that there has to be a point when initial review is concluded, and more than happy for that to happen with advanced things still to kick off.
|
|
|
|
|
|
|
|
MP: also going to make all the review tick marks ticked off.
|
|
|
|
|
|
|
|
GA: On main review page there is a section for review statement.
|
|
|
|
|
|
|
|
SS: Also happy with review statement. Happy to sign off, and revisit high-spin case, etc.
|
|
|
|
|
|
|
|
Action: MP will put review statement (including any necessary modifications) on the main review wiki page. He will then email review chairs wrapping up review.
|
|
|
|
|
|
|
|
All: Many thanks to the reviewers for their hard work!
|
|
|
|
|
|
|
|
|
