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

Hide dynamically defined metaclass base from Sphinx. #4532

Merged
merged 1 commit into from Nov 14, 2013

Conversation

takluyver
Copy link
Member

The inheritance diagram extension in Sphinx 1.2 stores class references, which Sphinx then tries to pickle. This chokes on our dynamically defined base classes from the with_metaclass function. Starting the name with an underscore makes Sphinx ignore the class.

(Bug obscurity/fix simplicity) ratio is high.

The inheritance diagram extension in Sphinx 1.2 stores class references,
which Sphinx then tries to pickle. This chokes on our dynamically
defined base classes from the with_metaclass function. Starting the name
with an underscore makes Sphinx ignore the class.

(Bug obscurity/fix simplicity) ratio is high.
@takluyver
Copy link
Member Author

For cross-reference, I have filed a bug against Sphinx for this, but I imagine it's fairly low priority there, since it only affects the combination of an extension and fairly weird subject code.

minrk added a commit that referenced this pull request Nov 14, 2013
Hide dynamically defined metaclass base from Sphinx.
@minrk minrk merged commit 8b7876b into ipython:master Nov 14, 2013
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
…aclasses

Hide dynamically defined metaclass base from Sphinx.
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.

None yet

2 participants