python fit throws error about prophet

[abuccts]

Hi @datafool ,
I met the same problem with you, environment python 2.7 and anaconda 2.2. I solved it by upgrading pandas from version 0.17.1 to 0.19.2.
I think you can upgrade pandas to the newest version by python -m pip install --upgrade pandas and the original code should work.

[iampawansingh]

Hi @abuccts thanks! yes upgrade to newest version works fine. I think requirement of latest version of pandas should go in setup.py so that pandas is upgraded automatically, rather than we facing the problem and then updating it

[abuccts]

I agree with you. Either changing pandas.Series.apply to pandas.Series.dt.days or adding a minimum pandas version will fix this issue.

I can make a PR for this. But which solution will be better?

[iampawansingh]

I will go with pandas.Series.dt.days, as this is pandas inbuilt so all the heavy lifting will happen at the same place, unlike apply, and as per my understanding apply would be slower than internal pandas functions

[abuccts]

That's right. pandas.Series.dt.days is more efficient, but is unavailable in some old versions of pandas, I have no idea about the minimum version.

[iampawansingh]

I checked the pandas documentation back till 0.15, pandas.Series.dt.days is available from 0.16 onward. Also, 0.16 came back in June 2015, chances are everyone might have shifted to a higher version.

[seanjtaylor]

Thanks for diagnosing and fixing this! Will merge the PR soon.

[raymondmars]

Nice ! I met same problem today.

[deehzee]

The problem is still there. Using pandas 0.20.1 (also tested on pandas 0.19.2) and fbprphet 0.1.1.