Allow sampling in aligned spin and spin magnitude
All threads resolved!
All threads resolved!
Compare changes
+ 0
− 11
@@ -667,17 +667,6 @@ class TestPriorClasses(unittest.TestCase):
Maintenance will be performed on git.ligo.org, containers.ligo.org, and docs.ligo.org on Tuesday 1 April 2025 starting at approximately 9am PDT. It is expected to take around 30 minutes and there will be several periods of downtime throughout the maintenance. Please address any comments, concerns, or questions to the helpdesk.
This MR will allow us to sample in the spin magnitude and the aligned component of the spin, rather than magnitude and tilt.
The reason to do this is that the aligned component of the spin is the best measured per component spin parameter. From preliminary tests, it looks like this will improve convergence with at least MultiNest.
Currently, this will only support the vanilla flat in magnitude, isotropic in orientation prior. It would take more effort to figure out the generic case.
This required making the CBCPriorDict
a subclass of the ConditionalPriorDict
, I know that people have reported seeing issues with this in the past, but I don't remember what they were. @moritz.huebner @rory-smith?
EDIT I think the issue with the conditional priors was the _check_if_prior_can_be_sampled
method which I have removed. That was falling over for some conditions/constraints. I think the current version should be suitably robust.