Skip to content

Conversation

@adorilson
Copy link
Contributor

@adorilson adorilson commented Oct 28, 2025

It prevents LaTeX PDF build error: Too deeply nested. (Regression from #139509.).

Besides, it also makes the diagram richer.

It is an alternative to #140709.

#139588: Docs: Convert the execution model "textual" diagram to a visual diagram

image image

📚 Documentation preview 📚: https://cpython-previews--140721.org.readthedocs.build/

@StanFromIreland StanFromIreland changed the title gh139588: Docs: Convert the execution model "textual" diagram to a visual diagram gh-139588: Docs: Convert the execution model "textual" diagram to a visual diagram Oct 28, 2025
@StanFromIreland
Copy link
Member

Note that the error has now been fixed by the other PR.

Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

How does this work with translations? Is it extracted by the gettext builder?

@rffontenelle
Copy link
Contributor

gettext builder extracts the current text for translation, but the product of this pull request is not extracted for translation. (just tested)

@StanFromIreland
Copy link
Member

StanFromIreland commented Oct 28, 2025

Thanks for testing, Rafael.

As such, I am -0 on this, although we could keep the textual ones just for translations (instead of the graphic) using the .. only:: translation tag I added recently.

@rffontenelle
Copy link
Contributor

The best would be to have this graphic but the labels translated, but I don't even know if it is possible.

@StanFromIreland
Copy link
Member

It also breaks on EPUB:

WARNING: dot command 'dot' cannot be run (needed for graphviz output), check the graphviz_dot setting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants