ENH: implement time marginalization into multiband likelihood#842
Merged
mj-will merged 1 commit intobilby-dev:mainfrom Nov 1, 2024
Merged
ENH: implement time marginalization into multiband likelihood#842mj-will merged 1 commit intobilby-dev:mainfrom
mj-will merged 1 commit intobilby-dev:mainfrom
Conversation
Collaborator
|
It looks like this was broken by the requirement that we don't have superfluous waveform kwargs. |
ColmTalbot
reviewed
Oct 31, 2024
SMorisaki
force-pushed
the
time_marg_multiband
branch
from
f09d62a to
2a87ea9
Compare
- bilby/gw/likelihood/multiband.py: implement time marginalization and resampling of time - test/gw/likelihood/marginalization_test.py: add marginalization tests for multiband likelihood
SMorisaki
force-pushed
the
time_marg_multiband
branch
from
2a87ea9 to
d481f93
Compare
ColmTalbot
approved these changes
Nov 1, 2024
This was referenced Nov 1, 2024
mj-will
approved these changes
Nov 1, 2024
mj-will
changed the title
JasperMartins
pushed a commit
to JasperMartins/bilby
that referenced
this pull request
Nov 5, 2024
…dev#842) - bilby/gw/likelihood/multiband.py: implement time marginalization and resampling of time - test/gw/likelihood/marginalization_test.py: add marginalization tests for multiband likelihood
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This MR implements time marginalization into multiband likelihood. Time marginalization for multiband may be useful since the slow down factor of time marginalization is not as large as that for ROQ or relative binning. The plots below show the likelihood evaluation time of multiband/standard likelihood and with/without time marginalization for 16s XPHM, 64s XPHM, and 128s Pv2_NRtidalv2. The slow down is less than 2 for the first 2 cases.