Add prophet to requirements #265
Conversation
| @@ -54,6 +54,7 @@ oauthlib==2.1.0 | |||
| openpyxl==3.1.2; python_version >= '3.6' | |||
| packaging==23.2; python_version >= '3.7' | |||
| pillow==10.0.1; python_version >= '3.8' | |||
| prophet==1.1.1; python_version >= '3.7' | |||
There was a problem hiding this comment.
This should be added to the Pipfile and locked — which I could not get to happen so best of luck to you!
Gloccck
force-pushed
the
add-prophet-requirement
branch
from
d92760d
to
2714c4e
Compare
| zipp==3.17.0; python_version >= '3.8' | ||
| zope.event==5.0; python_version >= '3.7' | ||
| zope.interface==6.1; python_version >= '3.7' | ||
| zstd==1.5.5.1 | ||
| alembic==1.12.1; python_version >= '3.7' |
There was a problem hiding this comment.
requirements-dev.txt seemed to contain both default and dev dependencies, I've regenerated it with a --dev-only option, so now it contains only the dev dependencies.
| @@ -6,6 +6,7 @@ name = "pypi" | |||
| [packages] | |||
| apache-superset = "==3.0.1" | |||
| cython = "==3.0.3" | |||
| holidays = "==0.23" | |||
There was a problem hiding this comment.
Version has to be set explicitly (newer version is not compatible with prophet).
|
Does |
It does.
However, to update lock file, I used |
Adds possibility of using predictive analytics by resolving missing dependency.
Note: prophet version higher than 1.1.1 is not compatible with current holidays version.