-
Notifications
You must be signed in to change notification settings - Fork 32
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
[FEATURE] Autogenerate class/module diagrams for Python #695
Comments
Hi, I would like to begin contributing to this project and this issue seems to be a good candidate to start with. Is this issue still needed and up to date? If it's needed, a couple of questions on the diagrams:
Thanks |
Hi @emabre , Great to hear you are willing to pick this up!
The issue still seems to be up-to-date, indeed. I will make some decisions on where in the docs to add the graph and update the description.
I think pruning is a good idea. There are multiple ways to do that (e.g., splitting up into smaller diagrams, removing details, ...). Any improvement is already a huge one, so I will leave that up to you. If you prefer, you can also send an email to martijn.govers@alliander.com, and we can briefly talk about the options)
I also think that SVG is indeed the way to go, but I am not an expert on this. Note that we already have some graphs embedded into the docs using Hope that helps! If you have any other questions, feel free to ask them here, on a draft PR, or otherwise contact me directly via martijn.govers@alliander.com Martijn |
@mgovers thank you very much for your reply. I'll reach out in case I have further questions. |
Use
pyreverse
(distributed together withpylint
by default) to create module graph and class diagramdocs/advanced_documentation/python-wrapper-design.md
that contains a small introduction + the diagrams.docs/index.md
part on Advanced Documentation.docs/advanced_documentation/core-design.md
but that only talks about the design of the C API core. Therefore, that is not the right location to put it.Relates to #749 and #525
The text was updated successfully, but these errors were encountered: