Skip to content

Conversation

tupui
Copy link
Member

@tupui tupui commented Mar 29, 2022

Pin was added in #15868 and since #15443 it can be removed.

[skip actions] [skip circle]
@tupui tupui added the CI Items related to the CI tools such as CircleCI, GitHub Actions or Azure label Mar 29, 2022
@tupui tupui self-assigned this Mar 29, 2022
@tupui
Copy link
Member Author

tupui commented Mar 29, 2022

@rossbar seems like the issue is still there with the new version of numpydoc (1.2):

Traceback (most recent call last):
  File "/home/vsts/work/1/s/tools/refguide_check.py", line 46, in <module>
    from numpydoc.docscrape_sphinx import get_doc_object
  File "/opt/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/site-packages/numpydoc/docscrape_sphinx.py", line 11, in <module>
    from sphinx.jinja2glue import BuiltinTemplateLoader
  File "/opt/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/site-packages/sphinx/jinja2glue.py", line 15, in <module>
    from jinja2 import FileSystemLoader, BaseLoader, TemplateNotFound, contextfunction
ImportError: cannot import name 'contextfunction' from 'jinja2' (/opt/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/site-packages/jinja2/__init__.py)

@rossbar
Copy link
Contributor

rossbar commented Mar 29, 2022

Okay the root issue is reported here: sphinx-doc/sphinx#10291. Jinja 3.1.X (released ~5 days ago) is incompatible with sphinx 2.X and 3.X and the failing CI jobs look like they install sphinx 2.1.

numpydoc still supports all the way back to sphinx 1.8 (for now) so the jinja version will be pinned there, see numpy/numpydoc#369.

@tupui
Copy link
Member Author

tupui commented Mar 30, 2022

Nice and looks like you already have a release, let's try!

@tupui tupui closed this Mar 30, 2022
@tupui tupui reopened this Mar 30, 2022
@tupui tupui marked this pull request as ready for review March 30, 2022 09:04
@tupui tupui requested review from larsoner and andyfaff as code owners March 30, 2022 09:04
@tupui tupui added this to the 1.9.0 milestone Mar 30, 2022
[skip actions] [skip circle]
@tupui
Copy link
Member Author

tupui commented Mar 30, 2022

Now working fine (refguide_asv_check is green). I needed to also remove the pin on pip.

@tupui tupui merged commit 9c5d9a5 into scipy:main Apr 1, 2022
@tupui tupui deleted the jinja_pin branch April 1, 2022 08:19
patnr added a commit to patnr/scipy that referenced this pull request Apr 5, 2022
* master: (632 commits)
  Update _dual_annealing.py (scipy#15939)
  TST: stats: make `check_sample_var` two-sided (scipy#15723)
  DOC: sparse.linalg: add citations for COLAMD
  DOC: fix missing comma in conf
  ENH/MAINT: Version switcher from the sphinx theme (scipy#15380)
  MAINT: stats: remove support for `_rvs` without `size` parameter (scipy#15917)
  BUG: Handle base case for scipy.integrate.simpson when span along the axis is 0  (scipy#15824)
  MAINT: stats: adjust tolerance for failing test only
  MAINT: stats: adjust tolerance of failing TestTruncnorm
  MAINT: special: Clean up C style in ndtr.c
  CI: remove pin on Jinja2 (scipy#15895)
  STY: Remove white spaces
  MAINT: stats: exempt gilbrat from refguide_check
  MAINT: stats: rename continuous_gilbrat->continuous_gibrat
  MAINT: stats: correct name Gilbrat -> Gibrat
  [ENH] circvar calculated simply as 1-R (scipy#5747)
  DEP: deal with deprecation of ndim >1 in bspline
  MAINT: stats: more specific error type from `rv_continuous.fit` (scipy#15778)
  DOC: fix import in example in _morestats (scipy#15900)
  [BUG] make p-values consistent with the literature (scipy#15894)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Items related to the CI tools such as CircleCI, GitHub Actions or Azure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants