increase stack size to solve import recursion problem #912
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
documentation
Contributor
|
@Czaki have you run this locally with napari/#8051? When I tried increasing the recursion limit I still hit the same error, but maybe I was doing something wrong. I've updated the description and I'll approve and merge, but I'm just curious if you saw a successful build 😆 |
DragaDoncila
approved these changes
Jan 16, 2026
Contributor
DragaDoncila
left a comment
DragaDoncila
left a comment
There was a problem hiding this comment.
Happy to try this so we can get that PR in.
Contributor
Author
Yes. Multiple times. Using isolated venvs. Maybe your environment contains even more packsges. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
References and relevant issues
unlock napari/napari#8051
Description
Currently napari/napari#8051 is failing the docs build because the
pydepsgraph being built during the "generating docs for UI section: Application status bar" step hits a recursion error. Although that PR doesn't really touch our code that much (and certainly doesn't touch the Application status bar), it does introduce a couple of new gallery dependencies, and their presence in the docs environment may be causing this error.As I have problems with finding the best way to improve pydeps, this PR is just increasing the stack limit to merge the example in 0.7.0 release.