Skip to content
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

pins jinja2 version < 3 #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

janash
Copy link

@janash janash commented Mar 10, 2023

This PR pins the version of jinja2 installed in the environment. The package pandas_profiling uses 'escape' from 'jinja2.utils', which is not present in jinja2 version 3. The environment file as in the original repo results in installation of the latest jinja version.

Tested on Windows 11 and Ubuntu 20.04 (through WSL).

Without pinning, jinja2 version 3.1.2 is installed. With pinning, jinja2 version 2.11.3 is installed.

Output from 0b-check_dependencies.ipynb without pin:

[ OK ] pandas version 1.0.5
[FAIL] Error in importing pandas_profiling: cannot import name 'escape' from 'jinja2.utils' (/home/janash/miniconda3/envs/bestpractices/lib/python3.7/site-packages/jinja2/utils.py)
[ OK ] matplotlib version 3.2.2
[ OK ] seaborn version 0.10.1
[ OK ] sklearn version 0.22.2.post1
[ OK ] scipy version 1.4.1
[ OK ] tqdm version 4.43.0
[ OK ] jupyter_client version 7.4.9
[ OK ] ipywidgets version 8.0.4
[ OK ] torch version 1.3.1

After pinning jinja2:

[ OK ] pandas version 1.0.5
[ OK ] pandas_profiling version 2.8.0
[ OK ] matplotlib version 3.2.2
[ OK ] seaborn version 0.10.1
[ OK ] sklearn version 0.22.2.post1
[ OK ] scipy version 1.4.1
[ OK ] tqdm version 4.43.0
[ OK ] jupyter_client version 7.4.9
[ OK ] ipywidgets version 8.0.4
[ OK ] torch version 1.3.1

@janash janash force-pushed the jinja-version branch 2 times, most recently from 423e314 to 7b0795d Compare March 10, 2023 21:24
@janash janash changed the title pins jinja version < 3 pins jinja2 version < 3 Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant