-
Notifications
You must be signed in to change notification settings - Fork 19
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
Prior uniform in m1-m2 space with a bound in chirp mass and mass ratio #164
Conversation
src/jimgw/single_event/transforms.py
Outdated
@@ -24,6 +26,131 @@ | |||
) | |||
|
|||
|
|||
@jaxtyped(typechecker=typechecker) | |||
class UniformInComponentMassSecondaryMassTransform(ConditionalBijectiveTransform): |
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.
Is this a transform or a prior? Also, is this used in the testing script?
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.
It is a transform
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.
There are two minor comments for this PR. Once they are addressed I am happy to merge this
src/jimgw/single_event/utils.py
Outdated
@@ -38,6 +38,31 @@ def inner_product( | |||
return 4.0 * jnp.real(trapezoid(integrand, dx=df)) | |||
|
|||
|
|||
def Mc_m1_to_m2(Mc: Float, m1: Float): |
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.
The return typing hint is missing here
Added the transform into the Pv2 testing script and updated the |
This PR is to have the prior defined as uniform in component mass, while the bound is defined by chirp mass and mass ratio.
Although using the bound-to-unbound in chirp mass and mass ratio space and the current initialization procedure would solve the sampling side problem, the evidence estimation will be off due to a constant shift in the log posterior. This PR is to fix this issue.
The following test script is used, with the scatter plot of the samples shown.