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.

Sign up for GitHub

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

Jump to bottom

sphinx-build failures TypeError: can only concatenate str (not "_StrPath") to str #1482

Closed
dweindl opened this issue Oct 8, 2024 · 1 comment
Closed

sphinx-build failures TypeError: can only concatenate str (not "_StrPath") to str #1482

dweindl opened this issue Oct 8, 2024 · 1 comment
Assignees
@dweindl
Labels
documentation fixed but not released

Comments

@dweindl
Copy link
Member

dweindl commented Oct 8, 2024 •
edited
Loading

sphinx-build is failing with:

Exception occurred:
  File "<template>", line 2, in top-level template code
TypeError: can only concatenate str (not "_StrPath") to str

e.g. https://github.com/ICB-DCM/pyPESTO/actions/runs/11228723603/job/31213085069
@dweindl
Copy link
Member Author

dweindl commented Oct 8, 2024

That's a sphinx>=8 issue. Not sure why it popped up just now. I guess, some updated dependency that so far required sphinx<8.

@dweindl dweindl self-assigned this Oct 8, 2024
dweindl added a commit to dweindl/pyPESTO that referenced this issue Oct 8, 2024
@dweindl
Fix sphinx config
48502f5 
Convert path-li_StrPathke object to str. Changed in sphinx>=8.

Fixes ICB-DCM#1482.
@dweindl dweindl mentioned this issue Oct 8, 2024
Merged
dweindl added a commit to dweindl/AMICI that referenced this issue Oct 8, 2024
@dweindl
Update sphinx config for sphinx8
8056f6f 
Not a problem yet, but this change prevents issues once we upgrade to sphinx 8 and doc2path will return something path-like.

See ICB-DCM/pyPESTO#1482
@dweindl dweindl mentioned this issue Oct 8, 2024
Merged
dweindl added a commit to AMICI-dev/AMICI that referenced this issue Oct 8, 2024
@dweindl
Pin sphinx<8 / update sphinx config for sphinx8 (#2532)
2784887 
Some dependency lifted there sphinx<8 requirement, but this leads to a number of issues with other dependencies. Therefore, require sphinx<8 for now.

Anyways fix some upcoming issue with sphinx 8 when doc2path will return something path-like (see ICB-DCM/pyPESTO#1482).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dweindl dweindl

Labels
documentation fixed but not released
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dweindl @PaulJonasJost