Closed gotten closed 8 months ago
Also setting ndim=5 in dynesty is expected to crash on making the plots because of the hardcoded best_pars. It does converge as it used to.
Edit: it didn't crash because the tests do not trigger stationtrackplot.
Also closes #38 .
Yes, I was being lazy by inserting the RV prior into the multivariate PPF. It's an independent measure anyway. We can make it a separate prior that can be overridden (with another normal distr, or uniform distr). This might give a slight performance boost too because the multivariate PPF has less dimensions.
I can separate out the RV prior but I don't know the best way of implementing an override.
Closes #35.
Potentially could use another diagnostic plot where host star parameters are compared with the prior to make sure it is consistent.
Might crash if Gaia has no RV datapoint.