Hello all,
The precenter argument for lombscargle was made useless after #21277 was merged last year—and is less useful than the added floating_mean argument (as already detailed in the Notes, copied below for reference). #23476 would like to mark the argument precenter for deprecation.
Notes
The algorithm used will not automatically account for any unknown y offset, unless
floating_meanisTrue. Therefore, for most use cases, if there is a
possibility of a y offset, it is recommended to setfloating_meantoTrue.
Furthermore,floating_meanaccounts for sample weights, and will also correct for
any bias due to consistently missing observations at peaks and/or troughs.
The legacy concept of “pre-centering” entails removing the mean from parametery
before processing, i.e., passingy - y.mean()instead of setting the parameter
floating_meantoTrue.
Specifically, the functionality of precenter can be substituted by passing y - y.mean() to y. Let me know if there are any questions or concerns over on #23476.
Thanks,
Adam