-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Latest version of MarkupSafe doesn't work with Great Expectations #10908
Comments
Hi, thank you for reporting this issue. I'll share my investigation below: (.venv) ➜ python --version
Python 3.10.14
(.venv) ➜ pip --version
pip 25.0.1
(.venv) ➜ pip install --no-cache-dir great-expectations==1.3.5
(.venv) ➜ pip freeze | grep MarkupSafe
MarkupSafe==3.0.2
(.venv) ➜ pip install pipdeptree
(.venv) ➜ pipdeptree -r -p markupsafe
MarkupSafe==3.0.2
└── Jinja2==3.1.5 [requires: MarkupSafe>=2.0]
├── altair==4.2.2 [requires: Jinja2]
│ └── great-expectations==1.3.5 [requires: altair>=4.2.1,<5.0.0]
└── great-expectations==1.3.5 [requires: Jinja2>=2.10]
|
I will try upgrading Jinja and let you know on a weekday. Let me know if you require any other information, I will try to share my pip freeze output as well. I am installing it in a Databricks environment. Will send you more information. |
Thanks, that would be greatly appreciated! Our team decided to drop support for Jinja2 v2 regardless. I've marked this issue as resolved since that is likely the source of this error. However, please let us know if your issue persists for any other reason. |
Describe the bug
When importing great-expectations with the latest version, there's an issue with MarkupSafe. MarkupSafe latest version removes
soft_unicode
, which gx uses - thus resulting in the errorcannot import name 'soft_unicode' from 'markupsafe'
To Reproduce
Install latest version of gx, and
import great_expectations
Expected behavior
import should not fail
Environment (please complete the following information):
Comments
Please set a constraint in your pyproject for MarkupSafe>=2 (or whatever works) and MarkupSafe <3
The text was updated successfully, but these errors were encountered: