Auto_TS icon indicating copy to clipboard operation
Auto_TS copied to clipboard

Prophet 1.0.1 vs fbProphet

Open ghost opened this issue 3 years ago • 8 comments

Would we be able to upgrade to the newer version of Prophet, that is now just called prophet? It seems like it would be an easy fix overall that I would be happy to take on.

ghost avatar Apr 29 '22 14:04 ghost

Hi @KarateKat95 👍 This is a very good idea. Can you please create the changes in your forked version and submit a pull request with all the necessary changes? You have to test it first that your changes work!

Thanks AutoViML

AutoViML avatar May 01 '22 15:05 AutoViML

I have completed that. Thanks!

KarateKat94 avatar May 04 '22 23:05 KarateKat94

But doesn't work yet. I opened new pull request.

turkalpmd avatar Sep 16 '22 02:09 turkalpmd

But doesn't work yet. I opened new pull request.

@turkalpmd See #82, where I already made this pull request a few months ago. It just needs to be merged by @AutoViML.

ghost avatar Sep 16 '22 02:09 ghost

But doesn't work yet. I opened new pull request.

@turkalpmd See #82, where I already made this pull request a few months ago. It just needs to be merged by @AutoViML.

Yes, I want to trying this today but it didnt work properly, after that. To be honest, I created a new pull request without checking the issue section. Then I saw you did something similar. I can delete it of course, but maybe a new notification will go away to @AutoViML . It is very easy changing. Thank you.

turkalpmd avatar Sep 16 '22 02:09 turkalpmd

Hi @KarateKat95 and @turkalpmd Thank you so much for filing the pull requests. I am very concerned about changing from FBProphet to Prophet without testing it - can you guys please test it with a few datasets before I approve it? 👍

I am sure it will work fine but just want to be sure. Thanks AutoVimal

AutoViML avatar Sep 18 '22 23:09 AutoViML

Hi @KarateKat95 and @turkalpmd Thank you so much for filing the pull requests. I am very concerned about changing from FBProphet to Prophet without testing it - can you guys please test it with a few datasets before I approve it? +1

I am sure it will work fine but just want to be sure. Thanks AutoVimal

I will testing this. But I didn't change any architecture, I changed just library name.

turkalpmd avatar Sep 20 '22 20:09 turkalpmd

Hi @KarateKat95 and @turkalpmd Thank you so much for filing the pull requests. I am very concerned about changing from FBProphet to Prophet without testing it - can you guys please test it with a few datasets before I approve it? +1 I am sure it will work fine but just want to be sure. Thanks AutoVimal

I will testing this. But I didn't change any architecture, I changed just library name.

Thanks @turkalpmd; I will run a few tests with some datasets on my end as well this weekend.

ghost avatar Sep 21 '22 01:09 ghost

Hi @KarateKat94 @turkalpmd

Any updates? Is the upgrade and change from one version to another smooth? Can you please update the code everywhere there is an import statement of FB prophet to Prophet?

I'd like you to test it on a few datasets after installing and importing auto-ts with the new version of prophet in your computers.

You can use pypi-test.org by uploading and testing your new versions of auto-ts. Hope that helps,

Thank you in advance 👍 AutoVimal

AutoViML avatar Sep 27 '22 12:09 AutoViML

I was on-call this week. I can be doing some updates tomorrow. I don't know if I haven't tested the codes on GitHub before, but I'll learn. I'll fix all other import commands today. Thank you for your patience. Kind regards.

turkalpmd avatar Sep 27 '22 14:09 turkalpmd

Hi @KarateKat94 @turkalpmd

Any updates? Is the upgrade and change from one version to another smooth? Can you please update the code everywhere there is an import statement of FB prophet to Prophet?

I'd like you to test it on a few datasets after installing and importing auto-ts with the new version of prophet in your computers.

You can use pypi-test.org by uploading and testing your new versions of auto-ts. Hope that helps,

Thank you in advance +1 AutoVimal

$ pip install git+git://github.com/AutoViML/Auto_TS

Collecting git+git://github.com/AutoViML/Auto_TS Cloning git://github.com/AutoViML/Auto_TS to /tmp/pip-req-build-0xdvyn41 Running command git clone --filter=blob:none --quiet git://github.com/AutoViML/Auto_TS /tmp/pip-req-build-0xdvyn41 fatal: unable to connect to github.com: github.com[0: 140.82.121.4]: errno=Connection timed out

error: subprocess-exited-with-error

× git clone --filter=blob:none --quiet git://github.com/AutoViML/Auto_TS /tmp/pip-req-build-0xdvyn41 did not run successfully. │ exit code: 128 ╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip. error: subprocess-exited-with-error

× git clone --filter=blob:none --quiet git://github.com/AutoViML/Auto_TS /tmp/pip-req-build-0xdvyn41 did not run successfully. │ exit code: 128 ╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

fbprophet contain just test/test_auto_ts.py line 18 and auto_ts/build_prophet.py line 19-22 32 and 545

this changing is very simple. unfortunately Git command and pip command doesnt work.

turkalpmd avatar Sep 30 '22 18:09 turkalpmd

Hi @turkalpmd : There is a simple way to install auto-ts without dependencies 👍

pip install auto-ts --ignore-installed --no-deps

Then you can install FB prophet independently using the following command:

conda install -c conda-forge fbprophet

Hope this helps, AutoVimal

AutoViML avatar Oct 11 '22 11:10 AutoViML

Hi @turkalpmd : There is a simple way to install auto-ts without dependencies 👍

pip install auto-ts --ignore-installed --no-deps

Then you can install FB prophet independently using the following command:

conda install -c conda-forge fbprophet

Hope this helps, AutoVimal

I will try this weekend

turkalpmd avatar Oct 26 '22 11:10 turkalpmd

Hi @turkalpmd @KarateKat94 I have updated Auto_TS with the latest version of prophet 1.0 - please double check if this works and let me know Thank you for all you pull requests and commitment! AutoVimal

AutoViML avatar Feb 22 '23 13:02 AutoViML

I will try in next 24 hour :D. Thank you.

turkalpmd avatar Feb 22 '23 15:02 turkalpmd

Yes It is works perfect. But I found another bug from univariable notebook, and you must change pypi release, pypi installed old .68 version;

---------------------------------------------------------------------------
UnboundLocalError                         Traceback (most recent call last)
/tmp/ipykernel_7312/3941123415.py in <module>
      4                 model_type=['best'],dask_xgboost_flag=False,
      5                 verbose=2)
----> 6 model.fit(traindata, ts_column,target)

~/.local/lib/python3.9/site-packages/auto_ts/__init__.py in fit(self, traindata, ts_column, target, sep, cv)
    573 
    574             self.ml_dict[name]['model'] = model
--> 575             self.ml_dict[name]['forecast'] = forecast_df_folds
    576             self.ml_dict[name][self.score_type] = score_val
    577             self.ml_dict[name]['model_build'] = model_build

UnboundLocalError: local variable 'forecast_df_folds' referenced before assignment

turkalpmd avatar Feb 26 '23 22:02 turkalpmd