MNGuenther/allesfitter

Wotan not really optional

Closed this issue · 2 comments

jvines commented

Hi,

I was trying to run allesfitter without wotan installed and it wouldn't run as it wouldn't import flatten in time_series and thus fail in lightcurves when trying to import flatten.

Cheers

MNGuenther commented

Hi José,

Can you replace your local file lightcurves.py with this one - lightcurves.txt - and let me know if it solves things? If it does, I can go ahead and push 1.2.4. (You'll need to rename it from .txt into .py manually because github prohibits attaching .py files)

I think the issue is that I had my own function called flatten in time_series, but apparently I decided some time ago to remove it and use wotan's flatten instead. But the old lightcurves.py still tried to import my own flatten from time_series.

The general allesfitter user actually doesn't need any of these functions, they are just for making fancy TESS plots or searching for transits etc. So wotan should remain optional, indeed.

Best
Max

MNGuenther commented

Solved with version 1.2.4.