-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump pandas and sqlalchemy #400
Conversation
Found some issues with Pandas while running all the notebooks, will try to fix later |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a stack for working on this. There is an error on CI wrt. to PyCaret. I hope it will turn out well. Maybe PyCaret exclusively needs pandas<2 or SQLAlchemy<2?
File /opt/hostedtoolcache/Python/3.11.8/x64/lib/python3.11/site-packages/pycaret/internal/memory.py:25
ImportError: cannot import name '_format_load_msg' from 'joblib.memory' (/opt/hostedtoolcache/Python/3.11.8/x64/lib/python3.11/site-packages/joblib/memory.py)
-- https://github.com/crate/cratedb-examples/actions/runs/8604266577/job/23577980009?pr=400#step:6:859
If this is the case, we may need to split up the ingredients to be separated by pandas/SQLAlchemy major versions in one way or another, either by providing dedicated requirements.txt
files, or by separating notebooks into different folders.
There is an error on CI wrt. to PyCaret: I discovered the same error here: This adjustment aims to mitigate it. Effectively, just downgrade to joblib 1.3. See also: |
There is also a compatibility flaw with Dask and most recent Python 3.11.9, orthogonal to the Joblib 1.4 compatibility issue. |
The TSML stack modulo LLM/LangChain needed two fixes, as outlined above. After rebasing, relevant obstacles should be remedied for you. |
More havocs caused by recent updates had to be remedied, to make relevant artefacts work on Google Colab again. Thanks for your support, @marijaselakovic.
@surister: You may now want to rebase again, to be in a better position than before. |
Hi there. I just rebased your patch, also resolving slight conflicts, and would like to verify it on CI. However, I can't push easily, because you forked the repository into a private one on your account. May I ask you to add corresponding permissions, so I can push, if you are fine with that?
|
This patch is currently blocked on errors like this, evaluated on behalf of GH-430 1:
-- https://github.com/crate/cratedb-examples/actions/runs/8975962618/job/24651748395?pr=430#step:6:1839 Footnotes |
Summary of the changes / Why this is an improvement
Update pandas and sqlalchemy to latest (or fairly modern versions)
Checklist