Skip to content
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

Merged
merged 26 commits into from
Feb 3, 2020

Conversation

veronica-villa
Copy link
Contributor

Adding code to calculate source probabilities from chirp mass and effective distance

@veronica-villa
Copy link
Contributor Author

Example of an upload to gracedb-playground: https://gracedb-playground.ligo.org/events/T228684/view/

pycbc/io/live.py Outdated Show resolved Hide resolved
pycbc/io/live.py Outdated Show resolved Hide resolved
pycbc/io/live.py Outdated Show resolved Hide resolved
pycbc/io/live.py Outdated Show resolved Hide resolved
pycbc/io/live.py Outdated Show resolved Hide resolved
pycbc/mchirp_area.py Outdated Show resolved Hide resolved
@titodalcanton
Copy link
Contributor

@veronica-villa feel free to click its "Resolve conversation" button when you think you have addressed one of my comments.

bin/pycbc_live Outdated Show resolved Hide resolved
pycbc/io/live.py Outdated Show resolved Hide resolved
pycbc/mchirp_area.py Outdated Show resolved Hide resolved
@titodalcanton titodalcanton changed the title Source probabilities Add calculation of source type probabilities to PyCBC Live Jan 30, 2020
@titodalcanton
Copy link
Contributor

@veronica-villa I have a few remaining comments, once those are addressed I am happy with this.

@veronica-villa
Copy link
Contributor Author

This is how it looks like in gracedb https://gracedb-playground.ligo.org/events/T250279/view/

Copy link
Contributor

@titodalcanton titodalcanton left a 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.

@titodalcanton titodalcanton merged commit 011fec9 into gwastro:master Feb 3, 2020
lenona pushed a commit to lenona/pycbc that referenced this pull request Sep 14, 2020
)

* 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
OliverEdy pushed a commit to OliverEdy/pycbc that referenced this pull request Apr 3, 2023
)

* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants