search

AutoViML / Auto_TS

Automatically build ARIMA, SARIMAX, VAR, FB Prophet and XGBoost Models on Time Series data sets with a Single Line of Code. Created by Ram Seshadri. Collaborators welcome.
Apache License 2.0
729 stars 114 forks source link

Change requirements to `pyyaml` instead of `pyyaml==5.4.1` #104

Closed pranavvp16 closed 11 months ago

pranavvp16 commented 11 months ago

Build for pyyaml==5.4.1 is disabled due to cpython updates. This makes installation of auto-ts fail and completely unusable currently. Let me know why is the pyyaml version specific to 5.4.1, if there is no good reason I'll make PR from fork which will fix installation of auto-ts pyyaml issue

AutoViML commented 11 months ago

Hi @pranavvp16 👍

yes go ahead and make PR and I'll approve it if it works. Can you please make sure that certain pyyaml dependencies are not violated with multiple versions (such as 5.4.1 and 6.0 etc.)? I don't want to approve the PR and have a bunch of people complain it breaks things. Thanks Auto Vimal

pranavvp16 commented 11 months ago

@AutoViML Creating a PR, as nothing seems to breaking with the upgrade in pyyaml==6.0.1, I tried running an example from this repository with my fork and everything works fine. Even installation of auto-ts has become easy in google collab now. Here is the example

And I have also worked with pyyaml=6.0.1 version for a month and faced no errors.

Please get this merged ASAP as there's a pending PR that needs this to be fixed

AutoViML commented 11 months ago

Hi @pranavvp16 thanks for the pull request. I have made a slight change 👍 Hope this helps Auto Vimal

pranavvp16 commented 11 months ago

@AutoViML awesome I neglected that thanks !. One more thing how much time will it take to reflect this change in pip install auto-ts as it is still trying to install the older version of Pyyaml