Skip to content

Conversation

AA-Turner
Copy link
Contributor

Reference issue

What does this implement/fix?

Support for string values in html_sidebars has been officially removed since Sphinx 2.0, but somehow seemed to work. Nevertheless, this resovlves the issue before a new version of Sphinx is resolved.

Additional information

A

@AA-Turner AA-Turner requested a review from tupui as a code owner July 16, 2024 19:03
@github-actions github-actions bot added the Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org label Jul 16, 2024
@nickodell nickodell added the CI Items related to the CI tools such as CircleCI, GitHub Actions or Azure label Jul 16, 2024
Copy link
Member

@lucascolley lucascolley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @AA-Turner ! Happy if CI is happy.

@lucascolley lucascolley changed the title Fix type of html_sidebars value in conf.py DOC: Fix type of html_sidebars value in conf.py Jul 16, 2024
@tylerjereddy tylerjereddy added this to the 1.15.0 milestone Jul 16, 2024
@melissawm
Copy link
Member

It looks like we now have some new autosummary warnings:

https://app.circleci.com/pipelines/github/scipy/scipy/28817/workflows/93e254c6-dbc9-4d06-94e2-ceceec5fb7fb/jobs/97225?invite=true#step-104-341480_107

They look pretty straightforward so if you don't have the bandwidth @AA-Turner I'm happy to push a commit for that. Thanks!

@AA-Turner
Copy link
Contributor Author

Feel free! I plan to release Sphinx 7.4.5 with the linked fix, and after that look into a few other autosummary issues that have been reported (sphinx-doc/sphinx#12589, sphinx-doc/sphinx#9743 (comment)). If this is one of those, or related, please let me know.

A

May be fixed by upstream in the future.

[docs only]
@melissawm
Copy link
Member

It seems like it was indeed sphinx-doc/sphinx#12589, I am suppressing these warnings for now. Thanks!

Copy link
Member

@tupui tupui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks folks

@tupui tupui merged commit 5723615 into scipy:main Jul 16, 2024
@AA-Turner AA-Turner deleted the fix-html_sidebars branch July 16, 2024 22:02
@tylerjereddy
Copy link
Contributor

From local testing, I believe this also needs a backport..

@tylerjereddy tylerjereddy added the backport-candidate This fix should be ported by a maintainer to previous SciPy versions. label Aug 13, 2024
@tylerjereddy tylerjereddy modified the milestones: 1.15.0, 1.14.1 Aug 13, 2024
@tylerjereddy tylerjereddy mentioned this pull request Aug 13, 2024
5 tasks
@tylerjereddy tylerjereddy removed the backport-candidate This fix should be ported by a maintainer to previous SciPy versions. label Aug 20, 2024
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 Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: documentation build failing?
6 participants