-
Notifications
You must be signed in to change notification settings - Fork 357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add calculation of source type probabilities to PyCBC Live #3077
Add calculation of source type probabilities to PyCBC Live #3077
Conversation
Example of an upload to gracedb-playground: https://gracedb-playground.ligo.org/events/T228684/view/ |
@veronica-villa feel free to click its "Resolve conversation" button when you think you have addressed one of my comments. |
80585fc
to
5e1ffee
Compare
5e1ffee
to
dad2999
Compare
@veronica-villa I have a few remaining comments, once those are addressed I am happy with this. |
This is how it looks like in gracedb https://gracedb-playground.ligo.org/events/T250279/view/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks ok to merge now.
) * Introduce source probability calculation * Mass gap probabilities improved * Change save function to add probs file * Change in z_delta and mchirp_delta * Add argument for mchirp_delta coefficient * Add mchirp bbh condition * Change save function * Add CBC probabilities to upload method * Small corrections to mchirp_area.py * Add logging messages * Fix small errors * Fix small syntax error * Fix codeclimate issues * Fix codeclimate issues * Fix codeclimate issues * Fix Travis issue * Changes in pie plot code * Remove unnecessary serialization to JSON file * Move distance and redshift estimations to mchirp_area module * Fix small errors * Fix codeclimate issue * Add color codes for the different CBC sources * Change option names and help strings * Assign colors to CBC sources and tag JSON file and plot as EM followup * Fix codeclimate issues * Change mc_area_args to be an attribute of LiveEventManager
) * Introduce source probability calculation * Mass gap probabilities improved * Change save function to add probs file * Change in z_delta and mchirp_delta * Add argument for mchirp_delta coefficient * Add mchirp bbh condition * Change save function * Add CBC probabilities to upload method * Small corrections to mchirp_area.py * Add logging messages * Fix small errors * Fix small syntax error * Fix codeclimate issues * Fix codeclimate issues * Fix codeclimate issues * Fix Travis issue * Changes in pie plot code * Remove unnecessary serialization to JSON file * Move distance and redshift estimations to mchirp_area module * Fix small errors * Fix codeclimate issue * Add color codes for the different CBC sources * Change option names and help strings * Assign colors to CBC sources and tag JSON file and plot as EM followup * Fix codeclimate issues * Change mc_area_args to be an attribute of LiveEventManager
Adding code to calculate source probabilities from chirp mass and effective distance