light-curve / light-curve-feature

Time-series feature extraction Rust crate
GNU General Public License v3.0
9 stars 0 forks source link

Linexp with MCMC and default bounds panics #138

Closed hombit closed 1 year ago

hombit commented 1 year ago

When I working on porting LinexpFit to Python, I've found that LinexpFit('mcmc')(t, -mag, magerr) (note minus magnitude, not flux here):

thread '<unnamed>' panicked at 'Left boundary is larger than right one: 0 > -31835.271', /Users/hombit/.cargo/registry/src/index.crates.io-6f17d22bba15001f/light-curve-feature-0.6.0-beta.0/src/nl_fit/mcmc.rs:192:21
---------------------------------------------------------------------------
PanicException                            Traceback (most recent call last)
File <timed exec>:1

PanicException: Left boundary is larger than right one: 0 > -31835.271

Data to reproduce: https://ztf.snad.space/dr17/csv/821107100005285