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

Address Shapely 2.0 Deprecations #1930

Merged
merged 2 commits into from
Nov 13, 2021
Merged

Conversation

akrherz
Copy link
Contributor

@akrherz akrherz commented Nov 12, 2021

#1914 commits cherry-picked to v0.20.x branch, refs #1913

@akrherz
Copy link
Contributor Author

akrherz commented Nov 12, 2021

Unsure of docs failure

after excluding 0 files that had previously been run (based on MD5).

embedding documentation hyperlinks...
embedding documentation hyperlinks for gallery... [  2%] index.html
embedding documentation hyperlinks for gallery... [  4%] gridliner.html

Extension error (sphinx_gallery.docs_resolv):
Handler <function embed_code_links at 0x7f0005bd0ee0> for event 'build-finished' threw an exception (exception: list indices must be integers or slices, not str)
make: *** [Makefile:47: html] Error 2
make: Leaving directory '/root/project/docs'

@dopplershift
Copy link
Contributor

dopplershift commented Nov 12, 2021

It's caused by sphinx-gallery 0.10.0 and sphinx 4.3.0 being incompatible. In theory a new release of sphinx-gallery is coming. Not sure what we want to do in the meanwhile.

Copy link
Contributor

@greglucas greglucas left a comment

Choose a reason for hiding this comment

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

Sphinx-gallery released a new version which looks like it helped here. Thanks @akrherz!

@greglucas greglucas merged commit 5cfb76b into SciTools:v0.20.x Nov 13, 2021
@dopplershift dopplershift added this to the 0.20.2 milestone Nov 15, 2021
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.

3 participants