-
Notifications
You must be signed in to change notification settings - Fork 9
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]: Adding Engaging Visual Elements/Map in Documentation #94
Comments
hey @Fe-r-oz sorry we are just seeing this, we had something like this before, but this is MUCH clearer, we will probably merge it in later this week |
Hi @aarontrowbridge , No problem. I would like to have your thoughts on another approach. Let's call it a docstring based approach instead of a hard coded figure based approach. I think in the long run, a better, maintainable solution would be by using dynamic schematics created in docstring using Mermaid. This means that whenever a new functionality is added, we can just add update the code with ease instead of using hard coded schematics. Here is a rough version with default color scheme in Mermaid attached below. As it's a screenshot attached below, it's not clear, But it will be much clearer when rendered via browser. This approach might not be as fancy as the one above but it's maintainable and easily updateable. What do you suggest? I can submit a polished PR of this approach later this week. Please do let me know if there is any other flowcharts we can add, refer me to the paper (if any) and I will look into it as well |
Mermaid seems to flatten the hierarchical structure of the package. @Fe-r-oz Is this expected, or an artifact of not using the browser? For example, we see here that |
That looks good! Is there a way to tag what functions appear in the preview? Perhaps, a flag in the docstring parsed by the dynamic schematic that says "yes, this function should be displayed". My reason is that not all exported functions have the same priority for users. For example, |
Option 2 Option 3: We can only include Mermaid flowcharts is typically written in HTLML format |
I think |
i think including all exported functions is good. is there a way to attach hrefs to the source code or docs? |
Thanks for your suggestion. In the |
Feature Description
Importance
4
What does this feature affect?
Other information
Adding schematics like this allows the beginners to get a complete overview of the package in an instant. It also makes the documentation more engaging to read without having to explore the API.
The text was updated successfully, but these errors were encountered: