-
Notifications
You must be signed in to change notification settings - Fork 374
Add gated gaussian noise model that marginalizes over phase #5229
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
Open
acorreia61201
wants to merge
33
commits into
gwastro:master
Choose a base branch
from
acorreia61201:marg-rel-phase
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+426
−32
Conversation
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
…ure marg phase class to inherit from BaseGatedGaussian
…n new class to models init, waveform modes models to generator
…ent with other models
9aba291 to
cc0dcd6
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Standard information about the request
This PR adds a new model that allows for phase marginalization in gated gaussian noise PE runs. This is a new feature that affects parameter estimation. Changes should be purely additive; no general current functionality should be affected unless the new model is explicitly called.
Motivation
Sampling the overall phase of a waveform can greatly negatively impact convergence during PE. Specifically, likelihood calculations can be very sensitive to small changes in phase. There are existing models (i.e.
MarginalizedPhase) that circumvent this by completely marginalizing over the phase analytically. However, the assumptions behind that model break down when imposing a gated model.This PR adds a model that numerically marginalized over an overall phase term while also applying a gate. Marginalization is conducted using a Riemann sum, similar to the
GatedGaussianMargPolmodel. Unlike the marginalized polarization model, which splits the template into plus and cross polarizations, this new model splits the template into terms proportional to the cosine and sine of the overall phase.Specifically, if the template
hcontains an overall phase termphi, the model assumesh = h_c * cos(phi) + h_s * sin(phi). Then, to calculate the likelihood, the model:h_candh_sas well as the gated counterparts,<d - h, d - h>in terms ofh_candh_swithout the explicit phase dependence,cos(phi)andsin(phi)as necessary), andContents
A new model,
GatedGaussianMargPhase, has been added toinference/models, which conducts the procedure above to calculate the likelihood. This class necessitates a new waveform generator,FDomainDetFrameTwoPhaseGenerator, to generateh_candh_s.Testing performed
Below are the PE results of a zero-noise ringdown injection with
lmns = 221 331. The first plot usesGatedGaussianNoiseand samples overphi220andphi330, and the second usesGatedGaussianMargPhase, marginalizing overphi220and sampling overphi330. Red lines indicate the injected parameters. Both runs used theTdQNMfromFinalMassSpinapproximant and apply a 1 second gate ending at the injectedtc. Using 256 cores with MPI, the GGN run finished with ~3 million likelihood calls in ~1 hr 20 min, while the marginalized run finished with ~400 thousand likelihood calls in ~20 minutes.