Updated functionality for lnf_rv

Open kmhambleton opened this issue 1 year ago • 1 comments

Updated _calculate_cf function to adjust lnf calculation for radial velocities. The noise nuisance parameter for rvs no longer contains the rv amplitude.

Closes #752 and #895

TODO:

[ ] update documentation to mention change (and remove in release-2.5 branch)
[ ] add warning when using lnf with rv to mention change in behavior (and remove in release-2.5 branch)

Jun 13 '24 12:06 kmhambleton

looks like the base bugfix branch (#907) is also failing CI, so likely due to upstream changes. We can move forward with this if those errors are all the same, but then should probably try to get CI passing again before releasing.

Jun 21 '24 11:06 kecnry

Confirmed that the failing tests were the same as on the base branch. @kmhambleton does the warning I added look reasonable to you (since this is a behavior change and we want to make sure anyone that was relying on the old behavior for some reason will see this change - we can remove the warning in the next minor release)?

Aug 18 '24 20:08 kecnry