search

nespinoza / juliet

A versatile modelling tool for transiting and non-transiting (single and multiple) exoplanetary systems
MIT License
55 stars 30 forks source link

Fixed bug with transit time priors for more than one planet #82

Closed ssagear closed 1 year ago

ssagear commented 2 years ago

Hello,

I came across the same error as in Issue #81 that occurs when fitting more than one planet with TTVs. In setting the lightcurve prior dict, when a transit time prior is detected, 'parametrization' is only set to 'T' for the last planet in the system. I changed the indexing in line 570 so 'parametrization' = 'T' is set for all planets with transit time priors. After doing this, fitting multi-planet systems with transit time priors seems to proceed as expected.

Best, Sheila

ThomasGWilson commented 1 year ago

Hi @nespinoza,

Independently found the same bug as @ssagear with the same fix. I would recommend and support this pull request.

Thanks, Tom