-
-
Notifications
You must be signed in to change notification settings - Fork 2k
More legible inheritance diagrams #2292
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
More legible inheritance diagrams #2292
Conversation
Resolve links to "canonical" location of classes to "api" location of classes so that Bases and inheritance diagrams work.
|
👍 |
|
Related question unrelated to Astropy--I was trying to include an inheritance diagram for a different project in its docs. But I couldn't figure out how to say, given a class, generate an inheritance diagram including this class and all its subclasses. Any ideas? Would one of these plugins help? |
|
By the by, if this is merged don't let me forget to also merge it into astropy-helpers. |
|
The |
|
@mdboom - this looks good to me! Is it ready to merge? |
More legible inheritance diagrams
|
@mdboom - this is a tricky way to solve this, but it's nice, and I like the result! The one concern is that this may have unintended consequences when there's name overlap. E.g., if in the future someone adds a warning to This was still fine right now, because it's better than the alternative of not merging, but I want to make sure I understand this in case it ends up happening down the road. |
…itance-diagrams More legible inheritance diagrams


This makes the inheritance diagrams have less super tiny text by including only the class name, not the fully-qualified class name. This is most acutely different in
astropy.coordinates, for example.This also includes #2288 (and is really just a follow-on to it).